Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test(runtime): configure lower time bounds behind a feature flag #145

Merged
merged 2 commits into from
Jul 22, 2024

Conversation

th7nder
Copy link
Contributor

@th7nder th7nder commented Jul 18, 2024

Description

Fixes #139

This adds a feature flag testnet for the runtime and sets it as default.
When runtime compiled with testnet feature flag, the order of waiting for deals to happen is reduced to minutes instead of days so we can effectively integration test them.

This will be exposed to the clients.

I run it locally and submitted extrinsics, it works.

Checklist

  • Make sure that you described what this change does.
  • Have you tested this solution?

@th7nder th7nder added the bug Something isn't working label Jul 18, 2024
@th7nder th7nder added this to the Phase 1 milestone Jul 18, 2024
@th7nder th7nder self-assigned this Jul 18, 2024
@th7nder th7nder requested review from jmg-duarte and cernicc and removed request for jmg-duarte July 18, 2024 09:11
@th7nder th7nder added the ready for review Review is needed label Jul 18, 2024
@th7nder th7nder force-pushed the fix/139/configure-time-units-for-testnet branch from 951d473 to e3438fb Compare July 18, 2024 09:14
@th7nder th7nder added ready for review Review is needed and removed ready for review Review is needed labels Jul 18, 2024
@th7nder th7nder force-pushed the fix/139/configure-time-units-for-testnet branch from e3438fb to e3dce27 Compare July 18, 2024 09:15
@th7nder th7nder added ready for review Review is needed and removed ready for review Review is needed labels Jul 18, 2024
@th7nder th7nder force-pushed the fix/139/configure-time-units-for-testnet branch from e3dce27 to 0a2c6cd Compare July 18, 2024 09:21
@th7nder th7nder added ready for review Review is needed and removed ready for review Review is needed labels Jul 18, 2024
cernicc
cernicc previously approved these changes Jul 18, 2024
@th7nder th7nder requested a review from jmg-duarte July 18, 2024 10:46
runtime/src/configs/mod.rs Show resolved Hide resolved
runtime/src/configs/mod.rs Outdated Show resolved Hide resolved
runtime/src/configs/mod.rs Outdated Show resolved Hide resolved
@th7nder th7nder force-pushed the fix/139/configure-time-units-for-testnet branch 2 times, most recently from 2ce3e1c to ec90b3a Compare July 18, 2024 11:35
@th7nder th7nder requested review from jmg-duarte and cernicc July 18, 2024 11:35
@th7nder th7nder added ready for review Review is needed and removed ready for review Review is needed labels Jul 18, 2024
Copy link
Contributor

@jmg-duarte jmg-duarte left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit + Question, after this, it's G2G

pallets/market/src/lib.rs Outdated Show resolved Hide resolved
pallets/market/src/lib.rs Outdated Show resolved Hide resolved
runtime/src/configs/mod.rs Show resolved Hide resolved
@th7nder th7nder force-pushed the fix/139/configure-time-units-for-testnet branch from ec90b3a to 0efab99 Compare July 18, 2024 12:11
@th7nder th7nder removed the ready for review Review is needed label Jul 18, 2024
@th7nder th7nder added the ready for review Review is needed label Jul 18, 2024
@th7nder th7nder requested a review from jmg-duarte July 18, 2024 12:12
cernicc
cernicc previously approved these changes Jul 18, 2024
Copy link
Member

@cernicc cernicc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Go go go :D

@th7nder th7nder added ready for review Review is needed and removed ready for review Review is needed labels Jul 18, 2024
runtime/Cargo.toml Outdated Show resolved Hide resolved
@th7nder th7nder force-pushed the fix/139/configure-time-units-for-testnet branch from 29a3e49 to 4cc5394 Compare July 21, 2024 18:11
@th7nder th7nder added ready for review Review is needed and removed ready for review Review is needed labels Jul 21, 2024
@th7nder th7nder requested review from jmg-duarte and cernicc July 21, 2024 18:12
@cernicc cernicc added ready for review Review is needed and removed ready for review Review is needed labels Jul 22, 2024
@th7nder th7nder added ready for review Review is needed and removed ready for review Review is needed labels Jul 22, 2024
@jmg-duarte jmg-duarte enabled auto-merge July 22, 2024 15:47
@jmg-duarte jmg-duarte disabled auto-merge July 22, 2024 15:47
@jmg-duarte jmg-duarte force-pushed the fix/139/configure-time-units-for-testnet branch from 28ec866 to b317101 Compare July 22, 2024 15:47
@jmg-duarte jmg-duarte enabled auto-merge July 22, 2024 15:47
@jmg-duarte jmg-duarte added ready for review Review is needed and removed ready for review Review is needed labels Jul 22, 2024
@jmg-duarte jmg-duarte merged commit 1e186c3 into develop Jul 22, 2024
3 checks passed
@jmg-duarte jmg-duarte deleted the fix/139/configure-time-units-for-testnet branch July 22, 2024 17:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ready for review Review is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Integration Tests: misconfigured DAYS for Market and Storage Provider pallets prevent from testing
3 participants