From ef36e04dafd26147d174acce8b9ba4912ac4acc8 Mon Sep 17 00:00:00 2001 From: Stan Bondi Date: Wed, 29 Jun 2022 17:26:17 +0200 Subject: [PATCH] add @broken label to cucumber tests that use the constitution fixture --- integration_tests/features/ValidatorNode.feature | 6 ++++-- integration_tests/features/WalletCli.feature | 6 ++++-- 2 files changed, 8 insertions(+), 4 deletions(-) diff --git a/integration_tests/features/ValidatorNode.feature b/integration_tests/features/ValidatorNode.feature index ad5f813b28..65f706db9d 100644 --- 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 index 903ca552ba..c4e652fb55 100644 --- 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