Skip to content

Commit

Permalink
Try fixing the new unhandled exception test (#91714)
Browse files Browse the repository at this point in the history
The test is failing under NativeAOT in runtime-extra-platforms legs. Our test infra insists on createdump usage. We don't have one for NativeAOT testing.

```
      BEGIN EXECUTION
      /datadisks/disk1/work/A57B093F/p/nativeaottest.sh /datadisks/disk1/work/A57B093F/w/AE63091E/e/baseservices/exceptions/unhandled/unhandled/ unhandled.dll ''
      "DOTNET_DbgEnableMiniDump is set and the createdump binary does not exist: /datadisks/disk1/work/A57B093F/w/AE63091E/e/baseservices/exceptions/unhandled/unhandled/native/createdump"
      "Unhandled exception. System.Exception: Test"
      "   at TestUnhandledException.Program.Main(String[] args) + 0xbb"
      ""
      Missing Unhandled exception header
      Expected: 100
      Actual: 102
      END EXECUTION - FAILED
```
  • Loading branch information
MichalStrehovsky committed Sep 7, 2023
1 parent 25414bb commit 1aecc38
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions src/tests/baseservices/exceptions/unhandled/unhandled.cs
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,9 @@ static int Main(string[] args)

Process testProcess = new Process();

// We don't need to trigger createdump logic.
testProcess.StartInfo.Environment.Remove("DOTNET_DbgEnableMiniDump");

testProcess.StartInfo.FileName = Environment.ProcessPath;
testProcess.StartInfo.Arguments = Environment.CommandLine + " throw";
testProcess.StartInfo.RedirectStandardError = true;
Expand Down

0 comments on commit 1aecc38

Please sign in to comment.