Skip to content

feat(signals): recognize e2e, integration, and snapshot test paths - #1262

Merged
JSONbored merged 6 commits into
JSONbored:mainfrom
jaso0n0818:feat/signals-e2e-test-paths
Jun 26, 2026
Merged

feat(signals): recognize e2e, integration, and snapshot test paths#1262
JSONbored merged 6 commits into
JSONbored:mainfrom
jaso0n0818:feat/signals-e2e-test-paths

Conversation

@jaso0n0818

Copy link
Copy Markdown
Contributor

Summary

Extends isTestPath / classifyTestCoverage so slop and contributor evidence recognize modern e2e and snapshot test layouts (#1046 follow-up).

New test path recognition:

  • e2e/, integration/, playwright/, cypress/ directories
  • Cypress-style *.cy.ts / *.cy.js and *.e2e.ts suffixes
  • Jest/Vitest __snapshots__/ trees and *.snap files

Pure path-only matchers with unit coverage.

Test plan

  • npm test -- test/unit/test-evidence.test.ts

Extend isTestPath so slop and contributor evidence treat common
browser/e2e layouts as real test effort:

- e2e/, integration/, playwright/, cypress/ directories
- Cypress/Playwright-style *.cy.ts and *.e2e.ts suffixes
- Jest/Vitest __snapshots__/ trees and *.snap files

Keeps classifyTestCoverage ratios accurate for modern test layouts.

Co-authored-by: Cursor <cursoragent@cursor.com>
@jaso0n0818
jaso0n0818 requested a review from JSONbored as a code owner June 24, 2026 21:00
@dosubot dosubot Bot added the size:XS label Jun 24, 2026
@superagent-security

Copy link
Copy Markdown
Contributor

Superagent didn't find any vulnerabilities or security issues in this PR.

@codecov

codecov Bot commented Jun 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.40%. Comparing base (1f9b84b) to head (ad216dd).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1262   +/-   ##
=======================================
  Coverage   95.40%   95.40%           
=======================================
  Files         193      193           
  Lines       20948    20948           
  Branches     7575     7575           
=======================================
  Hits        19985    19985           
  Misses        383      383           
  Partials      580      580           
Files with missing lines Coverage Δ
src/signals/test-evidence.ts 100.00% <100.00%> (ø)
🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

The broad *.snap rule misclassified non-test churn files like src/state.snap,
breaking slop golden fixtures and failing test shard 2/3.

Co-authored-by: Cursor <cursoragent@cursor.com>

@JSONbored JSONbored left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Extends the test-path matcher; composes with #1306 rather than duplicating it (different function). Keying snapshots on the snapshots/ dir not a bare .snap is right. Hands-off — owner executes.

@dosubot dosubot Bot added the lgtm label Jun 26, 2026
@JSONbored
JSONbored merged commit 4a75525 into JSONbored:main Jun 26, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gittensor:bug Gittensor-scored bug fix — scores a 0.05x multiplier.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants