add @broken label to cucumber tests that use the constitution fixture

tari-project · Jun 30, 2022 · ef36e04 · ef36e04
1 parent efb343f
commit ef36e04
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/integration_tests/features/ValidatorNode.feature b/integration_tests/features/ValidatorNode.feature
@@ -3,7 +3,8 @@
 
 @dan
 Feature: Validator Node
-    @critical
+    # BROKEN: This is broken because the contract constitution fixture needs to have a contract id created from the contract definition
+    @critical @broken
     Scenario: Publish contract acceptance
         Given I have a seed node NODE1
         And I have wallet WALLET1 connected to all seed nodes
@@ -35,7 +36,8 @@ Feature: Validator Node
         And I mine 9 blocks using wallet WALLET1 on NODE1
         Then wallet WALLET1 will have a successfully mined constitution acceptance transaction for contract DEF1
 
-    @critical
+    # BROKEN: Contract ID is hard coded, but needs to be obtained from the contract definition
+    @critical @broken
     Scenario: Publish contract update proposal acceptance
         Given I have a seed node NODE1
         And I have wallet WALLET1 connected to all seed nodes

diff --git a/integration_tests/features/WalletCli.feature b/integration_tests/features/WalletCli.feature
@@ -152,7 +152,8 @@ Feature: Wallet CLI
         Then wallet WALLET has at least 1 transactions that are all TRANSACTION_STATUS_MINED_CONFIRMED and not cancelled
         Then WALLET is connected to BASE
 
-    @dan @critical
+    # BROKEN: This is broken because the contract constitution fixture needs to have a contract id created from the contract definition
+    @dan @critical @broken
     Scenario: As a user I want to publish a contract constitution via command line
         Given I have a base node BASE
         And I have wallet WALLET connected to base node BASE
@@ -167,7 +168,8 @@ Feature: Wallet CLI
         Then wallet WALLET has at least 2 transactions that are all TRANSACTION_STATUS_MINED_CONFIRMED and not cancelled
         Then WALLET is connected to BASE
 
-    @dan @critical
+    # BROKEN: This is broken because the contract constitution fixture needs to have a contract id created from the contract definition
+    @dan @critical @broken
     Scenario: As a user I want to publish a contract update proposal via command line
         Given I have a base node BASE
         And I have wallet WALLET connected to base node BASE