Skip to content

Integration test#67

Merged
JongbinLim merged 61 commits into
devfrom
iteration-final-test
Dec 7, 2025
Merged

Integration test#67
JongbinLim merged 61 commits into
devfrom
iteration-final-test

Conversation

@compactlions

Copy link
Copy Markdown

PR Title: Integration test

Related Issue(s):

for windows, jacoco can't analyze with coverage. so, I use github actions workflow to make coverage report.

PR Description:

some irrelavent codes are removed, and androidTest codes are updated. unit test codes will dismiss to our coverage report.

Changes Included:
  • [ x ] Added new feature(s)
  • [ x ] Fixed identified bug(s)
  • [ o ] Updated relevant documentation
Screenshots (if UI changes were made):
image ##### Notes for Reviewer: i'm not sure for merged version are well worked. after merge, I will do further integration test.

Reviewer Checklist:

  • Code is written in clean, maintainable, and idiomatic form.
  • Automated test coverage is adequate.
  • All existing tests pass.
  • Manual testing has been performed to ensure the PR works as
    expected.
  • Code review comments have been addressed or clarified.

Additional Comments:

None

compactlions and others added 30 commits December 6, 2025 18:04
Test coverage will be calculated with github action.
Removed the build step for the Android project in the integration test workflow.
Updated Android SDK installation to use sdkmanager.
Added steps to install Android SDK in the workflow.
Updated Android SDK installation steps in the workflow.
Added '--rerun-tasks' option to Gradle command for coverage report generation.
Add steps to upload and download coverage artifacts
@JongbinLim JongbinLim merged commit 9817066 into dev Dec 7, 2025
1 check passed
@compactlions compactlions deleted the iteration-final-test branch December 7, 2025 08:02
@compactlions compactlions restored the iteration-final-test branch December 7, 2025 08:02
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