From 72ea1734bf540040e3984b4d1367d44a1c23a39c Mon Sep 17 00:00:00 2001 From: Patrick Allwood Date: Mon, 19 Oct 2026 16:30:13 +0100 Subject: [PATCH 1/2] ci: Fix failing integration test on windows --- .../src/DbContexts/PersistedGrantDbContext.cs | 34 +++++++++---------- .../src/Options/OperationalStoreOptions.cs | 16 ++++----- .../PushedAuthorizationRequestStoreTests.cs | 1 + 3 files changed, 25 insertions(+), 26 deletions(-) diff --git a/src/EntityFramework.Storage/src/DbContexts/PersistedGrantDbContext.cs b/src/EntityFramework.Storage/src/DbContexts/PersistedGrantDbContext.cs index a2d75c471..3a3dc1ff6 100644 --- a/src/EntityFramework.Storage/src/DbContexts/PersistedGrantDbContext.cs +++ b/src/EntityFramework.Storage/src/DbContexts/PersistedGrantDbContext.cs @@ -77,6 +77,22 @@ public PersistedGrantDbContext(DbContextOptions options, OperationalStoreOptions /// The keys. /// public DbSet Keys { get; set; } + + /// + /// Gets or sets the server side sessions. + /// + /// + /// The server side sessions. + /// + public DbSet ServerSideSessions { get; set; } + + /// + /// Gets or sets the pushed authorization requests. + /// + /// + /// The pushed authorization requests. + /// + public DbSet PushedAuthorizationRequests { get; set; } /// /// Saves the changes. @@ -106,22 +122,4 @@ protected override void OnModelCreating(ModelBuilder modelBuilder) base.OnModelCreating(modelBuilder); } - - //Schema compatibility, placeholders unused - - /// - /// Gets or sets the server side sessions. - /// - /// - /// The server side sessions. - /// - public DbSet ServerSideSessions { get; set; } - - /// - /// Gets or sets the pushed authorization requests. - /// - /// - /// The pushed authorization requests. - /// - public DbSet PushedAuthorizationRequests { get; set; } } \ No newline at end of file diff --git a/src/EntityFramework.Storage/src/Options/OperationalStoreOptions.cs b/src/EntityFramework.Storage/src/Options/OperationalStoreOptions.cs index 59bd5095b..4829b3327 100644 --- a/src/EntityFramework.Storage/src/Options/OperationalStoreOptions.cs +++ b/src/EntityFramework.Storage/src/Options/OperationalStoreOptions.cs @@ -33,6 +33,14 @@ public class OperationalStoreOptions: StoreOptions /// public TableConfiguration ServerSideSessions { get; set; } = new("ServerSideSessions"); + /// + /// Gets or sets the pushed authorization requests table configuration. + /// + /// + /// The pushed authorization requests config. + /// + public TableConfiguration PushedAuthorizationRequests { get; set; } = new("PushedAuthorizationRequests"); + /// /// Gets or sets a value indicating whether stale entries will be automatically cleaned up from the database. /// This is implemented by periodically connecting to the database (according to the TokenCleanupInterval) from the hosting application. @@ -68,12 +76,4 @@ public class OperationalStoreOptions: StoreOptions /// The keys table config. /// public TableConfiguration Keys { get; set; } = new("Keys"); - - /// - /// Gets or sets the pushed authorization requests table configuration. - /// - /// - /// The pushed authorization requests config. - /// - public TableConfiguration PushedAuthorizationRequests { get; set; } = new("PushedAuthorizationRequests"); } \ No newline at end of file diff --git a/src/EntityFramework.Storage/test/IntegrationTests/Stores/PushedAuthorizationRequestStoreTests.cs b/src/EntityFramework.Storage/test/IntegrationTests/Stores/PushedAuthorizationRequestStoreTests.cs index 4e4658679..a6504ea62 100644 --- a/src/EntityFramework.Storage/test/IntegrationTests/Stores/PushedAuthorizationRequestStoreTests.cs +++ b/src/EntityFramework.Storage/test/IntegrationTests/Stores/PushedAuthorizationRequestStoreTests.cs @@ -122,6 +122,7 @@ private async Task StoreRequest(DbContextOptions option DateTime storedExpires) { await using var context = new PersistedGrantDbContext(options, StoreOptions); + await context.Database.EnsureCreatedAsync(); context.PushedAuthorizationRequests .Add(new PushedAuthorizationRequest() From daf3c71186ab0519d4845dad0869b03d2398f4aa Mon Sep 17 00:00:00 2001 From: Patrick Allwood Date: Mon, 19 Oct 2026 18:19:46 +0100 Subject: [PATCH 2/2] test: failing test due to log level change --- .../Endpoints/PushedAuthorization/PushedAuthorizationTests.cs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/Open.IdentityServer/test/Open.IdentityServer.UnitTests/Endpoints/PushedAuthorization/PushedAuthorizationTests.cs b/src/Open.IdentityServer/test/Open.IdentityServer.UnitTests/Endpoints/PushedAuthorization/PushedAuthorizationTests.cs index 6755ea4e0..ce8dcea9b 100644 --- a/src/Open.IdentityServer/test/Open.IdentityServer.UnitTests/Endpoints/PushedAuthorization/PushedAuthorizationTests.cs +++ b/src/Open.IdentityServer/test/Open.IdentityServer.UnitTests/Endpoints/PushedAuthorization/PushedAuthorizationTests.cs @@ -72,7 +72,7 @@ public async Task ProcessAsync_should_log_end_processing() var _ = await sut.ProcessAsync(context); - logger.Verify(x => x.Log(LogLevel.Trace, It.IsAny(), It.Is((v, t) => v.ToString()!.Contains("End processing pushed authorization request")), It.IsAny(), (Func)It.IsAny()), Times.Once); + logger.Verify(x => x.Log(LogLevel.Debug, It.IsAny(), It.Is((v, t) => v.ToString()!.Contains("End processing pushed authorization request")), It.IsAny(), (Func)It.IsAny()), Times.Once); } [Fact]