build: run tests on multiple operating systems (#699)

* build: run tests on multiple operating systems

* test: use environment-specific newline

Author: olavloite

.github/workflows/build-and-test.yml

@@ -9,7 +9,10 @@ on:
 
 jobs:
   unit-tests:
-    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        os: [ubuntu-latest, macos-latest, windows-latest]
+    runs-on: ${{ matrix.os }}
     steps:
     - uses: actions/checkout@v6
     - name: Setup .NET

Google.Cloud.EntityFrameworkCore.Spanner.Tests/MigrationTests/MigrationMockServerTests.cs

@@ -48,11 +48,11 @@ public void TestMigrateUsesDdlBatch()
             var formattedVersion = $"{version.Major}.{version.Minor}.{version.Build}";
             _fixture.SpannerMock.AddOrUpdateStatementResult("SELECT 1", StatementResult.CreateException(MockSpannerService.CreateDatabaseNotFoundException("d1")));
             _fixture.SpannerMock.AddOrUpdateStatementResult(
-                $"INSERT INTO `EFMigrationsHistory` (`MigrationId`, `ProductVersion`)\nVALUES ('''20210309110233_Initial''', '''{formattedVersion}''')",
+                $"INSERT INTO `EFMigrationsHistory` (`MigrationId`, `ProductVersion`){Environment.NewLine}VALUES ('''20210309110233_Initial''', '''{formattedVersion}''')",
                 StatementResult.CreateUpdateCount(1)
             );
             _fixture.SpannerMock.AddOrUpdateStatementResult(
-                $"INSERT INTO `EFMigrationsHistory` (`MigrationId`, `ProductVersion`)\nVALUES ('''20210830_V2''', '''{formattedVersion}''')",
+                $"INSERT INTO `EFMigrationsHistory` (`MigrationId`, `ProductVersion`){Environment.NewLine}VALUES ('''20210830_V2''', '''{formattedVersion}''')",
                 StatementResult.CreateUpdateCount(1)
             );
             using var db = new MockMigrationSampleDbContext(ConnectionString);
@@ -83,7 +83,7 @@ public void TestMigrateUsesDdlBatch()
                         sql => Assert.StartsWith("CREATE TABLE `Concerts`", sql),
                         sql => Assert.StartsWith("CREATE TABLE `Tracks`", sql),
                         sql => Assert.StartsWith("CREATE TABLE `Performances`", sql),
-                        sql => Assert.StartsWith("CREATE TABLE `TicketSales` (\n    `Id` INT64 NOT NULL GENERATED BY DEFAULT AS IDENTITY (BIT_REVERSED_POSITIVE),\n", sql),
+                        sql => Assert.StartsWith($"CREATE TABLE `TicketSales` ({Environment.NewLine}    `Id` INT64 NOT NULL GENERATED BY DEFAULT AS IDENTITY (BIT_REVERSED_POSITIVE),{Environment.NewLine}", sql),
                         sql => Assert.StartsWith("CREATE INDEX `AlbumsByAlbumTitle2`", sql),
                         sql => Assert.StartsWith("CREATE INDEX `Idx_Singers_FullName`", sql),
                         sql => Assert.StartsWith("CREATE NULL_FILTERED INDEX `IDX_TableWithAllColumnTypes_ColDate_ColCommitTS`", sql),
@@ -97,10 +97,10 @@ public void TestMigrateUsesDdlBatch()
                     Assert.Collection(update.Statements,
                         sql => Assert.StartsWith(" DROP INDEX `IDX_TableWithAllColumnTypes_ColDate_ColCommitTS`", sql),
                         sql => Assert.StartsWith("DROP TABLE `TableWithAllColumnTypes`", sql),
-                        sql => Assert.StartsWith("CREATE TABLE `OtherSequenceKind` (\n    `Id` INT64 NOT NULL GENERATED BY DEFAULT AS IDENTITY (NON_EXISTING_KIND),\n", sql),
-                        sql => Assert.StartsWith("CREATE TABLE `NoSequenceKind` (\n    `Id` INT64 NOT NULL GENERATED BY DEFAULT AS IDENTITY,\n", sql),
-                        sql => Assert.StartsWith("CREATE TABLE `GenerationStrategyAlways` (\n    `Id` INT64 NOT NULL GENERATED ALWAYS AS IDENTITY (BIT_REVERSED_POSITIVE),\n", sql),
-                        sql => Assert.StartsWith("CREATE TABLE `AutoIncrement` (\n    `Id` INT64 NOT NULL AUTO_INCREMENT,\n", sql)
+                        sql => Assert.StartsWith($"CREATE TABLE `OtherSequenceKind` ({Environment.NewLine}    `Id` INT64 NOT NULL GENERATED BY DEFAULT AS IDENTITY (NON_EXISTING_KIND),{Environment.NewLine}", sql),
+                        sql => Assert.StartsWith($"CREATE TABLE `NoSequenceKind` ({Environment.NewLine}    `Id` INT64 NOT NULL GENERATED BY DEFAULT AS IDENTITY,{Environment.NewLine}", sql),
+                        sql => Assert.StartsWith($"CREATE TABLE `GenerationStrategyAlways` ({Environment.NewLine}    `Id` INT64 NOT NULL GENERATED ALWAYS AS IDENTITY (BIT_REVERSED_POSITIVE),{Environment.NewLine}", sql),
+                        sql => Assert.StartsWith($"CREATE TABLE `AutoIncrement` ({Environment.NewLine}    `Id` INT64 NOT NULL AUTO_INCREMENT,{Environment.NewLine}", sql)
                     );
                 }
             );
@@ -128,7 +128,7 @@ public async Task TestStartMigrateAsync()
                         new object[] { "20210309110233_Initial", formattedVersion },
                     }));
             _fixture.SpannerMock.AddOrUpdateStatementResult(
-                $"INSERT INTO `EFMigrationsHistory` (`MigrationId`, `ProductVersion`)\nVALUES ('''20210830_V2''', '''{formattedVersion}''')",
+                $"INSERT INTO `EFMigrationsHistory` (`MigrationId`, `ProductVersion`){Environment.NewLine}VALUES ('''20210830_V2''', '''{formattedVersion}''')",
                 StatementResult.CreateUpdateCount(1)
             );
             await using var db = new MockMigrationSampleDbContext(ConnectionString);
@@ -144,10 +144,10 @@ public async Task TestStartMigrateAsync()
                     Assert.Collection(update.Statements,
                         sql => Assert.StartsWith(" DROP INDEX `IDX_TableWithAllColumnTypes_ColDate_ColCommitTS`", sql),
                         sql => Assert.StartsWith("DROP TABLE `TableWithAllColumnTypes`", sql),
-                        sql => Assert.StartsWith("CREATE TABLE `OtherSequenceKind` (\n    `Id` INT64 NOT NULL GENERATED BY DEFAULT AS IDENTITY (NON_EXISTING_KIND),\n", sql),
-                        sql => Assert.StartsWith("CREATE TABLE `NoSequenceKind` (\n    `Id` INT64 NOT NULL GENERATED BY DEFAULT AS IDENTITY,\n", sql),
-                        sql => Assert.StartsWith("CREATE TABLE `GenerationStrategyAlways` (\n    `Id` INT64 NOT NULL GENERATED ALWAYS AS IDENTITY (BIT_REVERSED_POSITIVE),\n", sql),
-                        sql => Assert.StartsWith("CREATE TABLE `AutoIncrement` (\n    `Id` INT64 NOT NULL AUTO_INCREMENT,\n", sql)
+                        sql => Assert.StartsWith($"CREATE TABLE `OtherSequenceKind` ({Environment.NewLine}    `Id` INT64 NOT NULL GENERATED BY DEFAULT AS IDENTITY (NON_EXISTING_KIND),{Environment.NewLine}", sql),
+                        sql => Assert.StartsWith($"CREATE TABLE `NoSequenceKind` ({Environment.NewLine}    `Id` INT64 NOT NULL GENERATED BY DEFAULT AS IDENTITY,{Environment.NewLine}", sql),
+                        sql => Assert.StartsWith($"CREATE TABLE `GenerationStrategyAlways` ({Environment.NewLine}    `Id` INT64 NOT NULL GENERATED ALWAYS AS IDENTITY (BIT_REVERSED_POSITIVE),{Environment.NewLine}", sql),
+                        sql => Assert.StartsWith($"CREATE TABLE `AutoIncrement` ({Environment.NewLine}    `Id` INT64 NOT NULL AUTO_INCREMENT,{Environment.NewLine}", sql)
                     );
                 }
             );