Adds a non-repeating scheduled reducer test#3233
Conversation
| public static void nonrepeating_test(ReducerContext ctx, NonrepeatingTestArg arg) | ||
| { | ||
| var deltaTime = ctx.Timestamp.TimeDurationSince(arg.prev_time); | ||
| Log.Trace($"Timestamp: {ctx.Timestamp}, Delta time: {deltaTime}"); |
There was a problem hiding this comment.
Where in the test harness do we actually check to see if we get this log? If this log isn't present how does the test fail?
There was a problem hiding this comment.
Update made with new tests actually being executed (in crates/testing/tests/standalone_integration_test.rs).
I've also ensured the tests run in Rust / C# / Typescript. The PR description has been updated with more details on these changes as well.
| .timestamp | ||
| .duration_since(arg.prev_time) | ||
| .expect("arg.prev_time is later than ctx.timestamp... huh?"); | ||
| log::trace!( |
There was a problem hiding this comment.
Same here: Where in the test harness do we actually check to see if we get this log? If this log isn't present how does the test fail?
There was a problem hiding this comment.
You are right, I don't believe those tests had ran as part of CI. They should now be run along side the other tests in crates/testing/tests/standalone_integration_test.rs
bfops
added
release-any
Signed-off-by: Ryan <r.ekhoff@clockworklabs.io>
4 participants
Description of Changes
Adds a non-repeating scheduled reducer integration test to the existing Rust and C# integration tests, in order to address #3213
API and ABI breaking changes
No
Expected complexity level and risk
1
Testing
cargo test -p spacetimedb-testingand all tests passed