starred/pocket-id
1
0
Fork 0
mirror of https://github.com/pocket-id/pocket-id.git synced 2025-12-06 09:13:19 +03:00
Code Issues 121 Packages Projects Releases 16 Wiki Activity

[PR #466] [MERGED] refactor: add ldap tests #781

New Issue
Open
opened 2025-10-07 00:22:14 +03:00 by OVERLORD · 0 comments
OVERLORD commented 2025-10-07 00:22:14 +03:00
Owner
Copy Link

📋 Pull Request Information

Original PR: https://github.com/pocket-id/pocket-id/pull/466
Author: @kmendell
Created: 4/20/2025
Status: ✅ Merged
Merged: 5/7/2025
Merged by: @stonith404

Base: main ← Head: refactor/add-ldap-test

📝 Commits (10+)

  • 63721ac refactor: add ldap tests
  • 013a442 Merge branch 'main' into refactor/add-ldap-test
  • cd69427 use lldap-cli for api calls
  • 3742a7c use cleaner lldap-cli approach
  • 9564b49 Merge branch 'main' into refactor/add-ldap-test
  • b87b5ce install lldap-cli tools from apt
  • a5cceb1 add missing repo for apt tools
  • 15a1a09 fix schema issues
  • b2fb63b add timout and retry for user additions
  • 2c0f60c use consistant group names

📊 Changes

7 files changed (+309 additions, -47 deletions)

View changed files

📝 .github/workflows/e2e-tests.yml (+56 -2)
📝 backend/internal/bootstrap/e2etest_router_bootstrap.go (+1 -1)
📝 backend/internal/controller/e2etest_controller.go (+10 -0)
📝 backend/internal/service/e2etest_service.go (+59 -3)
📝 frontend/tests/application-configuration.spec.ts (+0 -41)
➕ frontend/tests/ldap.spec.ts (+83 -0)
➕ scripts/tests/setup-lldap.sh (+100 -0)

📄 Description

No description provided

🔄 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/466 **Author:** [@kmendell](https://github.com/kmendell) **Created:** 4/20/2025 **Status:** ✅ Merged **Merged:** 5/7/2025 **Merged by:** [@stonith404](https://github.com/stonith404) **Base:** `main` ← **Head:** `refactor/add-ldap-test` --- ### 📝 Commits (10+) - [`63721ac`](https://github.com/pocket-id/pocket-id/commit/63721ac574a85c3f7d99365fa4d2e0ebd5b454f8) refactor: add ldap tests - [`013a442`](https://github.com/pocket-id/pocket-id/commit/013a44274cb73829c8d37eb24b4cf15f1e7263d9) Merge branch 'main' into refactor/add-ldap-test - [`cd69427`](https://github.com/pocket-id/pocket-id/commit/cd6942783b9892ed7affcef872619a287559375b) use lldap-cli for api calls - [`3742a7c`](https://github.com/pocket-id/pocket-id/commit/3742a7cbc234e498e78f217e66cce964811fc153) use cleaner lldap-cli approach - [`9564b49`](https://github.com/pocket-id/pocket-id/commit/9564b492673d6954690cc7491529066af29e9483) Merge branch 'main' into refactor/add-ldap-test - [`b87b5ce`](https://github.com/pocket-id/pocket-id/commit/b87b5ce712ef9a681fb3d9e354cb4ef8a2402294) install lldap-cli tools from apt - [`a5cceb1`](https://github.com/pocket-id/pocket-id/commit/a5cceb1f44eba6e7c4e5bb15dcee08cdf12eca55) add missing repo for apt tools - [`15a1a09`](https://github.com/pocket-id/pocket-id/commit/15a1a099a5ed42999ddd2f47623669a4ac6b8580) fix schema issues - [`b2fb63b`](https://github.com/pocket-id/pocket-id/commit/b2fb63bb35cfe6368f15b465dbf3bf63417d0db2) add timout and retry for user additions - [`2c0f60c`](https://github.com/pocket-id/pocket-id/commit/2c0f60c66f9683bb44b7869e79505d1e42da40fa) use consistant group names ### 📊 Changes **7 files changed** (+309 additions, -47 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/e2e-tests.yml` (+56 -2) 📝 `backend/internal/bootstrap/e2etest_router_bootstrap.go` (+1 -1) 📝 `backend/internal/controller/e2etest_controller.go` (+10 -0) 📝 `backend/internal/service/e2etest_service.go` (+59 -3) 📝 `frontend/tests/application-configuration.spec.ts` (+0 -41) ➕ `frontend/tests/ldap.spec.ts` (+83 -0) ➕ `scripts/tests/setup-lldap.sh` (+100 -0) </details> ### 📄 Description _No description provided_ --- <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:22:14 +03:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
breaking
bug
documentation
feature
needs more upvotes
open to pull requests
pull-request
Mirrored from GitHub Pull Request
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
OVERLORD
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/pocket-id#781
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?