dotnet / runtime

.NET is a cross-platform runtime for cloud, mobile, desktop, and IoT apps.
https://docs.microsoft.com/dotnet/core/
MIT License
15.48k stars 4.77k forks source link

Tests fail with System.BadImageFormatException #4852

Open richardlford opened 8 years ago

richardlford commented 8 years ago

The following tests fail with a mesage like this on a debug win64 test job:

Unhandled Exception: System.BadImageFormatException: 
Could not load file or assembly '<testname>.exe' or one of its dependencies.
An attempt was made to load a program with an incorrect format.

The failing tests are:

JIT\Directed\tls\mutualrecurthd-tls\mutualrecurthd-tls.cmd
JIT\Directed\tls\test-tls\test-tls.cmd
JIT\Regression\CLR-x86-EJIT\v1-m10\b07847\b07847\b07847.cmd
JIT\Regression\CLR-x86-JIT\V1.2-M01\b03689\b03689\b03689.cmd

category:correctness theme:testing skill-level:beginner cost:medium impact:small

sivarv commented 8 years ago

Please check whether TLS statics are supported by CoreCLR.

LLITCHEV commented 8 years ago

@richardlford The .data tls FieldData = XXXXXX construct is not supported on non-Windows platforms. It is really OS specific and there is no anything like that available on the non Windows platforms. These tests are expected to fail on non-Windows platforms. Please move them to the tests/testsUnsupportedOutsideWindows.txt file and close the issue.

RussKeldorph commented 8 years ago

TLS statics not supported by .NET Core. BadImageFormatException is expected on all platforms. We should replace these tests with a single test that verifies the exception is raised.

BruceForstall commented 7 years ago

@RussKeldorph Do we need to keep this issue open?

RussKeldorph commented 7 years ago

@BruceForstall If my last comment is no longer actionable or relevant, sure, close it. Otherwise, I'd rather keep it open to track the gap in testing.

marcusturewicz commented 4 years ago

These files no longer exist so can this issue be closed?

BruceForstall commented 4 years ago

These tests still all exist in the test tree (but the tests are not built).