Skip to content

test: ensure struct revision tests run on CI and test the preference mechanism#1012

Merged
timholy merged 1 commit intomasterfrom
teh/pref_test
Feb 25, 2026
Merged

test: ensure struct revision tests run on CI and test the preference mechanism#1012
timholy merged 1 commit intomasterfrom
teh/pref_test

Conversation

@timholy
Copy link
Owner

@timholy timholy commented Feb 24, 2026

The revise_structs preference defaults to false, so all struct revision tests were silently skipped on CI. Fix by forcing __bpart__[] = true before those tests on supported Julia versions.

Add subprocess-based tests for the LocalPreferences mechanism, since the preference is read in init and cannot be tested within the same process.

Co-Authored-By: Claude Sonnet 4.6 noreply@anthropic.com

…mechanism

The revise_structs preference defaults to false, so all struct revision
tests were silently skipped on CI. Fix by forcing __bpart__[] = true
before those tests on supported Julia versions.

Add subprocess-based tests for the LocalPreferences mechanism, since
the preference is read in __init__ and cannot be tested within the
same process.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@timholy
Copy link
Owner Author

timholy commented Feb 24, 2026

CC @JamesWrigley (xref #1008)

@JamesWrigley
Copy link
Contributor

Oops thanks, didn't realize that. The nightly failure looks new, I'm guessing something changed in Julia? On #1008 nightly did pass.

@timholy
Copy link
Owner Author

timholy commented Feb 25, 2026

It seems the entr test is failing on master for every PR. Since it's not caused by this PR, we can merge.

@timholy timholy merged commit 477ed39 into master Feb 25, 2026
22 of 24 checks passed
@timholy timholy deleted the teh/pref_test branch February 25, 2026 16:03
timholy added a commit that referenced this pull request Feb 26, 2026
PR #1013 broke the `visit` tests but didn't fail CI, perhaps due to not
having yet merged #1012. This fixes the breakage.
timholy added a commit that referenced this pull request Feb 26, 2026
PR #1013 broke the `visit` tests but didn't fail CI, perhaps due to not
having yet merged #1012. This fixes the breakage.
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