Skip to content

Conversation

@mandel-macaque
Copy link
Contributor

There are a number of tests that fail on device. We have an issue for the corlib tests (mono/mono#13641 ) and we do not have one for the icrosoft.CSharp_xunit-test.dll because it does not happen on 2019-02 yet we have issue
mono/mono#13639 that is related with that dll.

Added the ignore, which can be later removed on the new mono 2019-02
since the issues do not happen there.

Fixes: https://github.com/xamarin/maccore/issues/1495
Added the required ignores to get the devices green and workaround mono
issue mono/mono#13641
@mandel-macaque mandel-macaque added the run-bcl-tests Run the BCL tests label Mar 25, 2019
@monojenkins
Copy link
Collaborator

Build failure
Build succeeded
API Diff (from stable)
API Diff (from PR only) (no change)
Generator Diff (no change)
🔥 Test run failed 🔥

Test results

1 tests failed, 0 tests skipped, 218 tests passed.

Failed tests

  • [xUnit] Mono SystemLinqXunit/watchOS - simulator/Debug: Crashed

@mandel-macaque
Copy link
Contributor Author

Failure is a known issue: https://github.com/xamarin/maccore/issues/581

@mandel-macaque mandel-macaque merged commit 45f3e64 into dotnet:master Mar 26, 2019
@mandel-macaque mandel-macaque deleted the ignore-for-devices branch March 26, 2019 11:25
VincentDondain pushed a commit to VincentDondain/xamarin-macios that referenced this pull request Apr 24, 2019
* [XHarness] Ignore some tests that fail on device.

Added the ignore, which can be later removed on the new mono 2019-02
since the issues do not happen there.

Fixes: xamarin/maccore#1495

* [XHarness] Ignore corlib tets failing on devices.

Added the required ignores to get the devices green and workaround mono
issue mono/mono#13641
VincentDondain added a commit that referenced this pull request May 9, 2019
…5980)

* [tests] Don't run BitcodeNotSupported tests on watchOS. (#5744)

Ref: mono/mono#13007

* [tests] Check all architectures when verifying public symbols. (#5745)

This also means updating the whitelisted symbols.

* [XHarness] Don't run BitcodeNotSupported BCL tests on watchOS. (#5750)

* [xharness] Remove bitcode from tvOS apps. (#5781)

We don't need the bitcode for testing, it makes test apps bigger (so big that
the Apple TV might refuse to install them).

* [XHarness] Reduce the number of test applications for the BCL tests. (#5768)

This commit improves the state of the BCL testing in the following ways:

1. Improve the device tets running. Less apps, faster results.
2. WatchOS apps are left as they were to ensure that we do not have deplouyment/run issues.

We now support the ignore files per assembly name to simplify the
tracking of the ignored tests. All

* [XHarness] Ensure we do not crash if the framework node is missing. (#5792)

* [xharness] Fix BuildOnly logic (#5813)

#4884 introduced the logic of only building certain `RunTestTask`.
This was meant to disable iOS Extensions as part of a fix to xamarin/maccore#1008.
However this didn't quite work and iOS extensions were still running (and failing).
The reason being that `BuildOnly` was set to a `RunDeviceTask` that's added to a list which is then given to `CreateTestVariations` which creates new instances of `RunDeviceTask`.
We now propagate `BuildOnly` to the new variation instance.

* [XHarness] Fix IntermediateOutputPath getting inserted in the wrong place. (#5841)

* Ignore tests for devices (#5807)

* [XHarness] Ignore some tests that fail on device.

Added the ignore, which can be later removed on the new mono 2019-02
since the issues do not happen there.

Fixes: xamarin/maccore#1495

* [XHarness] Ignore corlib tets failing on devices.

Added the required ignores to get the devices green and workaround mono
issue mono/mono#13641

* [XHarness] Ignore a failing test on devices. (#5848)

* [XHarness] Update ignore for some failing tests. (#5887)

The mono bump added some new tests that fail on device + release
configurations. This is a known issue by mono and they'll fix it in the
next version but will not be backported to 2018-10.

* [XHarness] Ignore monotouch_Microsoft.CSharp_xunit-test.dll until issue 5868 is fixed. (#5869)

If not ignored, we will have an issue building the test app on device
resulting on a failure.

* [d16-1][xharness] Skip fsharp iOS tests until #5886 is merged

This can be reverted if #5886 makes it in.

* [XHarness] Fix issues building the today extensions for the BCL tests apps. (#5854)

* [d16-1][xharness] Ignore failing device tests because mono backports are impossible

* [d16-1][xharness] Ignore correct interp tests and more BCL group 1

* [d16-1][xharness] Ignore mscorlib interpreter (debug) test

* [tests] Adjust SetupBlockPerfTest to expect at least a 4x speedup instead of 6x

We hit this:

```
Expected: greater than 6
But was: 4.9317220348827826d
```

So adjusting to 4 to avoid those random failures.

See xamarin/maccore#649 for more details.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

run-bcl-tests Run the BCL tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants