Add review fixes

Signed-off-by: Kitsu <[email protected]>
hyperledger-iroha · Jun 30, 2018 · 7b40ac4 · 7b40ac4
1 parent 8572628
commit 7b40ac4
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/test/module/shared_model/validators/query_validator_test.cpp b/test/module/shared_model/validators/query_validator_test.cpp
@@ -73,11 +73,9 @@ TEST_F(QueryValidatorTest, StatelessValidTest) {
  */
 TEST_F(QueryValidatorTest, UnsetQuery) {
   iroha::protocol::Query qry;
-  auto *meta = new iroha::protocol::QueryPayloadMeta();
-  meta->set_created_time(created_time);
-  meta->set_creator_account_id(account_id);
-  meta->set_query_counter(counter);
-  qry.mutable_payload()->set_allocated_meta(meta);
+  qry.mutable_payload()->mutable_meta()->set_created_time(created_time);
+  qry.mutable_payload()->mutable_meta()->set_creator_account_id(account_id);
+  qry.mutable_payload()->mutable_meta()->set_query_counter(counter);
   auto answer = query_validator.validate(proto::Query(qry));
   ASSERT_TRUE(answer.hasErrors());
 }

diff --git a/test/module/shared_model/validators/transaction_validator_test.cpp b/test/module/shared_model/validators/transaction_validator_test.cpp
@@ -117,6 +117,7 @@ TEST_F(TransactionValidatorTest, UnsetCommand) {
   tx.mutable_payload()->set_creator_account_id(account_id);
   tx.mutable_payload()->set_created_time(created_time);
   auto answer = transaction_validator.validate(proto::Transaction(tx));
+  tx.mutable_payload()->add_commands()->clear_add_asset_quantity();
   ASSERT_TRUE(answer.hasErrors());
 }