Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

non-script adder collator test with mockable overseer #2696

Closed
wants to merge 4 commits into from
Closed

non-script adder collator test with mockable overseer #2696

wants to merge 4 commits into from

Conversation

drahnr
Copy link
Contributor

@drahnr drahnr commented Mar 24, 2021

Immediate goals:

  • unit test of adder collator with multiple validator groups

Extends to:

  • add an overseer mock
    • allow arbitrary message modifications as a filter stage to the real overseer
    • ...

Ref #2513
Ref #2443

@github-actions github-actions bot added the A3-in_progress Pull request is in progress. No review needed at this stage. label Mar 24, 2021
@drahnr drahnr added B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. labels Mar 25, 2021
drahnr added 2 commits March 25, 2021 16:05
@drahnr
add mousetrap runtime
745d493 
It has reduced time of 200ms for integration tests.
@drahnr
foo
5c7b6f3
@drahnr drahnr mentioned this pull request Jun 12, 2021
Merged
2 tasks
@rphmeier
Copy link
Contributor

Quite stale - close or stay open?

@drahnr
Copy link
Contributor Author

drahnr commented Jul 19, 2021

Closing, waaay to stale.

@drahnr drahnr closed this Jul 19, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@bkchr bkchr bkchr left review comments

Assignees
No one assigned
Labels
A3-in_progress Pull request is in progress. No review needed at this stage. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@drahnr @rphmeier @bkchr