Skip to content

Clean up fabric_doc_open_revs tests#4191

Merged
nickva merged 1 commit intomainfrom
cleanup-fabric-doc-open-revs
Sep 29, 2022
Merged

Clean up fabric_doc_open_revs tests#4191
nickva merged 1 commit intomainfrom
cleanup-fabric-doc-open-revs

Conversation

@nickva
Copy link
Copy Markdown
Contributor

@nickva nickva commented Sep 29, 2022

Use the TDEF_FE macro to save 2 lines per test.

Use the TDEF_FE macro to save 2 lines per test.
Copy link
Copy Markdown
Contributor

@jaydoane jaydoane left a comment

Choose a reason for hiding this comment

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

Another sweet cleanup!

❯ make eunit apps=fabric suites=fabric_doc_open_revs

======================== EUnit ========================
module 'fabric_doc_open_revs'
  fabric_doc_open_revs:376: -open_doc_revs_test_/0-fun-54- (check_empty_response_not_quorum)...[0.007 s] ok
  fabric_doc_open_revs:377: -open_doc_revs_test_/0-fun-52- (check_basic_response)...[0.005 s] ok
  fabric_doc_open_revs:378: -open_doc_revs_test_/0-fun-50- (check_finish_quorum)...ok
  fabric_doc_open_revs:379: -open_doc_revs_test_/0-fun-48- (check_finish_quorum_newer)...[0.006 s] ok
  fabric_doc_open_revs:380: -open_doc_revs_test_/0-fun-46- (check_no_quorum_on_second)...ok
  fabric_doc_open_revs:381: -open_doc_revs_test_/0-fun-44- (check_done_on_third)...ok
  fabric_doc_open_revs:382: -open_doc_revs_test_/0-fun-42- (check_specific_revs_first_msg)...ok
  fabric_doc_open_revs:383: -open_doc_revs_test_/0-fun-40- (check_revs_done_on_agreement)...ok
  fabric_doc_open_revs:384: -open_doc_revs_test_/0-fun-38- (check_latest_true)...ok
  fabric_doc_open_revs:385: -open_doc_revs_test_/0-fun-36- (check_ancestor_counted_in_quorum)...ok
  fabric_doc_open_revs:386: -open_doc_revs_test_/0-fun-34- (check_not_found_counts_for_descendant)...ok
  fabric_doc_open_revs:387: -open_doc_revs_test_/0-fun-32- (check_worker_error_skipped)...ok
  fabric_doc_open_revs:388: -open_doc_revs_test_/0-fun-30- (check_quorum_only_counts_valid_responses)...ok
  fabric_doc_open_revs:389: -open_doc_revs_test_/0-fun-28- (check_empty_list_when_no_workers_reply)...ok
  fabric_doc_open_revs:390: -open_doc_revs_test_/0-fun-26- (check_node_rev_stored)...ok
  fabric_doc_open_revs:391: -open_doc_revs_test_/0-fun-24- (check_node_rev_store_head_only)...ok
  fabric_doc_open_revs:392: -open_doc_revs_test_/0-fun-22- (check_node_rev_store_multiple)...ok
  fabric_doc_open_revs:393: -open_doc_revs_test_/0-fun-20- (check_node_rev_dont_store_errors)...ok
  fabric_doc_open_revs:394: -open_doc_revs_test_/0-fun-18- (check_node_rev_store_non_errors)...ok
  fabric_doc_open_revs:395: -open_doc_revs_test_/0-fun-16- (check_node_rev_store_concatenate)...ok
  fabric_doc_open_revs:396: -open_doc_revs_test_/0-fun-14- (check_node_rev_store_concantenate_multiple)...ok
  fabric_doc_open_revs:397: -open_doc_revs_test_/0-fun-12- (check_node_rev_unmodified_on_down_or_exit)...ok
  fabric_doc_open_revs:398: -open_doc_revs_test_/0-fun-10- (check_not_found_replies_are_removed_when_doc_found)...ok
  fabric_doc_open_revs:399: -open_doc_revs_test_/0-fun-8- (check_not_found_returned_when_one_of_docs_not_found)...ok
  fabric_doc_open_revs:400: -open_doc_revs_test_/0-fun-6- (check_not_found_returned_when_doc_not_found)...ok
  fabric_doc_open_revs:401: -open_doc_revs_test_/0-fun-4- (check_longer_rev_list_returned)...ok
  fabric_doc_open_revs:402: -open_doc_revs_test_/0-fun-2- (check_longer_rev_list_not_combined)...ok
  fabric_doc_open_revs:403: -open_doc_revs_test_/0-fun-0- (check_not_found_removed_and_longer_rev_list)...ok
  [done in 0.103 s]
=======================================================
  All 28 tests passed.

Code Coverage:
fabric_doc_open_revs :  95%

@nickva nickva merged commit 4f852a6 into main Sep 29, 2022
@nickva nickva deleted the cleanup-fabric-doc-open-revs branch September 29, 2022 16:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants