Skip to content
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.

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

add unit test step in gradle ci #11547

Merged

Conversation

fabienpuissant
Copy link
Collaborator

Fix #11513

Comment on lines 30 to 31
- name: 'Test: run tests'
run: ./gradlew clean integrationTest --no-daemon
Copy link
Contributor

Choose a reason for hiding this comment

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

@fabienpuissant : As @atomfrede said before, i think the clean should be removed from that command.

Thanks for fixing that issue!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

absolutely I've done this on my personal repo but not there xD

renanfranca
renanfranca previously approved these changes Dec 8, 2024
Copy link
Contributor

@renanfranca renanfranca left a comment

Choose a reason for hiding this comment

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

Thank you for your work!

@@ -25,5 +25,7 @@ jobs:
uses: ./.github/actions/setup
- name: Setup Gradle
uses: gradle/gradle-build-action@v2
- name: 'Test: run unit tests'
run: ./gradlew clean test --no-daemon
- name: 'Test: run tests'
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- name: 'Test: run tests'
- name: 'Test: run integration tests'

@fabienpuissant : I only saw that now, sorry.

@fabienpuissant fabienpuissant force-pushed the fix/run-unit-test-gradle-ci branch from 73164eb to 10f58d1 Compare December 8, 2024 21:39
@renanfranca renanfranca enabled auto-merge December 8, 2024 21:41
@renanfranca renanfranca merged commit a294887 into jhipster:main Dec 8, 2024
36 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Gradle CI doesn't launch unit tests
3 participants