Skip to content

Commit

Permalink
Fix event processing test after requiring security_protocol config (#…
Browse files Browse the repository at this point in the history
…1305)

### Description

This adds the missing default config to the test.

### Context

This was logging a `NullPointerException` due to new default configs
being added.

### Testing

- `./gradlew test`

### Documentation

N/A

### Known limitations

I added a TODO here because an exception was logged but the test didn't
fail.
  • Loading branch information
philipliu authored Mar 26, 2024
1 parent 355074d commit 95ec07a
Showing 1 changed file with 3 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@ class EventProcessingServerTests : AbstractIntegrationTests(TestConfig()) {
val session = eventService.createSession("testOk", TRANSACTION)
val quoteEvent = GsonUtils.getInstance().fromJson(testQuoteEvent, AnchorEvent::class.java)

// TODO: this test doesn't fail when the event is not published
session.publish(quoteEvent)
}
}
Expand Down Expand Up @@ -67,7 +68,8 @@ val eventConfigJson =
"retries": 0,
"lingerMs": 1000,
"batchSize": 10,
"pollTimeoutSeconds": 10
"pollTimeoutSeconds": 10,
"securityProtocol": "PLAINTEXT"
}
}
}
Expand Down

0 comments on commit 95ec07a

Please sign in to comment.