Skip to content

test: cover the interpretations - #1841

Merged
ianna merged 3 commits into
scikit-hep:mainfrom
ariostas:test/interpretation-coverage
Nov 12, 2026
Merged

ianna merged 3 commits into
scikit-hep:mainfrom
ariostas:test/interpretation-coverage

Conversation

@ariostas

Copy link
Copy Markdown
Member

🤖 AI text below 🤖

This replaces #1829, which was merged by mistake into test/existing-tests instead of main, so its changes never reached main. The content is the same, rebased on main.

Part of the split of #1814 (the list is in #1834). These are the new tests of #1814 for the interpretations (containers, AwkwardForth, identification, the Python language, libraries, objects and arrays), apart from the regression tests of its fixes, which are in the fix PRs. All of them pass on main.

It adds test files and deletes test_library_equality_is_safe of test_0016, which test_names_and_equality replaces.

New tests from scikit-hep#1814, apart from the regression tests of its fixes.

Assisted-by: Claude Code:claude-opus-5-5
The check that a sequence isn't equal to a string passed for anything but
True. The test of Node.__str__ expected its wrong indentation, which scikit-hep#1824
fixes; it is now there, with the right one. The test of Library equality in
test_0016 is deleted here, as test_names_and_equality covers it.

Assisted-by: Claude Code:claude-opus-5-5

@ianna ianna left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@ariostas -- Thanks!

@ianna
ianna enabled auto-merge (squash) November 12, 2026 16:06
@github-actions github-actions Bot added the type/test PR title type: test (set automatically) label Nov 12, 2026
@ianna
ianna merged commit 04bf980 into scikit-hep:main Nov 12, 2026
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type/test PR title type: test (set automatically)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants