Skip to content

Move the create-and-call test case from end-to-end to integration test.#4394

Merged
MathieuDutSik merged 3 commits intolinera-io:mainfrom
MathieuDutSik:move_tests_to_linera_core
Aug 22, 2025
Merged

Move the create-and-call test case from end-to-end to integration test.#4394
MathieuDutSik merged 3 commits intolinera-io:mainfrom
MathieuDutSik:move_tests_to_linera_core

Conversation

@MathieuDutSik
Copy link
Contributor

Motivation

The end-to-end tests are relatively expensive in CI. So, it is advantageous to move them to integration tests.

Proposal

We do the following:

  • We remove the &self of ActiveChain that is not needed.
  • We make the function ActiveChain public to be used in the operational test.
  • We introduce the function to return Bytecode since to instantiate, we cannot use the compressed bytecode.
  • We introduce the test that exercises the create-and-call smart contract.

Test Plan

The CI.

The PR that introduces the create-and-call also addresses the persistence of the contract and service blobs
that have been created. We tested that this integration test fails if the correction is removed.

Release Plan

  • Nothing to do / These changes follow the usual release cycle.

Links

None.

@MathieuDutSik MathieuDutSik marked this pull request as ready for review August 22, 2025 12:01
@MathieuDutSik MathieuDutSik requested a review from afck August 22, 2025 14:00
Copy link
Contributor

@ma2bd ma2bd left a comment

Choose a reason for hiding this comment

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

Awesome, thanks!
Then, maybe we could move everything to linera-sdk/tests ?

@MathieuDutSik MathieuDutSik added this pull request to the merge queue Aug 22, 2025
Merged via the queue into linera-io:main with commit 0def7e8 Aug 22, 2025
28 checks passed
@MathieuDutSik MathieuDutSik deleted the move_tests_to_linera_core branch August 22, 2025 17:39
@MathieuDutSik
Copy link
Contributor Author

Awesome, thanks! Then, maybe we could move everything to linera-sdk/tests?

I looked at this problem of moving the tests to linera-sdk/tests, but it does not seem easy.
We would have to change linera-sdk to be a virtual package and that does not seem a good idea.

What I propose is instead to create a linera-sdk-tests. That would have the same status as examples.
That would already gain something since the wasm code in the create-and-call / publish-read-data-blob
is compiled 4 times for wasm32 but executes 0 times in web assembly.

ma2bd pushed a commit that referenced this pull request Sep 27, 2025
… 4394) (#4671)

Backport of #4394 

## Motivation

Remove the test `create-and-call` that creates spammy output.

## Proposal

Simply remove the `test_create_and_call_end_to_end` from the
`linera-service` tests.
The integration test has not been added.

## Test Plan

CI.

## Release Plan

The change should go into `testnet_conway`.
It affects only the CI.

## Links

None.
MathieuDutSik added a commit that referenced this pull request Oct 7, 2025
… changes. (#4739)

Backports #4406, #4397, #4394, #4366

## Motivation

Several changes to the CI are welcomed and make working easier.
Fewer end-to-end tests for what is not viewed as essential.

## Proposal

Those 4 PRs are backported to TestNet Conway.

## Test Plan

The tests are tested in the same way as in the main.

## Release Plan

Backports to TestNet Conway.

## Links

None.
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.

2 participants