From 236999b76a43276560e7e5e1aa7f77ede05fedec Mon Sep 17 00:00:00 2001
From: Gregorius Soedharmo <arkatufus@yahoo.com>
Date: Fri, 29 Mar 2024 21:27:28 +0700
Subject: [PATCH] [GitHub Actions] Make PR validation run on Linux only (#27)

* Make PR validation run on Linux only

* Fix AltAdapterSpec HOCON configuration
---
 .github/workflows/pr_validation.yaml          |  3 +-
 .../EventStoreJournalAltAdapterSpec.cs        | 40 ++++++++++---------
 2 files changed, 23 insertions(+), 20 deletions(-)

diff --git a/.github/workflows/pr_validation.yaml b/.github/workflows/pr_validation.yaml
index d372e17..3506aa1 100644
--- a/.github/workflows/pr_validation.yaml
+++ b/.github/workflows/pr_validation.yaml
@@ -17,9 +17,10 @@ jobs:
     name: Test-${{matrix.os}}
     runs-on: ${{matrix.os}}
 
+    # The docker image for v5.0.9 doesn't exist for windows, we can only test on linux.
     strategy:
       matrix:
-        os: [ubuntu-latest, windows-latest]
+        os: [ubuntu-latest]
 
     steps:
       - name: "Checkout"
diff --git a/src/Akka.Persistence.EventStore.Tests/EventStoreJournalAltAdapterSpec.cs b/src/Akka.Persistence.EventStore.Tests/EventStoreJournalAltAdapterSpec.cs
index b23f5d2..49974f1 100644
--- a/src/Akka.Persistence.EventStore.Tests/EventStoreJournalAltAdapterSpec.cs
+++ b/src/Akka.Persistence.EventStore.Tests/EventStoreJournalAltAdapterSpec.cs
@@ -1,6 +1,7 @@
 using Akka.Configuration;
 using Akka.Persistence.TCK.Journal;
 using Xunit;
+using Xunit.Abstractions;
 
 namespace Akka.Persistence.EventStore.Tests
 {
@@ -10,31 +11,32 @@ public class EventStoreJournalAltAdapterSpec : JournalSpec, IClassFixture<Databa
     {
         protected override bool SupportsRejectingNonSerializableObjects { get; } = false;
 
-        public EventStoreJournalAltAdapterSpec(DatabaseFixture databaseFixture)
-            : base(CreateSpecConfig(databaseFixture), nameof(EventStoreJournalAltAdapterSpec))
+        public EventStoreJournalAltAdapterSpec(DatabaseFixture databaseFixture, ITestOutputHelper output)
+            : base(CreateSpecConfig(databaseFixture), nameof(EventStoreJournalAltAdapterSpec), output)
         {
             Initialize();
         }
        
         private static Config CreateSpecConfig(DatabaseFixture databaseFixture)
         {
-            var specString = @"
-                akka.test.single-expect-default = 10s
-                akka.persistence {
-                    publish-plugin-commands = on
-                    journal {
-                        plugin = ""akka.persistence.journal.eventstore""
-                        eventstore {
-                            class = ""Akka.Persistence.EventStore.Journal.EventStoreJournal, Akka.Persistence.EventStore""
-                            connection-string = """ + databaseFixture.ConnectionString + @"""
-                            connection-name = ""EventStoreJournalSpec""
-                            read-batch-size = 500
-                            adapter = ""Akka.Persistence.EventStore.Tests.AltAdapter, Akka.Persistence.EventStore.Tests""
-                        }
-                    }
-                }";
-
-            return ConfigurationFactory.ParseString(specString);
+            return ConfigurationFactory.ParseString( 
+$$"""
+akka.loglevel = DEBUG
+akka.test.single-expect-default = 10s
+akka.persistence {
+   publish-plugin-commands = on
+   journal {
+       plugin = "akka.persistence.journal.eventstore"
+       eventstore {
+           class = "Akka.Persistence.EventStore.Journal.EventStoreJournal, Akka.Persistence.EventStore"
+           connection-string = "{{databaseFixture.ConnectionString}}"
+           connection-name = "EventStoreJournalSpec"
+           read-batch-size = 500
+           adapter = "{{typeof(AltEventAdapter).AssemblyQualifiedName}}"
+       }
+   }
+}
+""").WithFallback(DefaultConfig);
         }
     }
 }