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] System.Reflection.Tests.MethodInfoTests.TestFunctionPointers failing with AOT on linux #82272

Closed
radical opened this issue Feb 16, 2023 · 2 comments · Fixed by #82282
Closed
Assignees
Labels
arch-wasm WebAssembly architecture area-System.Reflection disabled-test The test is disabled in source code against the issue test-failure
Milestone

Comments

@radical
Copy link
Member

radical commented Feb 16, 2023

System.Reflection.Tests log:

[22:28:04] fail: [FAIL] System.Reflection.Tests.MethodInfoTests.TestFunctionPointers
[22:28:04] info: System.ArgumentException : Object of type 'System.Reflection.Tests.OtherColorsInt' cannot be converted to type 'System.Reflection.Tests.ColorsShort'.
[22:28:04] info:    at System.Reflection.MethodInvoker.Invoke(Object , IntPtr* , BindingFlags )

System.Reflection.InvokeInterpreted.Tests:

[22:27:07] fail: [FAIL] System.Reflection.Tests.MethodInfoTests.TestFunctionPointers
[22:27:07] info: System.InvalidOperationException : Operation is not valid due to the current state of the object.
[22:27:07] info:    at System.Reflection.MethodInvoker.Invoke(Object , IntPtr* , BindingFlags )

System.Reflection.InvokeEmit.Tests:

[22:26:59] fail: [FAIL] System.Reflection.Tests.MethodInfoTests.TestFunctionPointers
[22:26:59] info: System.ArgumentException : Missing parameter does not have a default value. (Parameter 'parameters')
[22:26:59] info:    at System.Reflection.MethodInvoker.Invoke(Object , IntPtr* , BindingFlags )

This was added in #81006 and didn't get caught because we run only smoke tests for AOT/wasm on PRs.

Known Issue Error Message

Fill the error message using known issues guidance.

{
  "ErrorMessage": "",
  "BuildRetry": false
}

Report

Summary

24-Hour Hit Count 7-Day Hit Count 1-Month Count
0 0 0
@radical radical added arch-wasm WebAssembly architecture blocking-clean-ci Blocking PR or rolling runs of 'runtime' or 'runtime-extra-platforms' test-failure labels Feb 16, 2023
@ghost ghost added the untriaged New issue has not been triaged by the area owner label Feb 16, 2023
@ghost
Copy link

ghost commented Feb 16, 2023

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

Issue Details

System.Reflection.Tests log:

[22:28:04] fail: [FAIL] System.Reflection.Tests.MethodInfoTests.TestFunctionPointers
[22:28:04] info: System.ArgumentException : Object of type 'System.Reflection.Tests.OtherColorsInt' cannot be converted to type 'System.Reflection.Tests.ColorsShort'.
[22:28:04] info:    at System.Reflection.MethodInvoker.Invoke(Object , IntPtr* , BindingFlags )

System.Reflection.InvokeInterpreted.Tests:

[22:27:07] fail: [FAIL] System.Reflection.Tests.MethodInfoTests.TestFunctionPointers
[22:27:07] info: System.InvalidOperationException : Operation is not valid due to the current state of the object.
[22:27:07] info:    at System.Reflection.MethodInvoker.Invoke(Object , IntPtr* , BindingFlags )

System.Reflection.InvokeEmit.Tests:

[22:26:59] fail: [FAIL] System.Reflection.Tests.MethodInfoTests.TestFunctionPointers
[22:26:59] info: System.ArgumentException : Missing parameter does not have a default value. (Parameter 'parameters')
[22:26:59] info:    at System.Reflection.MethodInvoker.Invoke(Object , IntPtr* , BindingFlags )

This was added in #81006 and didn't get caught because we run only smoke tests for AOT/wasm on PRs.

Author: radical
Assignees: -
Labels:

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

Milestone: -

@radical radical changed the title [wasm] System.Reflection.Tests.MethodInfoTests.TestFunctionPointers failing with trimming on linux [wasm] System.Reflection.Tests.MethodInfoTests.TestFunctionPointers failing with AOT on linux Feb 16, 2023
@lewing
Copy link
Member

lewing commented Feb 16, 2023

cc @steveharter

@lewing lewing added this to the 8.0.0 milestone Feb 16, 2023
@ghost ghost removed the untriaged New issue has not been triaged by the area owner label Feb 16, 2023
@ghost ghost added the in-pr There is an active PR which will close this issue when it is merged label Feb 17, 2023
@maraf maraf 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' labels Feb 17, 2023
vargaz added a commit to vargaz/runtime that referenced this issue Feb 20, 2023
vargaz added a commit that referenced this issue Feb 21, 2023
* [mono][wasm] Fix ldftn in AOT mode, it should return a ftndesc.

Fixes #82272.

* Reenable tests.
@ghost ghost removed the in-pr There is an active PR which will close this issue when it is merged label Feb 21, 2023
@ghost ghost locked as resolved and limited conversation to collaborators Mar 23, 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.Reflection disabled-test The test is disabled in source code against the issue test-failure
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants