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

ink-e2e Test Failures in substrate-contracts-node (v0.27.0) due to Unsupported --ws-port Argument #1850

Merged
merged 5 commits into from Jul 19, 2023

Conversation

varex83
Copy link
Contributor

@varex83 varex83 commented Jul 19, 2023

Summary:

This PR addresses an issue with the latest version of substrate-contracts-node (v0.27.0) wherein the --ws-port argument is not being supported, causing the end-to-end tests to fail. The specific error message produced is: "We should find a port before the reader ends".

@varex83
Copy link
Contributor Author

varex83 commented Jul 19, 2023

I was doing it from the v4.1.0 tag, and didn't know where to merge, please @SkymanOne can you help me with that?

Copy link
Collaborator

@ascjones ascjones left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Please merge master to fix CI

@ascjones ascjones merged commit 6d7810e into use-ink:master Jul 19, 2023
22 checks passed
ascjones pushed a commit that referenced this pull request Aug 22, 2023
…supported --ws-port Argument (#1850)

* Update ink-e2e to the latest substrate-contracts-node

* Apply rust-fmt

* Change logging, to be up-to-date with master branch

* Apply rustfmt-nightly

(cherry picked from commit 6d7810e)
ascjones added a commit that referenced this pull request Aug 23, 2023
…1884)

* Manual backport of #1883: Use `contract-build` `3.2.0`

* ink-e2e Test Failures in substrate-contracts-node (v0.27.0) due to Unsupported --ws-port Argument (#1850)

* Update ink-e2e to the latest substrate-contracts-node

* Apply rust-fmt

* Change logging, to be up-to-date with master branch

* Apply rustfmt-nightly

(cherry picked from commit 6d7810e)

* Remove --ws-port arg

* Update substrate deps

* Manual backports of #1795 and #1855 to update subxt

* Update subxt in example

* Backport: fix e2e test, account for contract initial free balance (#1777)

* Use latest CI image

* 🦖 fix spellcheck for new version, add words to dict (#1836)

* Fix call-runtime tests

* enable call-runtime tests

* ui tests

* `[ink_e2e]` update `Determinism` enum to match Substrate definition (#1781)

* `[ink_e2e]` update `Determinism` enum to match Substrate definition

* fmt

* Fix multi_contract_caller example for new deposit behaviour

* Reenable examples-test step

* Fix integration test

* Remove riscv +stable

* Fix riscv build

---------

Co-authored-by: Bohdan Ohorodnii <35969035+varex83@users.noreply.github.com>
goastler pushed a commit to prosopo/ink_sr25519 that referenced this pull request Aug 31, 2023
…se-ink#1884)

* Manual backport of use-ink#1883: Use `contract-build` `3.2.0`

* ink-e2e Test Failures in substrate-contracts-node (v0.27.0) due to Unsupported --ws-port Argument (use-ink#1850)

* Update ink-e2e to the latest substrate-contracts-node

* Apply rust-fmt

* Change logging, to be up-to-date with master branch

* Apply rustfmt-nightly

(cherry picked from commit 6d7810e)

* Remove --ws-port arg

* Update substrate deps

* Manual backports of use-ink#1795 and use-ink#1855 to update subxt

* Update subxt in example

* Backport: fix e2e test, account for contract initial free balance (use-ink#1777)

* Use latest CI image

* 🦖 fix spellcheck for new version, add words to dict (use-ink#1836)

* Fix call-runtime tests

* enable call-runtime tests

* ui tests

* `[ink_e2e]` update `Determinism` enum to match Substrate definition (use-ink#1781)

* `[ink_e2e]` update `Determinism` enum to match Substrate definition

* fmt

* Fix multi_contract_caller example for new deposit behaviour

* Reenable examples-test step

* Fix integration test

* Remove riscv +stable

* Fix riscv build

---------

Co-authored-by: Bohdan Ohorodnii <35969035+varex83@users.noreply.github.com>
goastler pushed a commit to prosopo/ink_sr25519 that referenced this pull request Aug 31, 2023
…se-ink#1884)

* Manual backport of use-ink#1883: Use `contract-build` `3.2.0`

* ink-e2e Test Failures in substrate-contracts-node (v0.27.0) due to Unsupported --ws-port Argument (use-ink#1850)

* Update ink-e2e to the latest substrate-contracts-node

* Apply rust-fmt

* Change logging, to be up-to-date with master branch

* Apply rustfmt-nightly

(cherry picked from commit 6d7810e)

* Remove --ws-port arg

* Update substrate deps

* Manual backports of use-ink#1795 and use-ink#1855 to update subxt

* Update subxt in example

* Backport: fix e2e test, account for contract initial free balance (use-ink#1777)

* Use latest CI image

* 🦖 fix spellcheck for new version, add words to dict (use-ink#1836)

* Fix call-runtime tests

* enable call-runtime tests

* ui tests

* `[ink_e2e]` update `Determinism` enum to match Substrate definition (use-ink#1781)

* `[ink_e2e]` update `Determinism` enum to match Substrate definition

* fmt

* Fix multi_contract_caller example for new deposit behaviour

* Reenable examples-test step

* Fix integration test

* Remove riscv +stable

* Fix riscv build

---------

Co-authored-by: Bohdan Ohorodnii <35969035+varex83@users.noreply.github.com>
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

2 participants