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

Add test cases to reproduce issue #2343 #2349

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

crazytonyli
Copy link
Contributor

@crazytonyli crazytonyli commented Dec 10, 2024

I don't want to add failing test cases to the repo. To avoid blocking development and/or CI, the new test cases passes when running cargo test, but fails when adding a specific env: ISSUE_2342=1 cargo test.

Here is an example to verify the tests locally:

// Pass
cargo test --package uniffi-fixture-ext-types-sub-lib

// Panic
ISSUE_2343=1 cargo test --package uniffi-fixture-ext-types-sub-lib

From the crash log:

thread 'thread '<unnamed><unnamed>' panicked at .../uniffi_core/src/ffi/foreigncallbacks.rs:32:18' panicked at :
.../uniffi_core/src/ffi/foreigncallbacks.rs:Foreign pointer not set.  This is likely a uniffi bug.32

@crazytonyli crazytonyli requested a review from a team as a code owner December 10, 2024 08:03
@crazytonyli crazytonyli requested review from badboy and removed request for a team December 10, 2024 08:03
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.

1 participant