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

CLI: integration test, ts bindings #996

Closed
sreuland opened this issue Oct 3, 2023 · 1 comment · Fixed by #973
Closed

CLI: integration test, ts bindings #996

sreuland opened this issue Oct 3, 2023 · 1 comment · Fixed by #973

Comments

@sreuland
Copy link
Contributor

sreuland commented Oct 3, 2023

What problem does your feature solve?

no test coverage to exercise the ts generated from soroban contract bindings typescript --wasm

What would you like to see?

a test that uses soroban contract bindings typescript --wasm ... to generate ts binding from a sample contract like soroba-examples/hello-world or similar and then use javascript to invoke the contract fn from the generated ts and verify invocation results.

What alternatives are there?

generate cli ts bindings and invoke them manually from command line, or wait for soroban-examples-dapp to be updated to same version of cli as it will generate bindings as part of init.

@sreuland
Copy link
Contributor Author

sreuland commented Oct 4, 2023

it looks like @chadoh , has #973 that would establish this test coverage

@mollykarcher mollykarcher moved this from Backlog to Next Sprint Proposal in Platform Scrum Oct 10, 2023
chadoh pushed a commit to AhaLabs/stellar-cli that referenced this issue Oct 16, 2023
What
----

Update the typescript bindings with all the changes that landed in the futurenet-compatible `smartdeploy` branch since `main` stopped working with futurenet.

Why
---

Lots of good improvements and fixes are not yet in `main`, ~including tests for the generated libraries.~ _CI tests will be included in a future PR; reasonable time-sink exceeded for this one.

Closes: stellar#996

Known limitations
-----------------

The newly-added tests are not yet being exercised in CI; you can see the commit history for all the details of how we tried to work around this. At this point, we will:

- wait for stellar/quickstart#507 to get merged & included in a published quickstart image
- retry the `bindings-ts.yml` workflow, which was almost working except that friendbot was not starting properly when quickstart was run _in CI_ (though it does start properly, even if a bit slowly, when quickstart is run locally)
- after a basic run-tests-with-published-quickstart is working, the next step is to build a custom quickstart image using _this repository's_ RPC version, and use that custom quickstart to run the integration tests. This might be able to be accomplished [as described by ChatGPT here](https://chat.openai.com/share/0f355fbe-93b9-4c8d-9c12-7f54846134bd).
chadoh pushed a commit to AhaLabs/stellar-cli that referenced this issue Oct 16, 2023
What
----

Update the typescript bindings with all the changes that landed in the futurenet-compatible `smartdeploy` branch since `main` stopped working with futurenet.

Why
---

Lots of good improvements and fixes are not yet in `main`, ~including tests for the generated libraries.~ _CI tests will be included in a future PR; reasonable time-sink exceeded for this one.

Closes: stellar#996

Known limitations
-----------------

The newly-added tests are not yet being exercised in CI; you can see the commit history for all the details of how we tried to work around this. At this point, we will:

- wait for stellar/quickstart#507 to get merged & included in a published quickstart image
- retry the `bindings-ts.yml` workflow, which was almost working except that friendbot was not starting properly when quickstart was run _in CI_ (though it does start properly, even if a bit slowly, when quickstart is run locally)
- after a basic run-tests-with-published-quickstart is working, the next step is to build a custom quickstart image using _this repository's_ RPC version, and use that custom quickstart to run the integration tests. This might be able to be accomplished [as described by ChatGPT here](https://chat.openai.com/share/0f355fbe-93b9-4c8d-9c12-7f54846134bd).
chadoh pushed a commit to AhaLabs/stellar-cli that referenced this issue Oct 16, 2023
What
----

Update the typescript bindings with all the changes that landed in the futurenet-compatible `smartdeploy` branch since `main` stopped working with futurenet.

Why
---

Lots of good improvements and fixes are not yet in `main`, ~including tests for the generated libraries.~ _CI tests will be included in a future PR; reasonable time-sink exceeded for this one.

Closes: stellar#996

Known limitations
-----------------

The newly-added tests are not yet being exercised in CI; you can see the commit history for all the details of how we tried to work around this. At this point, we will:

- wait for stellar/quickstart#507 to get merged & included in a published quickstart image
- retry the `bindings-ts.yml` workflow, which was almost working except that friendbot was not starting properly when quickstart was run _in CI_ (though it does start properly, even if a bit slowly, when quickstart is run locally)
- after a basic run-tests-with-published-quickstart is working, the next step is to build a custom quickstart image using _this repository's_ RPC version, and use that custom quickstart to run the integration tests. This might be able to be accomplished [as described by ChatGPT here](https://chat.openai.com/share/0f355fbe-93b9-4c8d-9c12-7f54846134bd).
tsachiherman added a commit that referenced this issue Oct 17, 2023
* chore: update ts bindings

What
----

Update the typescript bindings with all the changes that landed in the futurenet-compatible `smartdeploy` branch since `main` stopped working with futurenet.

Why
---

Lots of good improvements and fixes are not yet in `main`, ~including tests for the generated libraries.~ _CI tests will be included in a future PR; reasonable time-sink exceeded for this one.

Closes: #996

Known limitations
-----------------

The newly-added tests are not yet being exercised in CI; you can see the commit history for all the details of how we tried to work around this. At this point, we will:

- wait for stellar/quickstart#507 to get merged & included in a published quickstart image
- retry the `bindings-ts.yml` workflow, which was almost working except that friendbot was not starting properly when quickstart was run _in CI_ (though it does start properly, even if a bit slowly, when quickstart is run locally)
- after a basic run-tests-with-published-quickstart is working, the next step is to build a custom quickstart image using _this repository's_ RPC version, and use that custom quickstart to run the integration tests. This might be able to be accomplished [as described by ChatGPT here](https://chat.openai.com/share/0f355fbe-93b9-4c8d-9c12-7f54846134bd).

* build: bump delta for flaky test

* build: increase deltas in cli simulation test

---------

Co-authored-by: Willem Wyndham <[email protected]>
Co-authored-by: Tsachi Herman <[email protected]>
@github-project-automation github-project-automation bot moved this from Next Sprint Proposal to Done in Platform Scrum Oct 17, 2023
chadoh added a commit to AhaLabs/stellar-cli that referenced this issue Oct 19, 2023
* chore: update ts bindings

What
----

Update the typescript bindings with all the changes that landed in the futurenet-compatible `smartdeploy` branch since `main` stopped working with futurenet.

Why
---

Lots of good improvements and fixes are not yet in `main`, ~including tests for the generated libraries.~ _CI tests will be included in a future PR; reasonable time-sink exceeded for this one.

Closes: stellar#996

Known limitations
-----------------

The newly-added tests are not yet being exercised in CI; you can see the commit history for all the details of how we tried to work around this. At this point, we will:

- wait for stellar/quickstart#507 to get merged & included in a published quickstart image
- retry the `bindings-ts.yml` workflow, which was almost working except that friendbot was not starting properly when quickstart was run _in CI_ (though it does start properly, even if a bit slowly, when quickstart is run locally)
- after a basic run-tests-with-published-quickstart is working, the next step is to build a custom quickstart image using _this repository's_ RPC version, and use that custom quickstart to run the integration tests. This might be able to be accomplished [as described by ChatGPT here](https://chat.openai.com/share/0f355fbe-93b9-4c8d-9c12-7f54846134bd).

* build: bump delta for flaky test

* build: increase deltas in cli simulation test

---------

Co-authored-by: Willem Wyndham <[email protected]>
Co-authored-by: Tsachi Herman <[email protected]>
tsachiherman added a commit that referenced this issue Oct 20, 2023
* chore: update ts bindings

What
----

Update the typescript bindings with all the changes that landed in the futurenet-compatible `smartdeploy` branch since `main` stopped working with futurenet.

Why
---

Lots of good improvements and fixes are not yet in `main`, ~including tests for the generated libraries.~ _CI tests will be included in a future PR; reasonable time-sink exceeded for this one.

Closes: #996

Known limitations
-----------------

The newly-added tests are not yet being exercised in CI; you can see the commit history for all the details of how we tried to work around this. At this point, we will:

- wait for stellar/quickstart#507 to get merged & included in a published quickstart image
- retry the `bindings-ts.yml` workflow, which was almost working except that friendbot was not starting properly when quickstart was run _in CI_ (though it does start properly, even if a bit slowly, when quickstart is run locally)
- after a basic run-tests-with-published-quickstart is working, the next step is to build a custom quickstart image using _this repository's_ RPC version, and use that custom quickstart to run the integration tests. This might be able to be accomplished [as described by ChatGPT here](https://chat.openai.com/share/0f355fbe-93b9-4c8d-9c12-7f54846134bd).

* build: bump delta for flaky test

* build: increase deltas in cli simulation test

---------

Co-authored-by: Willem Wyndham <[email protected]>
Co-authored-by: Tsachi Herman <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants