-
Notifications
You must be signed in to change notification settings - Fork 212
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
Shared wallets testing #2619
Shared wallets testing #2619
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great
test/e2e/Gemfile
Outdated
@@ -1,6 +1,7 @@ | |||
source 'https://rubygems.org' | |||
|
|||
gem 'cardano_wallet', '0.3.2' | |||
gem 'cardano_wallet', '0.3.4' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like pinning of all the other deps, but this one could have pessimistic versioning, because we control it.
@@ -1,6 +1,7 @@ | |||
source 'https://rubygems.org' | |||
|
|||
gem 'cardano_wallet', '0.3.2' | |||
gem 'cardano_wallet', '0.3.4' | |||
# gem 'cardano_wallet', path: "~/wb/cardano_wallet" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does gem
provide a command to link in a local package for development?
I remember that python pip had the pip install --editable ~/project
option..
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems there are other options https://stackoverflow.com/questions/26876000/ruby-gem-equivalent-of-pip-install-e, this being one of them.
test/e2e/helpers/utils.rb
Outdated
bins = absolute_path ENV['TESTS_E2E_BINDIR'] | ||
bins = (bins == '' ? bins : "#{bins}/") | ||
bins = '' if is_win? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The standard way of locating executables is by using the PATH
environment variable.
We should almost never need to add paths when spawning commands.
The test framework could add TESTS_E2E_BINDIR
onto the front of PATH
at startup.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
@@ -2,7 +2,7 @@ | |||
|
|||
describe CardanoWallet::Shelley::Wallets do | |||
|
|||
before(:each) do | |||
after(:each) do |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🙂
test/e2e/spec/spec_helper.rb
Outdated
pscript = JSON.parse(script_template) | ||
dscript = JSON.parse(script_template) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The json conversion step isn't needed, is it?
The script template could stay as a ruby dict.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
specifications/api/swagger.yaml
Outdated
@@ -5137,7 +5137,7 @@ paths: | |||
tags: ["Shared Wallets"] | |||
summary: Create | |||
description: | | |||
<p align="right">status: <strong>stable</strong></p> | |||
<p align="right">status: <strong>experimental</strong></p> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should probably upgrade this to "under development", just to leave no doubt.
c4f239a
to
9d60455
Compare
…E_BINDIR to point to executables to test
…the PATH but absolute path is still needed when starting node and wallet as services on Win.
…ng all the blocks for several minutes so wallet is not able to connect within 180s timeout. The only way to clear GH cache is to change key.
9d60455
to
b958f40
Compare
|
||
m24 = mnemonic_sentence(24) | ||
acc_ix = '0H' | ||
acc_xpub = cardano_address_get_acc_xpub(m24, "1852H/1815H/#{acc_ix}") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it is going to be 1854H very soon - just indicating
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep. 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sweat
bors r+ |
2619: Shared wallets testing r=piotr-iohk a=piotr-iohk # Issue Number ADP-660 # Overview <!-- Detail in a few bullet points the work accomplished in this PR --> - [ ] I have added e2e tests using also cardano-addresses from the bundle - [ ] marked shared eps as experimental for the time being # Comments <!-- Additional comments or screenshots to attach if any --> <!-- Don't forget to: ✓ Self-review your changes to make sure nothing unexpected slipped through ✓ Assign yourself to the PR ✓ Assign one or several reviewer(s) ✓ Jira will detect and link to this PR once created, but you can also link this PR in the description of the corresponding ticket ✓ Acknowledge any changes required to the Wiki ✓ Finally, in the PR description delete any empty sections and all text commented in <!--, so that this text does not appear in merge commit messages. --> Co-authored-by: Piotr Stachyra <piotr.stachyra@iohk.io> Co-authored-by: Rodney Lorrimar <rodney.lorrimar@iohk.io>
Build failed:
|
bors r+ |
2619: Shared wallets testing r=piotr-iohk a=piotr-iohk # Issue Number ADP-660 # Overview <!-- Detail in a few bullet points the work accomplished in this PR --> - [ ] I have added e2e tests using also cardano-addresses from the bundle - [ ] marked shared eps as experimental for the time being # Comments <!-- Additional comments or screenshots to attach if any --> <!-- Don't forget to: ✓ Self-review your changes to make sure nothing unexpected slipped through ✓ Assign yourself to the PR ✓ Assign one or several reviewer(s) ✓ Jira will detect and link to this PR once created, but you can also link this PR in the description of the corresponding ticket ✓ Acknowledge any changes required to the Wiki ✓ Finally, in the PR description delete any empty sections and all text commented in <!--, so that this text does not appear in merge commit messages. --> Co-authored-by: Piotr Stachyra <piotr.stachyra@iohk.io> Co-authored-by: Rodney Lorrimar <rodney.lorrimar@iohk.io>
Build failed:
|
bors r+ |
Build succeeded: |
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>
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>
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>
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>
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>
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>
Issue Number
ADP-660
Overview
Comments