Skip to content

fix managed tests on linux - #93

Merged
jmquigs merged 1 commit into
masterfrom
claude/managed-code-tests-linux-ek8v0r
Sep 9, 2026
Merged

jmquigs merged 1 commit into
masterfrom
claude/managed-code-tests-linux-ek8v0r

Conversation

@jmquigs

@jmquigs jmquigs commented Sep 9, 2026

Copy link
Copy Markdown
Owner

Authored by Claude (claude-opus-5)

Util.TestDataDir was windows-only in three ways: CodeBase.Replace("file:///","") drops the leading slash of a unix path, the candidate paths used literal backslash separators (ordinary filename chars on unix, so GetFullPath never resolved them), and they were combined against the dll file path rather than its directory. Replaced with an upward search for TestData starting at the assembly's directory, which works under both the VS and dotnet output layouts.

TestMesh and TestMeshTransform also opened "monolithref.mmobj"; the file on disk is MonolithRef.mmobj, which only matters on a case-sensitive filesystem.

All 20 tests now pass under mono on linux.

Claude-Session: https://claude.ai/code/session_0148v6m1593drJ3ZVBJxZzsU

Authored by Claude (claude-opus-5)

Util.TestDataDir was windows-only in three ways: CodeBase.Replace("file:///","")
drops the leading slash of a unix path, the candidate paths used literal
backslash separators (ordinary filename chars on unix, so GetFullPath never
resolved them), and they were combined against the dll file path rather than
its directory.  Replaced with an upward search for TestData starting at the
assembly's directory, which works under both the VS and dotnet output layouts.

TestMesh and TestMeshTransform also opened "monolithref.mmobj"; the file on
disk is MonolithRef.mmobj, which only matters on a case-sensitive filesystem.

All 20 tests now pass under mono on linux.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0148v6m1593drJ3ZVBJxZzsU
@jmquigs
jmquigs merged commit 6994e20 into master Sep 9, 2026
1 check failed
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