Skip to content

Conversation

kirill-stepanishin
Copy link
Contributor

Summary

Integrates JavaScript Gremlin examples (node) into CI to ensure they remain functional in future releases. Examples now execute automatically after integration tests and fail the build if broken.

Changes Made

  • Added example execution to gremlin-js-integration-tests container in docker-compose.yml
  • Made server URLs and vertex labels configurable via environment variables (replaces hardcoded localhost:8182)
  • Added environment detection for traversal source - uses gmodern in CI, default connection locally
  • Use specialized labels to isolate test data with consistent vertex labeling

- Added example execution to gremlin-js-integration-tests container in docker-compose.yml
- Made server URLs and vertex labels configurable via environment variables
- Use specialized labels to isolate test data
- Build fails if any example fails to execute
@codecov-commenter
Copy link

codecov-commenter commented Oct 16, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.23%. Comparing base (9b46b67) to head (19bca92).
⚠️ Report is 387 commits behind head on 3.7-dev.

Additional details and impacted files
@@              Coverage Diff              @@
##             3.7-dev    #3240      +/-   ##
=============================================
+ Coverage      76.14%   76.23%   +0.09%     
- Complexity     13152    13303     +151     
=============================================
  Files           1084     1092       +8     
  Lines          65160    67657    +2497     
  Branches        7285     7378      +93     
=============================================
+ Hits           49616    51580    +1964     
- Misses         12839    13330     +491     
- Partials        2705     2747      +42     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@kirill-stepanishin kirill-stepanishin marked this pull request as ready for review October 16, 2025 00:50
@andreachild
Copy link
Contributor

VOTE +1

1 similar comment
@xiazcy
Copy link
Contributor

xiazcy commented Oct 17, 2025

VOTE +1

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.

4 participants