[PR #877] [MERGED] tests: fix postgres e2e tests #566

New Issue

OVERLORD · 2025-10-07T00:18:36+03:00

OVERLORD commented

2025-10-07 00:18:36 +03:00

📋 Pull Request Information

Original PR: https://github.com/pocket-id/pocket-id/pull/877
Author: @stonith404
Created: 8/24/2025
Status: ✅ Merged
Merged: 8/24/2025
Merged by: @stonith404

Base: main ← Head: tests/fix-postgres-e2e

📝 Commits (8)

3e906f5 add missing environment variables for postgres tests
1fda509 replace invalid uuid
af750a4 fix playwright browser installation
4c7893b use matrix for postgres and sqlite tests
01e6297 Merge branch 'main' into tests/fix-postgres-e2e
a417ca4 fix invalid uuids
f241d52 use uuid for bad user
57bdae4 fix: sort order incorrect for apps when using postgres

📊 Changes

5 files changed (+13 additions, -11 deletions)

View changed files

📝 backend/internal/service/e2etest_service.go (+3 -3)
📝 backend/internal/service/oidc_service.go (+2 -3)
📝 tests/data.ts (+4 -4)
📝 tests/setup/docker-compose-postgres.yml (+3 -0)
📝 tests/specs/oidc.spec.ts (+1 -1)

📄 Description

The Postgres end-to-end tests didn't actually use Postgres but Sqlite 🤦‍♂️.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/pocket-id/pocket-id/pull/877 **Author:** [@stonith404](https://github.com/stonith404) **Created:** 8/24/2025 **Status:** ✅ Merged **Merged:** 8/24/2025 **Merged by:** [@stonith404](https://github.com/stonith404) **Base:** `main` ← **Head:** `tests/fix-postgres-e2e` --- ### 📝 Commits (8) - [`3e906f5`](https://github.com/pocket-id/pocket-id/commit/3e906f5e6d41ea3752429348a302dd5e335b9a3a) add missing environment variables for postgres tests - [`1fda509`](https://github.com/pocket-id/pocket-id/commit/1fda509d3f0c59195cdbff3537a0fe35ee19ed15) replace invalid uuid - [`af750a4`](https://github.com/pocket-id/pocket-id/commit/af750a4ac932b58aa15f1b44352dcf0c522ee03c) fix playwright browser installation - [`4c7893b`](https://github.com/pocket-id/pocket-id/commit/4c7893b521a1bd04aadece29b573555ba31cdbc7) use matrix for postgres and sqlite tests - [`01e6297`](https://github.com/pocket-id/pocket-id/commit/01e6297682dd2dbf389ceec8f48ec70b83605e95) Merge branch 'main' into tests/fix-postgres-e2e - [`a417ca4`](https://github.com/pocket-id/pocket-id/commit/a417ca40433aa28a9569aad893032e0793b2db7f) fix invalid uuids - [`f241d52`](https://github.com/pocket-id/pocket-id/commit/f241d5217f925e1d696c0b5eb1374d57f2a39870) use uuid for bad user - [`57bdae4`](https://github.com/pocket-id/pocket-id/commit/57bdae49b89056a22d8c2438571c1d3d59f3743f) fix: sort order incorrect for apps when using postgres ### 📊 Changes **5 files changed** (+13 additions, -11 deletions) <details> <summary>View changed files</summary> 📝 `backend/internal/service/e2etest_service.go` (+3 -3) 📝 `backend/internal/service/oidc_service.go` (+2 -3) 📝 `tests/data.ts` (+4 -4) 📝 `tests/setup/docker-compose-postgres.yml` (+3 -0) 📝 `tests/specs/oidc.spec.ts` (+1 -1) </details> ### 📄 Description The Postgres end-to-end tests didn't actually use Postgres but Sqlite 🤦‍♂️. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

OVERLORD added the pull-request label 2025-10-07 00:18:36 +03:00

OVERLORD closed this issue

2025-10-07 00:18:37 +03:00

OVERLORD referenced this issue

2025-10-07 00:20:46 +03:00

[PR #636] [MERGED] chore: Add docs link and rename to Federated Client Credentials #693

OVERLORD referenced this issue

2025-10-07 00:20:47 +03:00

[PR #640] feat: allow introspection and device code endpoints to use Federated Client Credentials #695

OVERLORD referenced this issue

2025-10-07 00:21:20 +03:00

[PR #566] [MERGED] feat: JWT bearer assertions for client authentication #725