feat: TimeGate ADO #529
feat: TimeGate ADO #529
Conversation
WalkthroughThe Andromeda Time Gate module has been enhanced with a new smart contract for managing time gate functionalities, a structured API for instantiation and querying, and improved testing utilities. These updates facilitate efficient interactions within the Andromeda ecosystem and provide a robust framework for time-based operations. Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant Contract
User->>Contract: Instantiate contract with gate settings
Contract-->>User: Confirmation of instantiation
User->>Contract: Set new gate time
Contract-->>User: Confirmation of gate time update
User->>Contract: Query current gate addresses
Contract-->>User: Return current gate addresses
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files ignored due to path filters (1)
Cargo.lockis excluded by!**/*.lock
Files selected for processing (12)
- contracts/modules/andromeda-time-gate/.cargo/config (1 hunks)
- contracts/modules/andromeda-time-gate/Cargo.toml (1 hunks)
- contracts/modules/andromeda-time-gate/examples/schema.rs (1 hunks)
- contracts/modules/andromeda-time-gate/src/contract.rs (1 hunks)
- contracts/modules/andromeda-time-gate/src/lib.rs (1 hunks)
- contracts/modules/andromeda-time-gate/src/mock.rs (1 hunks)
- contracts/modules/andromeda-time-gate/src/state.rs (1 hunks)
- contracts/modules/andromeda-time-gate/src/testing/mock.rs (1 hunks)
- contracts/modules/andromeda-time-gate/src/testing/mod.rs (1 hunks)
- contracts/modules/andromeda-time-gate/src/testing/tests.rs (1 hunks)
- packages/andromeda-modules/src/lib.rs (1 hunks)
- packages/andromeda-modules/src/time_gate.rs (1 hunks)
Files skipped from review due to trivial changes (3)
- contracts/modules/andromeda-time-gate/.cargo/config
- contracts/modules/andromeda-time-gate/examples/schema.rs
- contracts/modules/andromeda-time-gate/src/testing/mod.rs
Additional comments not posted (50)
packages/andromeda-modules/src/lib.rs (1)
4-4: Module declaration looks good.The new module
time_gateis correctly declared and aligns with the existing structure.contracts/modules/andromeda-time-gate/src/lib.rs (4)
1-1: Module declaration looks good.The module
contractis correctly declared and aligns with the overall structure.
2-2: Module declaration looks good.The module
stateis correctly declared and aligns with the overall structure.
3-4: Module declaration looks good.The module
testingis correctly declared and conditionally compiled for tests.
6-7: Module declaration looks good.The module
mockis correctly declared and conditionally compiled for non-WASM32 architectures with thetestingfeature.contracts/modules/andromeda-time-gate/src/state.rs (3)
1-2: Imports look good.The imports are correct and necessary for the functionality.
4-4: Constant declaration looks good.The constant
GATE_ADDRESSESis correctly declared as anItemwith the key "gate_addresses".
5-5: Constant declaration looks good.The constant
GATE_TIMEis correctly declared as anItemwith the key "gate_time".contracts/modules/andromeda-time-gate/Cargo.toml (6)
1-6: Package metadata looks good.The package metadata is correctly defined.
7-9: Crate type definition looks good.The crate type is correctly defined as both
cdylibandrlib.
10-16: Feature flags look good.The feature flags are correctly defined and useful for debugging and testing.
18-27: Dependencies look good.The dependencies are correctly defined and appropriate for the project.
28-31: Target-specific dependencies look good.The target-specific dependencies are correctly defined and appropriate for non-WASM targets.
32-33: Dev-dependencies look good.The dev-dependencies are correctly defined and appropriate for development purposes.
contracts/modules/andromeda-time-gate/src/testing/mock.rs (8)
1-4: Imports look good.The imports are correctly defined and appropriate for the mock implementations.
5-13: Additional imports look good.The additional imports are correctly defined and appropriate for the mock implementations.
15-35:proper_initializationfunction looks good.The function correctly initializes the mock dependencies and instantiates the contract.
37-45:set_gate_timefunction looks good.The function correctly sets the gate time using the
ExecuteMsg::SetGateTimemessage.
47-55:update_gate_addressesfunction looks good.The function correctly updates the gate addresses using the
ExecuteMsg::UpdateGateAddressesmessage.
57-63:query_gate_timefunction looks good.The function correctly queries the gate time using the
QueryMsg::GetGateTimemessage.
65-71:query_gate_addressesfunction looks good.The function correctly queries the gate addresses using the
QueryMsg::GetGateAddressesmessage.
73-79:query_pathfunction looks good.The function correctly queries the path using the
QueryMsg::GetPathByCurrentTimemessage.packages/andromeda-modules/src/time_gate.rs (5)
1-4: Imports look good.The imports are correctly defined and appropriate for the "TimeGate ADO" feature.
5-10:InstantiateMsgstruct looks good.The struct correctly includes the necessary fields for instantiation.
12-21:ExecuteMsgenum looks good.The enum correctly includes the necessary executable messages.
23-33:QueryMsgenum looks good.The enum correctly includes the necessary query messages.
35-49:GateTimeandGateAddressesstructs look good.The structs correctly include the necessary fields for date, time, and addresses.
contracts/modules/andromeda-time-gate/src/mock.rs (7)
1-11: LGTM! Imports and module attributes are appropriate.The imports and module attributes are necessary for the mock implementation and are correctly used.
13-14: LGTM! Struct definition and macro usage are appropriate.The struct
MockTimeGateand themock_adomacro are correctly used for the mock implementation.
16-38: LGTM! Instantiation method is correct.The
instantiatemethod correctly creates and instantiates theMockTimeGatecontract.
40-53: LGTM! Execution method for setting gate time is correct.The
execute_set_gate_timemethod correctly handles the execution of theSetGateTimecontract function.
55-68: LGTM! Execution method for updating gate addresses is correct.The
execute_update_gate_addressesmethod correctly handles the execution of theUpdateGateAddressescontract function.
70-86: LGTM! Query methods are correct.The
query_gate_time,query_gate_addresses, andquery_pathmethods correctly handle querying the contract states.
89-106: LGTM! Helper functions are correct.The helper functions
mock_andromeda_time_gateandmock_time_gate_instantiate_msgare correctly implemented and necessary for the mock implementation.contracts/modules/andromeda-time-gate/src/testing/tests.rs (5)
1-8: LGTM! Imports are appropriate.The imports are necessary for the unit tests and are correctly used.
10-45: LGTM! Instantiation test is correct.The
test_instantiationfunction correctly verifies the instantiation of the TimeGate ADO and the initial state.
47-104: LGTM! Set gate time test is correct.The
test_set_gate_timefunction correctly verifies the execution of theSetGateTimefunction and the updated state.
106-148: LGTM! Update gate addresses test is correct.The
test_update_gate_addressesfunction correctly verifies the execution of theUpdateGateAddressesfunction and the updated state.
150-169: LGTM! Query path test is correct.The
test_query_pathfunction correctly verifies the execution of theGetPathByCurrentTimequery and the expected result.contracts/modules/andromeda-time-gate/src/contract.rs (11)
1-18: LGTM! Imports and module attributes are appropriate.The imports and module attributes are necessary for the contract implementation and are correctly used.
20-22: LGTM! Constant definitions are appropriate.The constants for the contract name and version are necessary for versioning and are correctly defined.
24-52: LGTM! Instantiation function is correct.The
instantiatefunction correctly initializes the contract state and ensures the gate time is valid.
54-68: LGTM! Execution function is correct.The
executefunction correctly routes the execution to the appropriate handler.
70-92: LGTM! Handle execute function is correct.The
handle_executefunction correctly handles the execution of the provided message.
94-122: LGTM! Set gate time execution function is correct.The
execute_set_gate_timefunction correctly handles the execution of theSetGateTimecontract function.
124-150: LGTM! Update gate addresses execution function is correct.The
execute_update_gate_addressessfunction correctly handles the execution of theUpdateGateAddressescontract function.
152-160: LGTM! Query function is correct.The
queryfunction correctly routes the query to the appropriate handler.
162-165: LGTM! Get gate addresses query function is correct.The
get_gate_addressesfunction correctly handles querying the gate addresses.
167-170: LGTM! Get gate time query function is correct.The
get_gate_timefunction correctly handles querying the gate time.
199-202: LGTM! Migration function is correct.The
migratefunction correctly handles the migration of the contract state.
|
Please address Code Rabbit's comments and replace |
There was a problem hiding this comment.
Actionable comments posted: 0
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (2)
- contracts/modules/andromeda-time-gate/src/contract.rs (1 hunks)
- packages/andromeda-modules/src/time_gate.rs (1 hunks)
Files skipped from review as they are similar to previous changes (2)
- contracts/modules/andromeda-time-gate/src/contract.rs
- packages/andromeda-modules/src/time_gate.rs
There was a problem hiding this comment.
Actionable comments posted: 0
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files ignored due to path filters (1)
Cargo.lockis excluded by!**/*.lock
Files selected for processing (9)
- CHANGELOG.md (1 hunks)
- DEADJOE (1 hunks)
- contracts/modules/andromeda-time-gate/src/contract.rs (1 hunks)
- contracts/modules/andromeda-time-gate/src/mock.rs (1 hunks)
- contracts/modules/andromeda-time-gate/src/state.rs (1 hunks)
- contracts/modules/andromeda-time-gate/src/testing/mock.rs (1 hunks)
- contracts/modules/andromeda-time-gate/src/testing/mod.rs (1 hunks)
- contracts/modules/andromeda-time-gate/src/testing/tests.rs (1 hunks)
- packages/andromeda-modules/src/time_gate.rs (1 hunks)
Files skipped from review due to trivial changes (2)
- DEADJOE
- contracts/modules/andromeda-time-gate/src/testing/mod.rs
Files skipped from review as they are similar to previous changes (6)
- contracts/modules/andromeda-time-gate/src/contract.rs
- contracts/modules/andromeda-time-gate/src/mock.rs
- contracts/modules/andromeda-time-gate/src/state.rs
- contracts/modules/andromeda-time-gate/src/testing/mock.rs
- contracts/modules/andromeda-time-gate/src/testing/tests.rs
- packages/andromeda-modules/src/time_gate.rs
Additional comments not posted (1)
CHANGELOG.md (1)
19-19: Entry for "Time Gate ADO" looks good.The entry is correctly formatted and consistent with the rest of the file.
|
Is there a way we can make this cycle through a vector of addresses? Say every hour it moves to the next address in the vector, when it reaches the end of the vector it cycles back to the start. |
It can be implemented. |
There was a problem hiding this comment.
Actionable comments posted: 15
Outside diff range, codebase verification and nitpick comments (3)
contracts/modules/andromeda-time-gate/src/mock.rs (2)
14-15: Consider adding documentation comments.Adding documentation comments to
MockTimeGateand its methods would improve code readability and maintainability by explaining their purpose and usage./// MockTimeGate is a mock implementation for testing the TimeGate contract. pub struct MockTimeGate(Addr);
120-123: Add documentation formock_andromeda_time_gate.Adding a comment to describe the purpose of
mock_andromeda_time_gatewould improve code readability./// Creates a mock contract for the Andromeda Time Gate. pub fn mock_andromeda_time_gate() -> Box<dyn Contract<Empty>> {contracts/modules/andromeda-time-gate/src/contract.rs (1)
105-131: Add logging for update operations.Consider adding logging for operations like
execute_update_cycle_start_timeto facilitate debugging and tracking contract state changes.log::info!("Updating cycle start time to {:?}", cycle_start_time);
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files ignored due to path filters (1)
Cargo.lockis excluded by!**/*.lock
Files selected for processing (6)
- contracts/modules/andromeda-time-gate/src/contract.rs (1 hunks)
- contracts/modules/andromeda-time-gate/src/mock.rs (1 hunks)
- contracts/modules/andromeda-time-gate/src/state.rs (1 hunks)
- contracts/modules/andromeda-time-gate/src/testing/mock.rs (1 hunks)
- contracts/modules/andromeda-time-gate/src/testing/tests.rs (1 hunks)
- packages/andromeda-modules/src/time_gate.rs (1 hunks)
Additional comments not posted (5)
contracts/modules/andromeda-time-gate/src/state.rs (1)
5-7: Verify the integration of storage items.Ensure that
GATE_ADDRESSES,CYCLE_START_TIME, andTIME_INTERVALare correctly used within the contract logic to store and retrieve data as intended.Verification successful
Storage Items Properly Integrated in Contract Logic
The storage items
GATE_ADDRESSES,CYCLE_START_TIME, andTIME_INTERVALare correctly used within the contract logic. They are imported intocontract.rsand utilized for saving and retrieving data, indicating proper integration into the contract's functionality.
- File:
contracts/modules/andromeda-time-gate/src/contract.rs- Usage: Methods like
saveandloadare used with these items, confirming their role in managing contract state.Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify the usage of storage items in the codebase. # Test: Search for the usage of each storage item. Expect: Proper integration within contract logic. rg --type rust 'GATE_ADDRESSES|CYCLE_START_TIME|TIME_INTERVAL'Length of output: 2518
packages/andromeda-modules/src/time_gate.rs (1)
95-95: Fix the error message for second validation.The error message for the second validation should indicate "Second must be less than 60".
- ContractError::InvalidParameter { error: Some("Wrong minute range".to_string())} + ContractError::InvalidParameter { error: Some("Second must be less than 60".to_string())}contracts/modules/andromeda-time-gate/src/testing/mock.rs (1)
1-100: Verify test coverage for mock functions.Ensure that the mock functions provided in this file are adequately covered by unit tests to validate their behavior.
Verification successful
Adequate Test Coverage for Mock Functions Verified
The mock functions in
contracts/modules/andromeda-time-gate/src/testing/mock.rsare adequately covered by unit tests. The test filecontracts/modules/andromeda-time-gate/src/testing/tests.rscontains multiple test functions and assertions that validate the behavior of these mock functions.
- Test functions found:
test_instantiation,test_update_cycle_start_time,test_update_gate_addresses,test_query_current_ado_path_not_started_yet,test_query_current_ado_path.- Assertions are present to ensure expected outcomes.
Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify test coverage for mock functions. # Test: Search for test functions that utilize the mock functions. Expect: Adequate test coverage. rg --type rust 'proper_initialization|update_cycle_start_time|update_gate_addresses|update_time_interval|query_cycle_start_time|query_gate_addresses|query_time_interval|query_current_ado_path'Length of output: 7470
Script:
#!/bin/bash # Description: Check for test functions and assertions involving mock functions. # Search for test functions in the relevant test file rg --type rust 'fn test_' contracts/modules/andromeda-time-gate/src/testing/tests.rs # Search for assertions involving mock functions in the test file rg --type rust 'assert' contracts/modules/andromeda-time-gate/src/testing/tests.rsLength of output: 692
contracts/modules/andromeda-time-gate/src/testing/tests.rs (1)
56-122: Ensure comprehensive error checking.The
test_update_cycle_start_timefunction checks for invalid parameters. Ensure that all possible error scenarios are covered, such as unauthorized access or invalid date formats.// Verify unauthorized access let unauthorized_info = mock_info("unauthorized_user", &[]); let err = update_cycle_start_time( deps.as_mut(), CycleStartTime { year: 2023, month: 1, day: 1, hour: 0, minute: 0, second: 0, }, unauthorized_info.sender.as_ref(), ) .unwrap_err(); assert_eq!(err, ContractError::Unauthorized {});contracts/modules/andromeda-time-gate/src/contract.rs (1)
211-257: Ensure correct time calculations inget_current_ado_path.The function
get_current_ado_pathperforms time calculations. Ensure that all edge cases, such as leap seconds or daylight saving time changes, are correctly handled.// Verify calculations with edge cases
There was a problem hiding this comment.
Actionable comments posted: 0
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (2)
- contracts/modules/andromeda-time-gate/src/contract.rs (1 hunks)
- contracts/modules/andromeda-time-gate/src/testing/tests.rs (1 hunks)
Files skipped from review as they are similar to previous changes (2)
- contracts/modules/andromeda-time-gate/src/contract.rs
- contracts/modules/andromeda-time-gate/src/testing/tests.rs
Version 2.0.0This version of Time Gate ADO returns the moved ado path of the vector each time. When it reaches the end of the vector, it loops back to the starting point. |
|
@coderabbitai review |
Actions performedReview triggered.
|
| } | ||
|
|
||
| #[cw_serde] | ||
| pub struct CycleStartTime { |
There was a problem hiding this comment.
What's the reasoning for not using Timestamp or Expiry here?
There was a problem hiding this comment.
Of course, we can use Timestamp or Expiry.
By the way, I thought that it is a good way to use more visual methods via year, month, day... than only milliseconds.
If you want Timestamp or Expiry for this, I will do so.
There was a problem hiding this comment.
For consistency let's use our Expiry type.
There was a problem hiding this comment.
Actionable comments posted: 1
Outside diff range, codebase verification and nitpick comments (3)
contracts/modules/andromeda-time-gate/src/testing/tests.rs (1)
150-214: LGTM with a suggestion!The test functions
test_query_current_ado_path_not_started_yetandtest_query_current_ado_pathcover the scenarios for querying the current ADO path. Thetest_query_current_ado_pathfunction tests the scenario just after the cycle starts, which is great.Consider adding an additional test case for the scenario just before the cycle starts to ensure the correct behavior:
#[test] fn test_query_current_ado_path_just_before_start() { // ... (setup code) env.block.time = env.block.time.plus_seconds(5000099); let res = query_current_ado_path(deps.as_ref(), env.clone()).unwrap_err(); assert_eq!( res, ContractError::CustomError { msg: "Cycle is not started yet".to_string() } ); }contracts/modules/andromeda-time-gate/src/contract.rs (2)
30-68: Refactor to simplify theinstantiatefunction.Consider the following improvements:
- Remove the unused
cycle_start_time_milliseconsvariable.- Combine the
cycle_start_timeandcycle_start_time_milliseconsvariables into a single struct to simplify the code.Apply this diff to implement the suggested changes:
let (cycle_start_time, _) = get_and_validate_start_time(&env, msg.cycle_start_time)?; -let cycle_start_time_millisecons = match msg.cycle_start_time.clone() { - None => Milliseconds::from_nanos(env.block.time.nanos()), - Some(start_time) => start_time.get_time(&env.block), -}; - let time_interval_seconds = msg.time_interval.unwrap_or(DEFAULT_TIME_INTERVAL); GATE_ADDRESSES.save(deps.storage, &msg.gate_addresses)?; -CYCLE_START_TIME.save( - deps.storage, - &(cycle_start_time, cycle_start_time_millisecons), -)?; +CYCLE_START_TIME.save(deps.storage, &cycle_start_time)?; TIME_INTERVAL.save(deps.storage, &time_interval_seconds)?;
232-260: Refactor to simplify theget_current_ado_pathfunction.Consider the following improvements:
- Remove the unused
cycle_start_time_millisecondsvariable.- Replace the
checked_mul,checked_sub,checked_div, andchecked_remfunctions with the standard arithmetic operators for better readability.Apply this diff to implement the suggested changes:
pub fn get_current_ado_path(deps: Deps, env: Env) -> Result<Addr, ContractError> { let storage = deps.storage; - let (cycle_start_time, cycle_start_time_milliseconds) = CYCLE_START_TIME.load(storage)?; + let cycle_start_time = CYCLE_START_TIME.load(storage)?; let gate_addresses = GATE_ADDRESSES.load(storage)?; let time_interval = TIME_INTERVAL.load(storage)?; ensure!( cycle_start_time.is_expired(&env.block), ContractError::CustomError { msg: "Cycle is not started yet".to_string() } ); let current_time_nanos = env.block.time.nanos(); - let cycle_start_nanos = cycle_start_time_milliseconds.nanos(); + let cycle_start_nanos = cycle_start_time.nanos(); - let time_interval_nanos = time_interval.checked_mul(1_000_000_000).unwrap(); + let time_interval_nanos = time_interval * 1_000_000_000; let gate_length = gate_addresses.len() as u64; - let time_delta = current_time_nanos.checked_sub(cycle_start_nanos).unwrap(); - let index = time_delta - .checked_div(time_interval_nanos) - .unwrap() - .checked_rem(gate_length) - .unwrap() as usize; + let time_delta = current_time_nanos - cycle_start_nanos; + let index = (time_delta / time_interval_nanos) % gate_length as usize; let current_ado_path = &gate_addresses[index]; let result = current_ado_path.get_raw_address(&deps)?; Ok(result) }
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files ignored due to path filters (1)
Cargo.lockis excluded by!**/*.lock
Files selected for processing (7)
- CHANGELOG.md (1 hunks)
- contracts/modules/andromeda-time-gate/Cargo.toml (1 hunks)
- contracts/modules/andromeda-time-gate/src/contract.rs (1 hunks)
- contracts/modules/andromeda-time-gate/src/state.rs (1 hunks)
- contracts/modules/andromeda-time-gate/src/testing/mock.rs (1 hunks)
- contracts/modules/andromeda-time-gate/src/testing/tests.rs (1 hunks)
- packages/andromeda-modules/src/time_gate.rs (1 hunks)
Additional comments not posted (23)
contracts/modules/andromeda-time-gate/src/state.rs (3)
1-3: LGTM!The code changes are approved.
5-5: LGTM!The code changes are approved.
6-7: LGTM!The code changes are approved.
contracts/modules/andromeda-time-gate/Cargo.toml (5)
1-5: LGTM!The package section is correctly configured with appropriate package name, version, Rust edition, and minimum supported Rust version.
7-8: LGTM!The lib section correctly sets the crate type to
["cdylib", "rlib"], which is appropriate for a Rust library that can be used as a native library and a WebAssembly module.
10-15: LGTM!The features section is correctly configured with appropriate feature flags:
- "backtraces" enables more explicit tests using
cosmwasm-std/backtraces.- "library" disables all instantiate/execute/query exports.
- "testing" enables "cw-multi-test" and "andromeda-testing" for testing purposes.
18-25: LGTM!The dependencies section is correctly configured with appropriate workspace dependencies:
cosmwasm-std,cosmwasm-schema,cw-storage-plus, andcw-utilsare standard CosmWasm dependencies.andromeda-stdandandromeda-modulesare Andromeda-specific dependencies.The empty features array for
andromeda-stdis also correct.
27-32: LGTM!The target-specific dependencies and dev-dependencies sections are correctly configured:
- The
cw-multi-testandandromeda-testingdependencies are correctly marked as optional and only included for non-wasm32 targets.- The
andromeda-appworkspace dependency is correctly included in the dev-dependencies section.packages/andromeda-modules/src/time_gate.rs (3)
12-16: LGTM!The code changes are approved.
20-24: LGTM!The code changes are approved.
29-38: LGTM!The code changes are approved.
CHANGELOG.md (1)
19-19: LGTM!The changelog entry for "Time Gate ADO" is consistent with the provided summary and follows the standard format. The pull request link is helpful for reference.
contracts/modules/andromeda-time-gate/src/testing/mock.rs (5)
19-42: LGTM!The code changes are approved.
44-58: LGTM!The code changes are approved.
60-70: LGTM!The code changes are approved.
72-80: LGTM!The code changes are approved.
82-94: Skipping the existing comment.The existing comment on line 91 is still valid and applicable.
contracts/modules/andromeda-time-gate/src/testing/tests.rs (3)
13-46: LGTM!The test function
test_instantiationcovers the basic instantiation scenario and verifies the gate addresses, cycle start time, and time interval. The code changes are approved.
48-94: LGTM!The test function
test_update_cycle_start_timecomprehensively covers the update scenarios for cycle start time and time interval. It also verifies the error handling for updating with the same cycle start time. The code changes are approved.
96-148: LGTM!The test function
test_update_gate_addressescovers the update scenarios for gate addresses and verifies the error handling for updating with the same gate addresses. The code changes are approved.contracts/modules/andromeda-time-gate/src/contract.rs (3)
70-84: LGTM!The code changes are approved.
204-213: LGTM!The code changes are approved.
152-176: Fix typo in the comment.The comment mentions
execute_update_gate_addressessbut the function name isexecute_update_gate_addresses. Remove the extra 's' at the end of the comment.
Motivation
A request comes in, the ADO checks if the current time/date and if it’s before a specific time it returns the path ADO1, if it’s after that time, then ADO2.
Implementation
Testing
Unit test cases are added to codebase.
Version Changes
The version is set as 1.0.0
Summary by CodeRabbit
New Features
Testing Enhancements
Module Structure
Data Integrity