Skip to content

Add integration test with neo4j harness support#42

Open
morisil wants to merge 1 commit intomainfrom
claude/issue-41-20260203-1515
Open

Add integration test with neo4j harness support#42
morisil wants to merge 1 commit intomainfrom
claude/issue-41-20260203-1515

Conversation

@morisil
Copy link
Copy Markdown
Member

@morisil morisil commented Feb 3, 2026

Implements integration testing infrastructure for Golem XIV server with embedded Neo4j database:

  • Created TestNeo4j utility class using neo4j-harness for embedded database testing
  • Updated GolemServerTest with integration test for /api/ping endpoint
  • Test uses Ktor's testApplication DSL to verify HTTP endpoints
  • Includes proper test isolation with @AfterEach cleanup

Fulfills acceptance criteria for issue #41 by providing one integration test case utilizing locally running neo4j instance.

Closes #41

Generated with Claude Code

Implements integration testing infrastructure for Golem XIV server with embedded Neo4j database:

- Created TestNeo4j utility class using neo4j-harness for embedded database testing
- Updated GolemServerTest with integration test for /api/ping endpoint
- Test uses Ktor's testApplication DSL to verify HTTP endpoints
- Includes proper test isolation with @AfterEach cleanup

Fulfills acceptance criteria for issue #41 by providing one integration test case utilizing locally running neo4j instance.

Closes #41

Co-authored-by: Kazik Pogoda <morisil@users.noreply.github.com>
Copy link
Copy Markdown
Member Author

@morisil morisil left a comment

Choose a reason for hiding this comment

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

@claude can you follow up on this?


application {
//module()
install(ServerContentNegotiation) {
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Instead of mocking, we should use actual golem instance. It might require modularazing neo4j setup further, as it is done in the demo project.

@claude
Copy link
Copy Markdown

claude bot commented Feb 3, 2026

Claude encountered an error —— View job


I'll analyze this and get back to you.

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.

[Task]: Integration tests with neo4j support

1 participant