Skip to content

Commit

Permalink
[xharness] Don't generate into the same directory/files for macOS Mod…
Browse files Browse the repository at this point in the history
…ern and macOS Full. (#9121)

* [xharness] Don't generate into the same directory/files for macOS Modern and macOS Full.

This meant that we were overwriting some generated files, which meant that we
were executing the Modern set of tests instead of the Full set of tests for at
least some configurations.

* Add a few ignored tests to the System.Configuration test now that we actually run it.
  • Loading branch information
rolfbjarne authored Jul 21, 2020
1 parent 6bbcfd4 commit 7d47d87
Show file tree
Hide file tree
Showing 2 changed files with 20 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -5,4 +5,22 @@ MonoTests.System.Configuration.AppSettingsSectionTest.TestFile

# Expected: not null
# But was: null
MonoTests.System.Configuration.ConfigurationManagerTest.TestConnectionStringRetrieval
MonoTests.System.Configuration.ConfigurationManagerTest.TestConnectionStringRetrieval

# The UnixSignalTest breaks NullReferenceException handling in the runtime.
# Any NRE will deadlock the process after this test, so just skip it.
MonoTests.Mono.Unix.UnixSignalTest

# Needs RabbitMQ:
# System.TypeInitializationException : The type initializer for '...' threw an exception.
# ----> System.IO.FileNotFoundException : Could not load the file 'Mono.Messaging.RabbitMQ, Version=4.0.0.0, Culture=neutral, PublicKeyToken=0738eb9f132ed756'.
MonoTests.System.Messaging.TransactionMessageTest
MonoTests.System.Messaging.AdminTest
MonoTests.System.Messaging.AsyncPeekTest
MonoTests.System.Messaging.AsyncReceiveTest
MonoTests.System.Messaging.BasicMessagingTest
MonoTests.System.Messaging.PeekTest
MonoTests.System.Messaging.SelectorTest

# System.ArgumentException : The value's length for key 'data source' exceeds it's limit of '128'.
DbLinqTest.DataContextTest.Ctor_FileOrServerOrConnectionIsFilename
Original file line number Diff line number Diff line change
Expand Up @@ -680,7 +680,7 @@ async Task<GeneratedProjects> GenerateMacTestProjectsAsync (IEnumerable<(string
if (!projectDefinition.Validate ())
throw new InvalidOperationException ("xUnit and NUnit assemblies cannot be mixed in a test project.");
// generate the required type registration info
var generatedCodeDir = Path.Combine (GeneratedCodePathRoot, projectDefinition.Name, "mac");
var generatedCodeDir = Path.Combine (GeneratedCodePathRoot, projectDefinition.Name, platform.ToString ());
Directory.CreateDirectory (generatedCodeDir);
var registerTypePath = Path.Combine (generatedCodeDir, "RegisterType-mac.cs");

Expand Down

5 comments on commit 7d47d87

@xamarin-release-manager
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔥 Tests failed catastrophically on VSTS: device tests iOS (Cambridge) 🔥

Not enough free space in the host.

Pipeline on Agent XI-SCARLET-WITCH

@xamarin-release-manager
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔥 Tests failed catastrophically on VSTS: device tests tvOS (DDFun) 🔥

Not enough free space in the host.

Pipeline on Agent XAMTESTMAC10

@xamarin-release-manager
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

❌ Device tests failed on VSTS: device tests tvOS (Cambridge) ❌

Device tests failed on VSTS: device tests tvOS (Cambridge).

Test results

150 tests' device not found, 0 tests passed.

Pipeline on Agent XAMTESTWIN10

@xamarin-release-manager
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Jenkins job (on internal Jenkins) succeeded

Provisioning succeeded
Build succeeded
✅ Packages built successfully

View packages

API Diff (from stable)
API Diff (from PR only) (no change)
Generator Diff (no change)
Test run succeeded

@xamarin-release-manager
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔥 Tests failed catastrophically on VSTS: device tests iOS (DDFun) (no summary found). 🔥

Result file /Users/xamarinqa/azdo/_work/122/s\Reports\TestSummary\TestSummary.md not found.

Pipeline on Agent XAMTESTMAC01

Please sign in to comment.