Skip to content

Fix test compilation and runtime issues from PR review feedback#48

Draft
Copilot wants to merge 3 commits intoadd-backend-tests-16104220138383017423from
copilot/sub-pr-47
Draft

Fix test compilation and runtime issues from PR review feedback#48
Copilot wants to merge 3 commits intoadd-backend-tests-16104220138383017423from
copilot/sub-pr-47

Conversation

Copy link

Copilot AI commented Feb 24, 2026

  • Resolve merge conflicts between this branch and base branch (add-backend-tests-16104220138383017423)
    • SessionControllerTest.kt: use org.mockito.kotlin.mock with mock<T>() syntax (consistent with base branch)
    • GlobalWebSocketHandlerTest.kt: use mutableMapOf(...) as Map<String, Any> cast pattern and whenever(...).then { } for sendMessage (consistent with base branch)

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

…ts, mutableMapOf

Co-authored-by: RobotHanzo <36107150+RobotHanzo@users.noreply.github.com>
Copy link
Author

Copilot AI commented Feb 24, 2026

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • jitpack.io
    • Triggering command: /usr/lib/jvm/temurin-17-jdk-amd64/bin/java /usr/lib/jvm/temurin-17-jdk-amd64/bin/java --add-opens=java.base/java.lang=ALL-UNNAMED --add-opens=java.base/java.lang.invoke=ALL-UNNAMED --add-opens=java.base/java.util=ALL-UNNAMED --add-opens=java.prefs/java.util.prefs=ALL-UNNAMED --add-exports=jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED --add-exports=jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED --add-opens=java.base/java.util=ALL-UNNAMED --add-opens=java.prefs/java.util.prefs=ALL-UNNAMED --add-opens=java.base/java.nio.charset=ALL-UNNAMED --add-opens=java.base/java.net=ALL-UNNAMED --add-opens=java.base/java.util.concurrent=ALL-UNNAMED --add-opens=java.base/java.util.concurrent.atomic=ALL-UNNAMED --add-opens=java.xml/javax.xml.namespace=ALL-UNNAMED --add-opens=java.base/java.time=ALL-UNNAMED -XX:MaxMetaspaceSize=384m -XX:&#43;HeapDumpOnOutOfMemoryError -Xms256m -Xmx512m -Dfile.encoding=UTF-8 (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI changed the title [WIP] Add unit tests for backend controllers and WebSocket handler Fix test compilation and runtime issues from PR review feedback Feb 24, 2026
Copilot AI requested a review from RobotHanzo February 24, 2026 04:46
@RobotHanzo
Copy link
Owner

@copilot resolve merge conflicts

Co-authored-by: RobotHanzo <36107150+RobotHanzo@users.noreply.github.com>
Copy link
Author

Copilot AI commented Feb 24, 2026

@copilot resolve merge conflicts

Resolved in commit ce4ddd9. The base branch had a commit (ce996d8) that fixed the same review issues with slightly different approaches — aligned this branch to match exactly, resolving the conflicts in SessionControllerTest.kt (now uses org.mockito.kotlin.mock<T>()) and GlobalWebSocketHandlerTest.kt (now uses mutableMapOf(...) as Map<String, Any> cast and whenever(...).then { } for sendMessage).

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.

2 participants