Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[wasm] S.R.IS.JavaScript.Tests.JSImportExportTest.JsImportTaskTypes #77334

Closed
radical opened this issue Oct 22, 2022 · 7 comments · Fixed by #88728 or #89676
Closed

[wasm] S.R.IS.JavaScript.Tests.JSImportExportTest.JsImportTaskTypes #77334

radical opened this issue Oct 22, 2022 · 7 comments · Fixed by #88728 or #89676
Assignees
Labels
arch-wasm WebAssembly architecture area-System.Runtime.InteropServices.JavaScript disabled-test The test is disabled in source code against the issue Known Build Error Use this to report build issues in the .NET Helix tab test-failure
Milestone

Comments

@radical
Copy link
Member

radical commented Oct 22, 2022

Build: https://github.com/dotnet/runtime/pull/87959/checks?check_run_id=15010861292

fail: [FAIL] System.Runtime.InteropServices.JavaScript.Tests.JSImportExportTest.JsImportTaskTypes
info: System.ObjectDisposedException : Cannot access a disposed object.
info: Object name: 'value'.
info:    at System.Runtime.InteropServices.JavaScript.Tests.JSImportExportTest.JsImportTaskTypes()
info: --- End of stack trace from previous location ---

Log

cc @pavelsavara

{
  "ErrorMessage": "[FAIL] System.Runtime.InteropServices.JavaScript.Tests.JSImportExportTest.JsImportTaskTypes",
  "BuildRetry": false
}

Known issue validation

Build: 🔎
Result validation: ⚠️ Validation could not be done without an Azure DevOps build URL on the issue. Please add it to the "Build: 🔎" line.
Validation performed at: 7/13/2023 1:57:15 PM UTC

Report

Build Definition Test Pull Request
338245 dotnet/runtime normal-System.Runtime.InteropServices.JavaScript.Tests.WorkItemExecution #87260
338230 dotnet/runtime normal-System.Runtime.InteropServices.JavaScript.Tests.WorkItemExecution #87959
337739 dotnet/runtime WasmTestOnBrowser-System.Runtime.InteropServices.JavaScript.Tests.WorkItemExecution #88794
337641 dotnet/runtime WasmTestOnBrowser-System.Runtime.InteropServices.JavaScript.Tests.WorkItemExecution #88503
337476 dotnet/runtime normal-System.Runtime.InteropServices.JavaScript.Tests.WorkItemExecution #88415

Summary

24-Hour Hit Count 7-Day Hit Count 1-Month Count
0 0 5
@radical radical added arch-wasm WebAssembly architecture blocking-clean-ci Blocking PR or rolling runs of 'runtime' or 'runtime-extra-platforms' test-failure labels Oct 22, 2022
@radical radical added this to the 8.0.0 milestone Oct 22, 2022
@ghost
Copy link

ghost commented Oct 22, 2022

Tagging subscribers to 'arch-wasm': @lewing
See info in area-owners.md if you want to be subscribed.

Issue Details

Build, and log:

fail: [FAIL] System.Runtime.InteropServices.JavaScript.Tests.JSImportExportTest.JsImportTaskTypes
info: System.ObjectDisposedException : Cannot access a disposed object.
info: Object name: 'value'.
info:    at System.Runtime.InteropServices.JavaScript.Tests.JSImportExportTest.JsImportTaskTypes()
info: --- End of stack trace from previous location ---

cc @pavelsavara

Author: radical
Assignees: -
Labels:

arch-wasm, blocking-clean-ci, test-failure

Milestone: 8.0.0

@radical radical changed the title [wasm][aot] S.R.IS.JavaScript.Tests.JSImportExportTest.JsImportTaskTypes failing on V8 [wasm][aot] S.R.IS.JavaScript.Tests.JSImportExportTest.JsImportTaskTypes Oct 26, 2022
@radical
Copy link
Member Author

radical commented Oct 26, 2022

More rolling build hits on chrome, and v8.

@radical
Copy link
Member Author

radical commented Oct 26, 2022

This is happening irrespective of the host OS, and AOT/interpreter.

@pavelsavara
Copy link
Member

I tried to reproduce here, no dice.
Any hints ?

@radical
Copy link
Member Author

radical commented Oct 28, 2022

This is being hit on PRs too.

  • Does running the entire test a hundred times locally help?
  • Or maybe doing the same thing on CI, but running the helix work item many times

@pavelsavara pavelsavara changed the title [wasm][aot] S.R.IS.JavaScript.Tests.JSImportExportTest.JsImportTaskTypes [wasm] S.R.IS.JavaScript.Tests.JSImportExportTest.JsImportTaskTypes Nov 9, 2022
@jkotas jkotas added the Known Build Error Use this to report build issues in the .NET Helix tab label Nov 21, 2022
@jkotas
Copy link
Member

jkotas commented Nov 21, 2022

Hit in #78593

@radical
Copy link
Member Author

radical commented Nov 23, 2022

@radical radical added disabled-test The test is disabled in source code against the issue and removed blocking-clean-ci Blocking PR or rolling runs of 'runtime' or 'runtime-extra-platforms' Known Build Error Use this to report build issues in the .NET Helix tab labels Nov 30, 2022
@ghost ghost added the in-pr There is an active PR which will close this issue when it is merged label Jul 12, 2023
@ghost ghost removed the in-pr There is an active PR which will close this issue when it is merged label Jul 12, 2023
@pavelsavara pavelsavara added the Known Build Error Use this to report build issues in the .NET Helix tab label Jul 13, 2023
@pavelsavara pavelsavara reopened this Jul 13, 2023
pavelsavara added a commit to pavelsavara/runtime that referenced this issue Jul 29, 2023
@ghost ghost added the in-pr There is an active PR which will close this issue when it is merged label Jul 29, 2023
pavelsavara added a commit that referenced this issue Aug 2, 2023
@ghost ghost removed the in-pr There is an active PR which will close this issue when it is merged label Aug 2, 2023
@ghost ghost locked as resolved and limited conversation to collaborators Sep 1, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
arch-wasm WebAssembly architecture area-System.Runtime.InteropServices.JavaScript disabled-test The test is disabled in source code against the issue Known Build Error Use this to report build issues in the .NET Helix tab test-failure
Projects
None yet
3 participants