Skip to content

Creation of a new branch - test-infrastructure, which is responsible … - #4

Merged
vkutuev merged 30 commits into
vkutuev:mainfrom
GRIGAeo:test-infrastructure
Jul 13, 2026
Merged

Creation of a new branch - test-infrastructure, which is responsible …#4
vkutuev merged 30 commits into
vkutuev:mainfrom
GRIGAeo:test-infrastructure

Conversation

@GRIGAeo

@GRIGAeo GRIGAeo commented Mar 22, 2026

Copy link
Copy Markdown

…for test automation. All changes related to test coverage will be made in this branch going forward. This commit includes the following changes: renaming the file ci.yaml to ci-docs.yaml and creating a new file ci-tests.yaml, which runs tests on every push and pull request in any branch

…for test automation. All changes related to test coverage will be made in this branch going forward. This commit includes the following changes: renaming the file ci.yaml to ci-docs.yaml and creating a new file ci-tests.yaml, which runs tests on every push and pull request in any branch
@GRIGAeo

GRIGAeo commented Mar 22, 2026

Copy link
Copy Markdown
Author

Adding a new branch and creating a configuration file that automatically runs tests on pushes and pull requests.

@gsvgit gsvgit left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Бездумная нейронка детектед. Лучше пользоваться максимально свежими версиями action-ов.

filenames: |
solver/build/reports/jacoco/test/jacocoTestReport.xml
generator/build/reports/jacoco/test/jacocoTestReport.xml
test-shared/build/reports/jacoco/test/jacocoTestReport.xml

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Я правильно понимаю, что мы собираем тестовое покрытие тесов? Несколько загадочно выглядит.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Это так, меня не много смутил данный action для работы с многомодульными проектами. Я решил явно указать пути до xml отчётов, но я щас проверил ещё раз. По-моему этот action вообще не принимает дополнительных аргументов, а сам всё находит. Так что щас подправлю эту часть кода и в следующем PR всё будет нормально

@GRIGAeo

GRIGAeo commented May 13, 2026

Copy link
Copy Markdown
Author

This PR introduces the following changes:

  1. Test Coverage Reporting Setup

    Configured generation of test coverage reports (XML, CSV, HTML) for the generator and solver modules.

    The following coverage thresholds have been set per module:

     Instructions: 95%
    
     Branches: 80%
    
     Lines: 80%
    
     Methods: 85%
    
     Classes: 90%
    

Current Coverage Metrics:

solver module:

    INSTRUCTIONS: 58.24%

    BRANCHES: 43.55%

    LINES: 54.48%

    CYCLOMATIC: 44.77%

    METHODS: 53.33%

    CLASSES: 78.21%

generator module: all metrics are currently 0%.
  1. CI Workflow for Coverage Reports

    Added a CI workflow that generates a test coverage report for the entire project on:

     push events
    
     pull_request events
    
     manual workflow dispatch
    
  2. Publishing to GitHub Packages

    Integrated and configured the maven-publish plugin in the solver and generator modules to publish JAR files to GitHub Packages.

  3. Automated Publication on New Releases

    Added a configuration file to automatically publish JAR files to GitHub Packages when a new version of the project is released.

@GRIGAeo

GRIGAeo commented Jun 5, 2026

Copy link
Copy Markdown
Author

This PR introduces the following changes:

1. Fixed JAR file publishing

  • Added a separate publisher module responsible for publishing the JAR file and POM file to GitHub Packages.
  • The published JAR file will now contain only the compiled source code files, while the POM file will include references to the project's dependencies.

2. Fixed and improved the project build files for test coverage reporting

  • In the generator module, any newly added unit tests will now be taken into account when generating the coverage report.
  • Fixed the jacocoTestCoverageVerification task in the solver and generator modules.

@vkutuev
vkutuev merged commit b0c089d into vkutuev:main Jul 13, 2026
1 check failed
@github-actions

Copy link
Copy Markdown
Overall Project 51.31%

There is no coverage information present for the Files changed

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