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

e2e tests: fix docker workflow #2623

Merged
merged 1 commit into from
Apr 29, 2021
Merged

e2e tests: fix docker workflow #2623

merged 1 commit into from
Apr 29, 2021

Conversation

piotr-iohk
Copy link
Contributor

Issue Number

Overview

  • I have added a step for docker workflow to get the latest cardano-wallet bundle before starting tests in order to have cardano-address on the path which is required for recently added e2e shared wallet tests

Comments

This was slight omission from #2619. Other workflows already have it by default.

Tested here on the docker workflow -> https://github.com/input-output-hk/cardano-wallet/actions/runs/784885875

@piotr-iohk piotr-iohk self-assigned this Apr 26, 2021
@piotr-iohk
Copy link
Contributor Author

bors r+

iohk-bors bot added a commit that referenced this pull request Apr 26, 2021
2623: e2e tests: fix docker workflow r=piotr-iohk a=piotr-iohk


# Issue Number

<!-- Put here a reference to the issue that this PR relates to and which requirements it tackles. Jira issues of the form ADP- will be auto-linked. -->


# Overview

<!-- Detail in a few bullet points the work accomplished in this PR -->

- [ ] I have added a step for docker workflow to get the latest cardano-wallet bundle before starting tests in order to have cardano-address on the path which is required for recently added e2e shared wallet tests


# Comments
This was slight omission from #2619. Other workflows already have it by default.

Tested here on the docker workflow -> https://github.com/input-output-hk/cardano-wallet/actions/runs/784885875


Co-authored-by: Piotr Stachyra <piotr.stachyra@iohk.io>
@iohk-bors
Copy link
Contributor

iohk-bors bot commented Apr 26, 2021

Build failed:

This seems to be a corrupted .stack-work cache on Buildkite packet-ipxe-3.

@rvl
Copy link
Contributor

rvl commented Apr 27, 2021

bors r+

iohk-bors bot added a commit that referenced this pull request Apr 27, 2021
2623: e2e tests: fix docker workflow r=rvl a=piotr-iohk


# Issue Number

<!-- Put here a reference to the issue that this PR relates to and which requirements it tackles. Jira issues of the form ADP- will be auto-linked. -->


# Overview

<!-- Detail in a few bullet points the work accomplished in this PR -->

- [ ] I have added a step for docker workflow to get the latest cardano-wallet bundle before starting tests in order to have cardano-address on the path which is required for recently added e2e shared wallet tests


# Comments
This was slight omission from #2619. Other workflows already have it by default.

Tested here on the docker workflow -> https://github.com/input-output-hk/cardano-wallet/actions/runs/784885875


Co-authored-by: Piotr Stachyra <piotr.stachyra@iohk.io>
@iohk-bors
Copy link
Contributor

iohk-bors bot commented Apr 27, 2021

Build failed:

Failures:

  src/Test/Integration/Scenario/API/Shelley/StakePools.hs:344:26:
  1) API Specifications, SHELLEY_STAKE_POOLS, STAKE_POOLS_JOIN_01rewards - Can join a pool, earn rewards and collect them
       While verifying (Status {statusCode = 200, statusMessage = "OK"},Right (ApiWallet {id = ApiT {getApiT = WalletId {getWalletId = 748a99c2328eed6609b525d9733505371bf974f2}}, addressPoolGap = ApiT {getApiT = AddressPoolGap {getAddressPoolGap = 20}}, balance = ApiWalletBalance {available = Quantity {getQuantity = 999998149772}, total = Quantity {getQuantity = 1000005731862}, reward = Quantity {getQuantity = 7582090}}, assets = ApiWalletAssetsBalance {available = ApiT {getApiT = TokenMap (fromList [])}, total = ApiT {getApiT = TokenMap (fromList [])}}, delegation = ApiWalletDelegation {active = ApiWalletDelegationNext {status = Delegating, target = Just (ApiT {getApiT = PoolId {getPoolId = "\187\DC1L\179}u\250\ENQ&\ETX(\194\&5\163\218\226\149\163=\v\166t\165\235\RS>V\142"}}), changesAt = Nothing}, next = []}, name = ApiT {getApiT = WalletName {getWalletName = "Faucet Wallet"}}, passphrase = Just (ApiWalletPassphraseInfo {lastUpdatedAt = 2021-04-27 05:29:56.007366958 UTC}), state = ApiT {getApiT = Ready}, tip = ApiBlockReference {absoluteSlotNumber = ApiT {getApiT = SlotNo 7302}, slotId = ApiSlotId {epochNumber = ApiT {getApiT = EpochNo {unEpochNo = 146}}, slotNumber = ApiT {getApiT = SlotInEpoch {unSlotInEpoch = 2}}}, time = 2021-04-27 05:32:09.4 UTC, block = ApiBlockInfo {height = Quantity {getQuantity = 3623}}}}))
       Waited longer than 90s to resolve action: "Wallet has consumed rewards".
       expected: Quantity {getQuantity = 0}
        but got: Quantity {getQuantity = 7582090}

  To rerun use: --match "/API Specifications/SHELLEY_STAKE_POOLS/STAKE_POOLS_JOIN_01rewards - Can join a pool, earn rewards and collect them/"

#2415

@rvl
Copy link
Contributor

rvl commented Apr 28, 2021

bors r+

iohk-bors bot added a commit that referenced this pull request Apr 28, 2021
2623: e2e tests: fix docker workflow r=rvl a=piotr-iohk


# Issue Number

<!-- Put here a reference to the issue that this PR relates to and which requirements it tackles. Jira issues of the form ADP- will be auto-linked. -->


# Overview

<!-- Detail in a few bullet points the work accomplished in this PR -->

- [ ] I have added a step for docker workflow to get the latest cardano-wallet bundle before starting tests in order to have cardano-address on the path which is required for recently added e2e shared wallet tests


# Comments
This was slight omission from #2619. Other workflows already have it by default.

Tested here on the docker workflow -> https://github.com/input-output-hk/cardano-wallet/actions/runs/784885875


2626: Bump version from 2021-04-08 to 2021-04-28 r=rvl a=rvl



Co-authored-by: Piotr Stachyra <piotr.stachyra@iohk.io>
Co-authored-by: Rodney Lorrimar <rodney.lorrimar@iohk.io>
@iohk-bors
Copy link
Contributor

iohk-bors bot commented Apr 28, 2021

Build failed (retrying...):

iohk-bors bot added a commit that referenced this pull request Apr 28, 2021
2623: e2e tests: fix docker workflow r=rvl a=piotr-iohk


# Issue Number

<!-- Put here a reference to the issue that this PR relates to and which requirements it tackles. Jira issues of the form ADP- will be auto-linked. -->


# Overview

<!-- Detail in a few bullet points the work accomplished in this PR -->

- [ ] I have added a step for docker workflow to get the latest cardano-wallet bundle before starting tests in order to have cardano-address on the path which is required for recently added e2e shared wallet tests


# Comments
This was slight omission from #2619. Other workflows already have it by default.

Tested here on the docker workflow -> https://github.com/input-output-hk/cardano-wallet/actions/runs/784885875


Co-authored-by: Piotr Stachyra <piotr.stachyra@iohk.io>
@iohk-bors
Copy link
Contributor

iohk-bors bot commented Apr 28, 2021

Build failed:

#expected

@piotr-iohk
Copy link
Contributor Author

bors r+

iohk-bors bot added a commit that referenced this pull request Apr 28, 2021
2623: e2e tests: fix docker workflow r=piotr-iohk a=piotr-iohk


# Issue Number

<!-- Put here a reference to the issue that this PR relates to and which requirements it tackles. Jira issues of the form ADP- will be auto-linked. -->


# Overview

<!-- Detail in a few bullet points the work accomplished in this PR -->

- [ ] I have added a step for docker workflow to get the latest cardano-wallet bundle before starting tests in order to have cardano-address on the path which is required for recently added e2e shared wallet tests


# Comments
This was slight omission from #2619. Other workflows already have it by default.

Tested here on the docker workflow -> https://github.com/input-output-hk/cardano-wallet/actions/runs/784885875


Co-authored-by: Piotr Stachyra <piotr.stachyra@iohk.io>
@iohk-bors
Copy link
Contributor

iohk-bors bot commented Apr 28, 2021

Build failed:

  test/unit/Cardano/Wallet/Primitive/Migration/SelectionSpec.hs:128:9: 
  1) Cardano.Wallet.Primitive.Migration.Selection.Cardano.Wallet.Primitive.Migration.SelectionSpec, Creating selections, prop_create
       Falsified (after 1102 tests):
         (*)
         "mockConstraints":
             MockTxConstraints
                 { mockTxCostFunction = MockTxCostFunction
                     { baseCost = Coin 295
                     , sizeCost = Coin 3
                     }
                 , mockTxBaseSize = 931
                 , mockTxInputSize = 4
                 , mockTxOutputMaximumSize = 400
                 , mockTxOutputMaximumTokenQuantity = 1224
                 , mockTxOutputMinimumAdaQuantity = MockTxOutputMinimumAdaQuantity
                     { perOutput = Coin 5
                     , perOutputAsset = Coin 1
                     }
                 , mockTxMaximumSize = 2547
                 }
         
         "correctness":
             SelectionCorrect
         
         "feeExcessExpected":
             Coin 3
         
         Condition violated: "Rebalancing the selection leaves it unchanged"

  To rerun use: --match "/Cardano.Wallet.Primitive.Migration.Selection/Cardano.Wallet.Primitive.Migration.SelectionSpec/Creating selections/prop_create/"

Randomized with seed 99256518

#2630

@rvl
Copy link
Contributor

rvl commented Apr 29, 2021

bors r+

@rvl rvl merged commit 5c414e2 into master Apr 29, 2021
@rvl rvl deleted the piotr/update_e2e_docker_tests branch April 29, 2021 13:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants