Skip to content

fix: Fix TestFullAppSimulation test and *disable* E2E tests#11

Merged
pbukva merged 5 commits intofeat/dropping_sudo_admins_conceptfrom
fix/attempted_ci_fixes
May 8, 2026
Merged

fix: Fix TestFullAppSimulation test and *disable* E2E tests#11
pbukva merged 5 commits intofeat/dropping_sudo_admins_conceptfrom
fix/attempted_ci_fixes

Conversation

@MissingNO57
Copy link
Copy Markdown

@MissingNO57 MissingNO57 commented May 8, 2026

Temporarily Disabling due to dual import of ibc-go module v8@v8.4.0 and v10@v10.3.0, what causes conflicts when each version of the ibc-go module registers the very same set of errors in the GLOBAL scope via errorsmod.Register(...).

This happens due to the fact, that this particular test depends on the
strangelove-ventures/interchaintest/v8@v8.8.1 module, which depends on ibc-go/v8@v8.4.0, however our
fetchai/tokenfactory depends on ibc-go/v10@v10.3.0.

At the moment, this is hard to resolve, since the strangelove-ventures/interchaintest would need to be forked,
and updated so it would depend on CosmWasm/wasmd@v0.61.2, CosmWasm/wasmvm/v3@v3.0.0 and ibc-go/v10@v10.3.0, what is non-trivial task, hence disabling this test temporarily.

MissingNO57 and others added 5 commits May 8, 2026 12:17
Disabling the test due to dual import of ibc-go module v8@v8.4.0 and v10@v10.3.0, what causes conflicts when each version
of ibc-go module registers the very same set of errors in the *GLOBAL* scope via `errorsmod.Register(...)`.
This happens due to the fact, that this particular test depends on the
strangelove-ventures/interchaintest/v8@v8.8.1 module, which depends on ibc-go/v8@v8.4.0, however our
fetchai/tokenfactory depends on ibc-go/v10@v10.3.0.
At the moment, this is hard to resolve, since the strangelove-ventures/interchaintest would need to be forked,
and updated so it would depend on CosmWasm/wasmd@v0.61.2, CosmWasm/wasmvm/v3@v3.0.0 and ibc-go/v10@v10.3.0,
what is non-trivial task, hence disabling this test temporarily.

TODO(pb): The test shall be re-enabled once the issue with the strangelove-ventures/interchaintest module is resolved.
@pbukva pbukva changed the title fix: Attempted simulator fix fix: Fix TestFullAppSimulation and disable E2E tests May 8, 2026
@pbukva pbukva changed the title fix: Fix TestFullAppSimulation and disable E2E tests fix: Fix TestFullAppSimulation test and *disable* E2E tests May 8, 2026
Copy link
Copy Markdown
Collaborator

@pbukva pbukva left a comment

Choose a reason for hiding this comment

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

LGTM

@pbukva pbukva merged commit 52f3b0d into feat/dropping_sudo_admins_concept May 8, 2026
8 checks passed
MissingNO57 added a commit that referenced this pull request May 8, 2026
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