Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: correct epoch counting #5749

Open
wants to merge 3 commits into
base: feature-dan2
Choose a base branch
from

add non-decreasing test

7b41d15
Select commit
Loading
Failed to load commit list.
Sign in for the full log view
Open

fix: correct epoch counting #5749

add non-decreasing test
7b41d15
Select commit
Loading
Failed to load commit list.
GitHub Actions / Test Results (Integration tests) failed Apr 16, 2024 in 0s

1 fail, 29 pass in 12m 3s

 2 files  ±0  11 suites  ±0   12m 3s ⏱️ +28s
30 tests ±0  29 ✅  - 1  0 💤 ±0  1 ❌ +1 
31 runs  +1  30 ✅ ±0  0 💤 ±0  1 ❌ +1 

Results for commit 7b41d15. ± Comparison against earlier commit 545dd8c.

Annotations

Check warning on line 0 in /

See this annotation in the file changed.

@github-actions github-actions / Test Results (Integration tests)

1 out of 2 runs failed: Scenario: Zero-conf transactions: tests/features/Mempool.feature:124:3

artifacts/junit-cucumber/cucumber-output-junit.xml [took 1s]
Raw output
Step panicked. Captured output: called `Result::unwrap()` on an `Err` value: Status { code: Unavailable, message: "error trying to connect: tcp connect error: Connection refused (os error 111)", source: Some(tonic::transport::Error(Transport, hyper::Error(Connect, ConnectError("tcp connect error", Os { code: 111, kind: ConnectionRefused, message: "Connection refused" })))) }
  Scenario: Zero-conf transactions
   ✔  Given I have 1 seed nodes
   ✔  When I have a base node SENDER connected to all seed nodes
   ✔  When I mine a block on SENDER with coinbase CB1
   ✔  When I mine a block on SENDER with coinbase CB2
   ✘  When I mine 4 blocks on SENDER
      Step failed:
      Defined: tests/features/Mempool.feature:129:5
      Matched: integration_tests/tests/steps/mining_steps.rs:62:1
      Step panicked. Captured output: called `Result::unwrap()` on an `Err` value: Status { code: Unavailable, message: "error trying to connect: tcp connect error: Connection refused (os error 111)", source: Some(tonic::transport::Error(Transport, hyper::Error(Connect, ConnectError("tcp connect error", Os { code: 111, kind: ConnectionRefused, message: "Connection refused" })))) }
      Client {
          base_nodes: {},
          blocks: {},
          miners: {},
          ffi_wallets: {},
          wallets: {},
          merge_mining_proxies: {},
          chat_clients: [],
          transactions: {},
          wallet_addresses: {},
          utxos: {
              "CB1": KeyManagerOutput {
                  version: V0,
                  value: MicroMinotari(
                      18462816327,
                  ),
                  spending_key_id: Imported {
                      key: 94a2453f07628050ed0caa4981f73b5a3be52cd0573f43a2d24d368df7ca9311,
                  },
                  features: OutputFeatures {
                      version: V0,
                      output_type: Coinbase,
                      maturity: 3,
                      coinbase_extra: [],
                      sidechain_feature: None,
                      range_proof_type: BulletProofPlus,
                  },
                  script: TariScript {
                      script: [
                          PushPubKey(
                              94f52d5a44e52cab413daec8b5ff7b92b347ed7223f67c237725a8dde882d773,
                          ),
                      ],
                  },
                  covenant: Covenant {
                      tokens: [],
                  },
                  input_data: ExecutionStack {
                      items: [],
                  },
                  script_private_key_id: Imported {
                      key: 94f52d5a44e52cab413daec8b5ff7b92b347ed7223f67c237725a8dde882d773,
                  },
                  sender_offset_public_key: eecc2d8aaaad244be27d70b6d9e32b69e80c29a8ab9adfe09500739813032d66,
                  metadata_signature: CommitmentAndPublicKeySignature {
                      ephemeral_commitment: HomomorphicCommitment(
                          0ca6cb6fc4853fbbf06be04df31975d41b925d82ace1dd40c67513c08db7124a,
                      ),
                      ephemeral_pubkey: 18f4ae4a8fb90a3bc0a4e0bc4967c9bc38b8f7a0cf3866df66240fd1ae952727,
                      u_a: RistrettoSecretKey(***),
                      u_x: RistrettoSecretKey(***),
                      u_y: RistrettoSecretKey(***),
                  },
                  script_lock_height: 0,
                  encrypted_data: EncryptedData {
                      data: [
                          117,
                          5,
                          102,
                          250,
                          218,
                          184,
                          134,
                          184,
                          138,
                          136,
                          124,
                          120,
                          8,
                          253,
                          159,
                          10,
                          125,
                          217,
                          32,
                          69,
                          172,
                          97,
                          152,
                          26,
                          34,
                          249,
                          140,
                          110,
                          186,
                          252,
                          99,
                          32,
                          118,
                          241,
                          44,
                          34,
                          105,
                          116,
                          100,
                          30,
                          208,
                          246,
                          196,
                          24,
                          161,
                          81,
                          169,
                          64,
                          57,
                          202,
                          41,
                          156,
                          185,
                          204,
                          32,
                          218,
                          239,
                          231,
                          7,
                          21,
                          59,
                          115,
                          219,
                          132,
                          49,
                          129,
                          206,
                          133,
                          246,
                          103,
                          193,
                          69,
                          56,
                          118,
                          237,
                          214,
                          125,
                          156,
                          38,
                          141,
                      ],
                  },
                  minimum_value_promise: MicroMinotari(
                      0,
                  ),
              },
              "CB2": KeyManagerOutput {
                  version: V0,
                  value: MicroMinotari(
                      18462800098,
                  ),
                  spending_key_id: Imported {
                      key: e6ff6e02172c6ef4af62e9b2779d0d1e8c376506bd70f60509cab6f558a39d47,
                  },
                  features: OutputFeatures {
                      version: V0,
                      output_type: Coinbase,
                      maturity: 4,
                      coinbase_extra: [],
                      sidechain_feature: None,
                      range_proof_type: BulletProofPlus,
                  },
                  script: TariScript {
                      script: [
                          PushPubKey(
                              94f52d5a44e52cab413daec8b5ff7b92b347ed7223f67c237725a8dde882d773,
                          ),
                      ],
                  },
                  covenant: Covenant {
                      tokens: [],
                  },
                  input_data: ExecutionStack {
                      items: [],
                  },
                  script_private_key_id: Imported {
                      key: 94f52d5a44e52cab413daec8b5ff7b92b347ed7223f67c237725a8dde882d773,
                  },
                  sender_offset_public_key: ac3b33695a7c4f8020758a1f73e18ba7794cd5de26b1d73b5f9accd13b171009,
                  metadata_signature: CommitmentAndPublicKeySignature {
                      ephemeral_commitment: HomomorphicCommitment(
                          c871de2648198ca126852029107374c8f637269c4d24c1dbc4eb2f6f32c0fd37,
                      ),
                      ephemeral_pubkey: 8c07b6487db307f779462a1ae86dc39b2dbc6ebd3e385517515af3eda2633240,
                      u_a: RistrettoSecretKey(***),
                      u_x: RistrettoSecretKey(***),
                      u_y: RistrettoSecretKey(***),
                  },
                  script_lock_height: 0,
                  encrypted_data: EncryptedData {
                      data: [
                          151,
                          235,
                          127,
                          40,
                          210,
                          199,
                          238,
                          247,
                          212,
                          74,
                          210,
                          63,
                          125,
                          77,
                          155,
                          0,
                          174,
                          147,
                          45,
                          203,
                          45,
                          85,
                          153,
                          194,
                          68,
                          101,
                          38,
                          103,
                          206,
                          178,
                          253,
                          30,
                          94,
                          214,
                          178,
                          176,
                          98,
                          218,
                          167,
                          125,
                          193,
                          131,
                          23,
                          6,
                          148,
                          226,
                          198,
                          21,
                          100,
                          69,
                          219,
                          35,
                          88,
                          110,
                          159,
                          120,
                          214,
                          213,
                          4,
                          124,
                          185,
                          121,
                          211,
                          249,
                          97,
                          61,
                          54,
                          35,
                          140,
                          30,
                          22,
                          120,
                          83,
                          57,
                          88,
                          56,
                          159,
                          120,
                          94,
                          100,
                      ],
                  },
                  minimum_value_promise: MicroMinotari(
                      0,
                  ),
              },
          },
          output_hash: None,
          pre_image: None,
          wallet_connected_to_base_node: {},
          seed_nodes: [
              "seed_node_0",
          ],
          wallet_tx_ids: {},
          errors: [],
          last_imported_tx_ids: [],
          last_merge_miner_response: Null,
      }

Check notice on line 0 in .github

See this annotation in the file changed.

@github-actions github-actions / Test Results (Integration tests)

30 tests found

There are 30 tests, see "Raw output" for the full list of tests.
Raw output
Scenario: A message is propagated between clients via 3rd party: tests/features/ChatFFI.feature:7:3
Scenario: A message is propagated between side loaded chat and client via 3rd party: tests/features/ChatFFI.feature:109:3
Scenario: A message receives a delivery receipt via FFI: tests/features/ChatFFI.feature:78:3
Scenario: A message receives a read receipt via FFI: tests/features/ChatFFI.feature:86:3
Scenario: As a client I want to receive contact liveness events: tests/features/WalletFFI.feature:73:5
Scenario: As a client I want to retrieve the mnemonic word list for a given language: tests/features/WalletFFI.feature:24:5
Scenario: As a wallet I want to submit a transaction: tests/features/WalletQuery.feature:18:3
Scenario: As a wallet send to a wallet connected to a different base node: tests/features/WalletTransfer.feature:9:3
Scenario: Callback for delivery confirmation received: tests/features/ChatFFI.feature:22:3
Scenario: Callback for new message received: tests/features/ChatFFI.feature:14:3
Scenario: Callback for read confirmation received: tests/features/ChatFFI.feature:31:3
Scenario: Chat shuts down without any errors: tests/features/ChatFFI.feature:63:3
Scenario: Clear out mempool: tests/features/Mempool.feature:46:3
Scenario: Create burn transaction: tests/features/WalletTransactions.feature:411:3
Scenario: Fetches all addresses from FFI conversations: tests/features/ChatFFI.feature:95:3
Scenario: Node rolls back reorg on invalid block: tests/features/Reorgs.feature:63:3
Scenario: Reply to message: tests/features/ChatFFI.feature:68:3
Scenario: Simple block sync: tests/features/Sync.feature:26:3
Scenario: Simple propagation: tests/features/Propagation.feature:26:3
Scenario: Simple reorg to stronger chain: tests/features/Reorgs.feature:8:3
Scenario: Simple reorg with burned output: tests/features/Reorgs.feature:33:3
Scenario: Sync burned output: tests/features/Sync.feature:34:3
Scenario: Syncing node while also mining before tip sync: tests/features/Sync.feature:142:5
Scenario: Transactions are propagated through a network: tests/features/Mempool.feature:8:3
Scenario: Verify UTXO and kernel MMR size in header: tests/features/BlockTemplate.feature:8:1
Scenario: Verify gprc cna create block with more than 1 coinbase: tests/features/BlockTemplate.feature:14:5
Scenario: Wallet recovery with connected base node staying online: tests/features/WalletRecovery.feature:8:5
Scenario: Wallet sending and receiving one-sided transactions: tests/features/WalletTransactions.feature:8:3
Scenario: When a new node joins the network, it receives all peers: tests/features/Sync.feature:77:3
Scenario: Zero-conf transactions: tests/features/Mempool.feature:124:3