Enhance integration tests with Kafka streaming triangle counting #364
Conversation
…update configurations
There was a problem hiding this comment.
Pull request overview
Adds Kafka to the integration environment and extends the integration test suite to exercise streaming triangle counting via the adstrmk/strian flow, while also removing hard-coded graph IDs from several Cypher tests.
Changes:
- Add Kafka service to the integration
docker-compose.ymland configure JasmineGraph to depend on it. - Extend
tests/integration/test.pywith Kafka setup/produce/consume helpers and a new streaming triangle counting test. - Add integration config files for Kafka and JasmineGraph server properties.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| tests/integration/test.py | Adds Kafka/streaming test helpers, runs a new strian integration test, and replaces hard-coded graph IDs with IDs parsed from lst. |
| tests/integration/env_init/config/kafka/consumer.properties | Adds Kafka host configuration for a non-default consumer config path. |
| tests/integration/env_init/config/jasminegraph-server.properties | Provides integration-time server config with Kafka bootstrap set to kafka:29092. |
| tests/integration/docker-compose.yml | Adds a Kafka service and wires JasmineGraph to depend on it; mounts the integration server properties file. |
| test-docker.sh | Makes HDFS safemode handling more robust and fixes the final docker tag command. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #364 +/- ##
======================================
Coverage 1.11% 1.11%
======================================
Files 110 110
Lines 27014 27014
Branches 17679 17679
======================================
Hits 302 302
Misses 26454 26454
Partials 258 258 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
…enhance error handling, and improve documentation
…conditional execution based on environment variable
…eout command execution in test scripts
…ing, and enhance readability
Nayanatharapmc
force-pushed
the
test-strian
branch
from
e5ad6c9 to
64e3de0
Compare
|
miyurud
left a comment
miyurud
left a comment
There was a problem hiding this comment.
I have put some comments. Please address all of those.
|
No description provided.