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

Failing forc unit test: ok_vec_buffer_ownership #6567

Open
sdankel opened this issue Sep 18, 2024 · 3 comments
Open

Failing forc unit test: ok_vec_buffer_ownership #6567

sdankel opened this issue Sep 18, 2024 · 3 comments
Labels
bug Something isn't working forc-test Everything related to the `forc-test` lib and `forc test` command. testing General testing

Comments

@sdankel
Copy link
Member

sdankel commented Sep 18, 2024

With the test uncommented, forc test --path sway-lib-std results in this error:

Verification failed at ok_vec_buffer_ownership::entry
library {
    entry fn ok_vec_buffer_ownership() -> (), !4 {
        local [u8; 4] __anon_0
        local { u64, ( () | u8 ) } __anon_1
        local { u64, ( () | u8 ) } __anon_2
        local mut { { u64, u64 }, u64 } bytes
        local mut { { u64, u64 }, u64 } bytes_
error: Internal compiler error: Verification failed: Struct field type mismatch: (Element type "u64" versus index type Some("u8")).
Please file an issue on the repository and include the code that triggered this error.

It also fails for ok_bytes_buffer_ownership

When fixed, we should uncomment the tests. Thanks @alfiedotwtf for discovering these

@sdankel
Copy link
Member Author

sdankel commented Sep 18, 2024

@bitzoic @IGI-111 for viz

These tests weren't failing in CI due to #6566

@sdankel sdankel added bug Something isn't working testing General testing forc-test Everything related to the `forc-test` lib and `forc test` command. labels Sep 18, 2024
@sdankel
Copy link
Member Author

sdankel commented Sep 18, 2024

It looks like this is the first instance they failed on master: https://github.com/FuelLabs/sway/actions/runs/10812086601/job/29992938006

From this PR: #6526

cc @xunilrj

@sdankel
Copy link
Member Author

sdankel commented Sep 18, 2024

Confirmed that changing this section of code back fixes the compiler error. I'm not sure if this is the best fix though. Thoughts?

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working forc-test Everything related to the `forc-test` lib and `forc test` command. testing General testing
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant