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

Port 'JcwGen-Tests' to .NET. #7949

Merged
merged 1 commit into from
Apr 13, 2023
Merged

Port 'JcwGen-Tests' to .NET. #7949

merged 1 commit into from
Apr 13, 2023

Conversation

jpobst
Copy link
Contributor

@jpobst jpobst commented Apr 11, 2023

Context: #7794

Convert tests/CodeGen-Binding/Xamarin.Android.JcwGen-Tests to .NET.

This ports 35 of the 41 tests in Xamarin.Android.JcwGen-Tests.
The remaining tests require additional projects to be ported to .NET
to run successfully or require additional custom MSBuild logic from
Xamarin.Android.JcwGen-Tests to be ported.

We are choosing to not prioritize the remaining tests at this point
and committing what we already have.

The unported tests are tracked in #7794.

@jpobst
Copy link
Contributor Author

jpobst commented Apr 12, 2023

I redid this to do the conversion "in-place" now that we no longer need to keep the Classic version around.

This PR replaces #7682.

@jpobst jpobst marked this pull request as ready for review April 12, 2023 15:00
@jonpryor jonpryor merged commit 82161e0 into main Apr 13, 2023
@jonpryor jonpryor deleted the jcwgen-net2 branch April 13, 2023 17:33
grendello added a commit to grendello/xamarin-android that referenced this pull request Apr 14, 2023
* main:
  Bump to xamarin/Java.Interop/main@554d819 (dotnet#7951)
  [Microsoft.Android.Sdk.ILLink] fix crash when TZ changes (dotnet#7956)
  [tests] Port 'Xamarin.Android.JcwGen-Tests.JcwGen-Tests' to .NET (dotnet#7949)
  [Xamarin.Android.Build.Tasks] remove `pdb2mdb` (dotnet#7950)
  [ci] Add some extra params to configure the test templates (dotnet#7955)
  Convert `/tools` and `/build-tools` projects from `net472` to `$(DotNetStableTargetFramework)` (dotnet#7943)
  [Xamarin.Android.Build.Tasks] fix cases of missing `@(Reference)` (dotnet#7947)
  Bump com.android.tools:r8 from 4.0.52 to 8.0.40 (dotnet#7934)
  Bump to xamarin/Java.Interop/main@a172402 (dotnet#7944)
  [Xamarin.Android] Remove OpenTK, sqlite-xamarin, System.EnterpriseServices. (dotnet#7940)
  [ci] Stop building classic test suites. (dotnet#7938)
  Bumping to the correct monodroid commit
  Trying to bump monodroid to run debugger-tests
  Pass timeout to runtime
grendello added a commit to grendello/xamarin-android that referenced this pull request Apr 17, 2023
* main:
  Bump to xamarin/Java.Interop/main@554d819 (dotnet#7951)
  [Microsoft.Android.Sdk.ILLink] fix crash when TZ changes (dotnet#7956)
  [tests] Port 'Xamarin.Android.JcwGen-Tests.JcwGen-Tests' to .NET (dotnet#7949)
  [Xamarin.Android.Build.Tasks] remove `pdb2mdb` (dotnet#7950)
  [ci] Add some extra params to configure the test templates (dotnet#7955)
@github-actions github-actions bot locked and limited conversation to collaborators Jan 22, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants