Skip to content
Snippets Groups Projects

Make raft_conf_change testable

Merged Make raft_conf_change testable
test-confchange into master
All threads resolved!
Merged Yaroslav Dynnikov requested to merge test-confchange into master
All threads resolved!
  1. Move it to the governor module. It's final destination is there.
  2. Change function signature to make it testable
  3. Provide the first several tests.

Merge request reports

Pipeline #13255 passed

Stage: test

Pipeline passed for cb9577fc on test-confchange

Approved by
Georgy Moshkin

Merged by Georgy MoshkinGeorgy Moshkin 2 years ago (Nov 8, 2022 9:44am UTC)

Loading

Pipeline #13257 passed

Stage: test

Pipeline passed for cb9577fc on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Yaroslav Dynnikov added 4 commits

    added 4 commits

    • cde75457 - 1 commit from branch master
    • 3e58bd6f - chore: move raft_conf_change to the governor module
    • 28a934e1 - chore: change raft_conf_change signature
    • e5f18588 - test: add few conf_change unit tests

    Compare with previous version

    Toggle commit list
  • Yaroslav Dynnikov added 1 commit

    added 1 commit

    • cb9577fc - test: add the first few conf_change unit tests

    Compare with previous version

  • Georgy Moshkin approved this merge request

    approved this merge request

  • Georgy Moshkin resolved all threads

    resolved all threads

  • Georgy Moshkin merged

    merged

    • Please register or sign in to reply