Skip to content

INS-1461 Setup Unit Testing Framework #151

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
amattu2 merged 16 commits into from
Dec 15, 2025
Merged

INS-1461 Setup Unit Testing Framework #151

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
9a475d4
init: Unit testing
amattu2 Dec 10, 2025
a0a040b
Add PR template
amattu2 Dec 10, 2025
5f0c998
Merge branch '3.2.0' into unit-test
amattu2 Dec 12, 2025
250a659
fix: Submodules missing
amattu2 Dec 12, 2025
f36360d
Add coverage computation
amattu2 Dec 15, 2025
bf77499
Merge branch '3.2.0' into unit-test
amattu2 Dec 15, 2025
b378a24
Add jacoco plugin
amattu2 Dec 15, 2025
cff3172
Merge branch '3.2.0' into unit-test
amattu2 Dec 15, 2025
4c15464
Remove legacy test
amattu2 Dec 15, 2025
18bf855
fix: Test CI job should not pass on failure
amattu2 Dec 15, 2025
36e7dca
Initial plan
Copilot Dec 15, 2025
0228b48
Add comprehensive unit tests for IndexController
Copilot Dec 15, 2025
04ab462
Complete IndexController unit tests with all checks passed
Copilot Dec 15, 2025
979700a
Revert non-test file changes and remove failing legacy tests
Copilot Dec 15, 2025
c9987e3
Merge pull request #157 from CBIIT/copilot/sub-pr-151
amattu2 Dec 15, 2025
8b0e65a
Add README badges
amattu2 Dec 15, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 11 additions & 0 deletions .github/PULL_REQUEST_TEMPLATE.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
### Overview

N/A

### Change Details (Specifics)

N/A

### Related Ticket(s)

N/A
39 changes: 39 additions & 0 deletions .github/workflows/test.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,39 @@
name: Test

on:
workflow_dispatch:
push:
branches:
- "*.*.*"
- "main"
- "master"
pull_request:
branches:
- "*"

permissions:
contents: read
Copy link

Copilot AI Dec 15, 2025

Choose a reason for hiding this comment

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

The workflow needs write permissions for pull-requests to allow the Coveralls action to post comments on PRs. Consider adding 'pull-requests: write' to the permissions section to enable coverage report comments on pull requests.

Suggested change
contents: read
contents: read
pull-requests: write

Copilot uses AI. Check for mistakes.

jobs:
test:
name: Test Changes
runs-on: ubuntu-latest
steps:
- name: Checkout Repository
uses: actions/checkout@v4
with:
submodules: true

- name: Setup Java
uses: actions/setup-java@v4
with:
java-version: "17"
distribution: "temurin"
cache: "maven"

- name: Run Tests
run: mvn test

- name: Coveralls GitHub Action
uses: coverallsapp/github-action@v2
Copy link

Copilot AI Dec 15, 2025

Choose a reason for hiding this comment

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

The Coveralls GitHub Action needs to be configured to use the JaCoCo coverage report format. By default, it looks for LCOV format, but JaCoCo generates XML reports. You should add the 'format' parameter set to 'jacoco' to ensure the coverage report is properly parsed.

Suggested change
uses: coverallsapp/github-action@v2
uses: coverallsapp/github-action@v2
with:
format: jacoco

Copilot uses AI. Check for mistakes.
continue-on-error: true
2 changes: 2 additions & 0 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
[![Codacy Badge](https://app.codacy.com/project/badge/Grade/197ca1f70b6a47618332548b6da480c1)](https://www.codacy.com/gh/CBIIT/bento-backend?utm_source=github.com&utm_medium=referral&utm_content=CBIIT/bento-backend&utm_campaign=Badge_Grade)
[![Test](https://github.com/CBIIT/INS-WebService/actions/workflows/test.yml/badge.svg)](https://github.com/CBIIT/INS-WebService/actions/workflows/test.yml)
[![Coverage Status](https://coveralls.io/repos/github/CBIIT/INS-WebService/badge.svg)](https://coveralls.io/github/CBIIT/INS-WebService)

# Bento Backend Framework Configuration Guide
This is the user documentation for the Bento Backend.
Expand Down
19 changes: 19 additions & 0 deletions pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -386,6 +386,25 @@
</execution>
</executions>
</plugin>
<plugin>
<groupId>org.jacoco</groupId>
<artifactId>jacoco-maven-plugin</artifactId>
<version>0.8.10</version>
<executions>
<execution>
<goals>
<goal>prepare-agent</goal>
</goals>
</execution>
<execution>
<id>report</id>
<phase>test</phase>
<goals>
<goal>report</goal>
</goals>
</execution>
</executions>
</plugin>
</plugins>
</build>
</project>
85 changes: 0 additions & 85 deletions src/test/java/gov/nih/nci/bento/EsServiceTest.java
View file
Open in desktop

This file was deleted.

127 changes: 0 additions & 127 deletions src/test/java/gov/nih/nci/bento/GraphQLControllerTest.java
View file
Open in desktop

This file was deleted.

Loading
Loading