Skip to content

test: add Java analyzer tests to test suite#298

Merged
askpt merged 2 commits into
mainfrom
copilot/add-java-tests-to-testsuit
May 13, 2026
Merged

test: add Java analyzer tests to test suite#298
askpt merged 2 commits into
mainfrom
copilot/add-java-tests-to-testsuit

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 13, 2026

javaAnalyzer.test existed in src/test/metricsAnalyzer/languages/ but was never registered in the test runner, so Java tests were silently skipped on every run.

Change

Added the missing entry to the testFiles array in src/test/suite/index.ts:

const testFiles = [
  "../metricsAnalyzer/languages/csharpAnalyzer.test",
  "../metricsAnalyzer/languages/goAnalyzer.test",
+ "../metricsAnalyzer/languages/javaAnalyzer.test",
  "../metricsAnalyzer/languages/javascriptAnalyzer.test",
  // ...
];
Original prompt

For the testSuit, it seems it is missing the java tests. Please add them. The file in question is: src/test/suite/index.ts

Created from VS Code.

Copilot AI self-assigned this May 13, 2026
Copilot AI review requested due to automatic review settings May 13, 2026 10:45
Copilot AI review requested due to automatic review settings May 13, 2026 10:45
Copilot AI requested review from Copilot and removed request for Copilot May 13, 2026 10:46
Copilot AI changed the title [WIP] Add Java tests to test suite test: add Java analyzer tests to test suite May 13, 2026
Copilot finished work on behalf of askpt May 13, 2026 10:47
Copilot AI requested a review from askpt May 13, 2026 10:47
@askpt askpt marked this pull request as ready for review May 13, 2026 10:50
Copilot AI review requested due to automatic review settings May 13, 2026 10:50
@codecov
Copy link
Copy Markdown

codecov Bot commented May 13, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 69.54%. Comparing base (6f9e355) to head (a3ea8fe).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #298   +/-   ##
=======================================
  Coverage   69.54%   69.54%           
=======================================
  Files          10       10           
  Lines        3054     3054           
  Branches      287      287           
=======================================
  Hits         2124     2124           
  Misses        929      929           
  Partials        1        1           

☔ 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.

@askpt askpt merged commit 222db15 into main May 13, 2026
27 checks passed
@askpt askpt deleted the copilot/add-java-tests-to-testsuit branch May 13, 2026 10:51
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR ensures the Java analyzer unit tests are actually executed by registering the existing javaAnalyzer.test file in the test suite loader (src/test/suite/index.ts), preventing silent skipping of Java-related test coverage.

Changes:

  • Added ../metricsAnalyzer/languages/javaAnalyzer.test to the testFiles list so it gets require()’d and its suites are registered at runtime.

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.

3 participants