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

unit test of ParachainHostClaimQueue runtime method fails #4482

Open
axaysagathiya opened this issue Jan 20, 2025 · 1 comment
Open

unit test of ParachainHostClaimQueue runtime method fails #4482

axaysagathiya opened this issue Jan 20, 2025 · 1 comment
Labels
A-debug issue requires detective debug work to figure out what's going wrong. S-runtime issues related to the lib/runtime package. T-bug this issue covers unexpected and/or wrong behaviour.

Comments

@axaysagathiya
Copy link
Contributor

Bug summary

Expected Behavior

  • should get some value if we don't get error.

Current Behavior

  • we don't get any error and still we get empty value.

Possible Solution

  • may be we need more test data. but what do we need?

To Reproduce

run TestInstance_ParachainHostClaimQueue unit test which is added in the PR: #4383.

Log output

> go test ./lib/runtime/wazero/... -v -run TestInstance_ParachainHostClaimQueue
=== RUN   TestInstance_ParachainHostClaimQueue
=== PAUSE TestInstance_ParachainHostClaimQueue
=== CONT  TestInstance_ParachainHostClaimQueue
2025-01-20T20:21:57+05:30 CRITICAL target=runtime::storage message=Corrupted state at `0x94eadf0156a8ad5156507773d0471e4a49f6c9aa90c04982c05388649310f22f`: Error	ext_logging_log_version_1	pkg=runtime module=wazero
  instance_test.go:1756: 
      	Error Trace:	/Users/axaysagathiya/code/github/gossamer/lib/runtime/wazero/instance_test.go:1756
      	Error:      	Should NOT be empty, but was map[]
      	Test:       	TestInstance_ParachainHostClaimQueue
--- FAIL: TestInstance_ParachainHostClaimQueue (3.55s)
FAIL
FAIL	github.com/ChainSafe/gossamer/lib/runtime/wazero	4.246s
FAIL
@axaysagathiya axaysagathiya added S-runtime issues related to the lib/runtime package. T-bug this issue covers unexpected and/or wrong behaviour. labels Jan 20, 2025
@EclesioMeloJunior
Copy link
Member

EclesioMeloJunior commented Jan 20, 2025

@axaysagathiya maybe you can try to use the runtime.WESTEND_RUNTIME_v1017001 for parachains host calls, since it was released Dec 17, 2024 and is the latest release

Just more info: Maybe the runtime you're using for testing does not have the correct on_chain_storage_version, which is 3 and maybe it is retrieving informations from the migration storage which is empty.

@P1sar P1sar added the A-debug issue requires detective debug work to figure out what's going wrong. label Jan 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-debug issue requires detective debug work to figure out what's going wrong. S-runtime issues related to the lib/runtime package. T-bug this issue covers unexpected and/or wrong behaviour.
Projects
None yet
Development

No branches or pull requests

3 participants