feat/#46 add unit test for position follow (#49)

Co-authored-by: Quoc Le <[email protected]>
canfoundation · Dec 8, 2020 · dcbfa57 · dcbfa57
1 parent 9a73523
commit dcbfa57
Show file tree

Hide file tree

Showing 6 changed files with 341 additions and 37 deletions.
diff --git a/community/community.wasm b/community/community.wasm
diff --git a/community/include/community.hpp b/community/include/community.hpp
@@ -556,7 +556,9 @@ CONTRACT community : public contract
             ((v1.position)(v1_position)(v1_position_table))
             ((v1.codeexec)(v1_sole_decision)(v1_code_sole_decision_table))
             ((v1.amenexec)(v1_sole_decision)(v1_amend_sole_decision_table))
-            ((v1.cproposal)(v1_code_proposal)(v1_code_proposals_table))
+            ((v1.filling)(v1_election_rule)(v1_election_table))
+            ((v1.pproposal)(v1_pos_proposal)(v1_posproposal_table))
+            ((v1.candidate)(v1_pos_candidate)(v1_poscandidate_table))
     )
 #endif
 };
diff --git a/community/src/community.cpp b/community/src/community.cpp
@@ -1264,7 +1264,6 @@ ACTION community::createpos(
     vector<name> code_actions;
     code_actions.push_back("configpos"_n);
 
-    // TO DO: create configpos with creator (exec_type = SOLE_DESICION) for code_execution_right amendment_execution_right
     auto configCode = _codes.emplace(ram_payer, [&](auto &row) {
         row.code_id = _codes.available_primary_key();
         row.code_name = PO_Config;
@@ -1666,9 +1665,9 @@ ACTION community::voteforpos(name community_account, uint64_t pos_id, name voter
     check(candidate_itr != _pos_candidate.end(), "ERR::CANDIDATE_NOT_ESIXT::The candidate does not exist");
 
     auto new_voters_detail = candidate_itr->voters;
-    auto voter_detail_itr = std::find(candidate_itr->voters.begin(), candidate_itr->voters.end(), voter);
+    auto voter_detail_itr = std::find(new_voters_detail.begin(), new_voters_detail.end(), voter);
 
-    if (voter_detail_itr == candidate_itr->voters.end())
+    if (voter_detail_itr == new_voters_detail.end())
     {
         if (vote_status == VoteStatus::VOTE) {
             new_voters_detail.push_back(voter);

diff --git a/test/community/community.abi b/test/community/community.abi
@@ -403,34 +403,6 @@
                 }
             ]
         },
-        {
-            "name": "hydraload",
-            "base": "",
-            "fields": [
-                {
-                    "name": "payload",
-                    "type": "hydraload_payload[]"
-                }
-            ]
-        },
-        {
-            "name": "hydraload_payload",
-            "base": "",
-            "fields": [
-                {
-                    "name": "table_name",
-                    "type": "name"
-                },
-                {
-                    "name": "scope",
-                    "type": "name"
-                },
-                {
-                    "name": "row_data",
-                    "type": "bytes"
-                }
-            ]
-        },
         {
             "name": "initadminpos",
             "base": "",
@@ -1277,11 +1249,6 @@
             "type": "execproposal",
             "ricardian_contract": ""
         },
-        {
-            "name": "hydraload",
-            "type": "hydraload",
-            "ricardian_contract": ""
-        },
         {
             "name": "initadminpos",
             "type": "initadminpos",

diff --git a/test/community/community.wasm b/test/community/community.wasm