Skip to content

Add SQL OpenIddict login regression coverage - #1323

Merged
SebastianStehle merged 1 commit into
Squidex:masterfrom
Soumilgit:fix/issue-1316-postgres-openiddict-login
Jul 28, 2026
Merged

Add SQL OpenIddict login regression coverage#1323
SebastianStehle merged 1 commit into
Squidex:masterfrom
Soumilgit:fix/issue-1316-postgres-openiddict-login

Conversation

@Soumilgit

@Soumilgit Soumilgit commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Adds shared EF OpenIddict persistence regression coverage that is generated for Postgres, MySQL, and SQL Server.
  • Verifies authorization/token rows without an application reference can be saved across SQL providers, covering the issue Cannot login after setup when using Postgres DB (v7.22+) #1316 login path after initial admin setup.

Testing

  • dotnet build backend/tests/Squidex.Data.Tests/Squidex.Data.Tests.csproj --no-restore --verbosity:minimal
  • Full Testcontainers execution not run locally: Docker/Testcontainers cannot connect to npipe://./pipe/docker_engine on this machine.

Closes #1316

@Soumilgit

Copy link
Copy Markdown
Contributor Author

Hi @SebastianStehle,
Please review this PR.

@SebastianStehle

Copy link
Copy Markdown
Contributor

Please do not write tests for individual providers, they should work for all databases.

@SebastianStehle SebastianStehle left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please do not write tests for individual providers, they should work for all databases.

@Soumilgit
Soumilgit force-pushed the fix/issue-1316-postgres-openiddict-login branch from 51e8477 to 3305ea8 Compare June 9, 2026 13:57
@Soumilgit Soumilgit changed the title Add Postgres OpenIddict login regression coverage Add SQL OpenIddict login regression coverage Jun 9, 2026
@Soumilgit

Copy link
Copy Markdown
Contributor Author

Please do not write tests for individual providers, they should work for all databases.

Updated, thanks @SebastianStehle . I moved the regression coverage out of the Postgres-specific migration test and into a shared EF OpenIddict test so the generator runs it for Postgres, MySQL, and SQL Server.

@Soumilgit
Soumilgit requested a review from SebastianStehle June 9, 2026 14:02
@SebastianStehle
SebastianStehle merged commit 10f0bf6 into Squidex:master Jul 28, 2026
8 checks passed
@Soumilgit
Soumilgit deleted the fix/issue-1316-postgres-openiddict-login branch August 22, 2026 08:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Cannot login after setup when using Postgres DB (v7.22+)

2 participants