New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[feature] send notifications to client while junit5 testing #299
Conversation
60bb3e5
to
533b316
Compare
server/src/main/java/org/jetbrains/bsp/bazel/server/sync/ExecuteService.kt
Outdated
Show resolved
Hide resolved
logger/src/main/java/org/jetbrains/bsp/bazel/logger/BspClientTaskNotifier.java
Outdated
Show resolved
Hide resolved
server/src/main/java/org/jetbrains/bsp/bazel/server/sync/JUnitTestParser.kt
Outdated
Show resolved
Hide resolved
server/src/main/java/org/jetbrains/bsp/bazel/server/sync/JUnitTestParser.kt
Outdated
Show resolved
Hide resolved
server/src/main/java/org/jetbrains/bsp/bazel/server/sync/JUnitTestParser.kt
Outdated
Show resolved
Hide resolved
205dbe1
to
4814b6d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ill take a look one more time tomorrow
logger/src/main/java/org/jetbrains/bsp/bazel/logger/BspClientTestNotifier.kt
Outdated
Show resolved
Hide resolved
logger/src/main/java/org/jetbrains/bsp/bazel/logger/BspClientTestNotifier.kt
Outdated
Show resolved
Hide resolved
logger/src/main/java/org/jetbrains/bsp/bazel/logger/BspClientTestNotifier.kt
Outdated
Show resolved
Hide resolved
server/src/main/java/org/jetbrains/bsp/bazel/server/sync/JUnitTestParser.kt
Outdated
Show resolved
Hide resolved
server/src/test/java/org/jetbrains/bsp/bazel/server/sync/JUnitTestParserTest.kt
Outdated
Show resolved
Hide resolved
server/src/main/java/org/jetbrains/bsp/bazel/server/sync/JUnitTestParser.kt
Outdated
Show resolved
Hide resolved
server/src/main/java/org/jetbrains/bsp/bazel/server/sync/JUnitTestParser.kt
Outdated
Show resolved
Hide resolved
logger/src/main/java/org/jetbrains/bsp/bazel/logger/BspClientTestNotifier.kt
Outdated
Show resolved
Hide resolved
server/src/main/java/org/jetbrains/bsp/bazel/server/sync/ExecuteService.kt
Outdated
Show resolved
Hide resolved
server/src/main/java/org/jetbrains/bsp/bazel/server/sync/JUnitTestParser.kt
Outdated
Show resolved
Hide resolved
ae9fbd9
to
bea8d83
Compare
Co-authored-by: azdrojowa123 <aleksandra.zdrojowa@jetbrains.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks great, just nitpicking + please update the changelog
server/src/main/java/org/jetbrains/bsp/bazel/server/sync/ExecuteService.kt
Outdated
Show resolved
Hide resolved
server/src/main/java/org/jetbrains/bsp/bazel/server/sync/JUnitTestParser.kt
Outdated
Show resolved
Hide resolved
logger/src/main/java/org/jetbrains/bsp/bazel/logger/BspClientTestNotifier.kt
Outdated
Show resolved
Hide resolved
server/src/main/java/org/jetbrains/bsp/bazel/server/sync/JUnitTestParser.kt
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks, lets gooo
bazel-bsp
server now notifies the BSP client about events happening during testing with JUnit, including single tests and their results, as well as their hierarchy in test suitesCo-authored-by: azdrojowa123 aleksandra.zdrojowa@jetbrains.com