Skip to content

fix: search not-found handling and add mock server for integration testing #7

Merged
wadahiro merged 3 commits into
mainfrom
dev
Apr 9, 2026
Merged

fix: search not-found handling and add mock server for integration testing #7
wadahiro merged 3 commits into
mainfrom
dev

Conversation

@wadahiro
Copy link
Copy Markdown
Contributor

@wadahiro wadahiro commented Apr 9, 2026

  • Fix executeQuery to return empty results instead of throwing exception when object not found, per ConnId specification
  • Add a Go-based mock server that emulates the Atlassian Crowd REST API with XML responses and PostgreSQL storage, enabling connector testing without an actual Crowd environment
  • Add integration tests (43 cases) using TestContainers for user/group CRUD, attributes, memberships, search, schema, and connection validation
  • Migrate build from OSSRH to Maven Central Portal
  • Push crowd-mock-server Docker image to GHCR on release

wadahiro added 2 commits April 9, 2026 22:02
…t found in search

Per ConnId specification, executeQuery (SearchOp) should not throw any
exception when an object is not found. It should simply not invoke the
handler, returning an empty result set.

Changed getByUid/getByName in CrowdUserHandler and CrowdGroupHandler to
catch UnknownUidException and return 0 results.

Added unit tests for getByUid/getByName not-found cases.
Add a Go-based mock server that emulates the Atlassian Crowd REST API
with XML responses and PostgreSQL storage, enabling connector integration
testing without an actual Crowd environment.

Add integration tests using TestContainers for user/group CRUD,
attributes, memberships, search, schema, and connection validation.
@wadahiro wadahiro merged commit 2d10c7a into main Apr 9, 2026
1 check passed
@wadahiro wadahiro deleted the dev branch April 9, 2026 13:50
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.

1 participant