Skip to content

Commit

Permalink
fixup! include jvmId in responses and notifications rather than conne…
Browse files Browse the repository at this point in the history
…ctUrl
  • Loading branch information
andrewazores committed Sep 5, 2024
1 parent 5b6faf0 commit eb52cd6
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions src/test/java/itest/GraphQLTest.java
Original file line number Diff line number Diff line change
Expand Up @@ -250,10 +250,8 @@ void testStartRecordingMutationOnSpecificTarget() throws Exception {
notification.getJsonObject("message").getJsonObject("recording");
MatcherAssert.assertThat(notificationRecording.getString("name"), Matchers.equalTo("test"));
MatcherAssert.assertThat(
notification.getJsonObject("message").getString("target"),
Matchers.equalTo(
String.format(
"service:jmx:rmi:///jndi/rmi://%s:%d/jmxrmi", "localhost", 0)));
notification.getJsonObject("message").getString("jvmId"),
Matchers.not(Matchers.blankOrNullString()));
JsonArray notificationLabels =
notificationRecording.getJsonObject("metadata").getJsonArray("labels");
Map<String, String> expectedLabels =
Expand Down

0 comments on commit eb52cd6

Please sign in to comment.