Skip to content

feat:Refactor CI/CD workflows and enhance testing coverage#1

Merged
aunchagaonkar merged 3 commits intomainfrom
feature/add-tests
Sep 7, 2025
Merged

feat:Refactor CI/CD workflows and enhance testing coverage#1
aunchagaonkar merged 3 commits intomainfrom
feature/add-tests

Conversation

@aunchagaonkar
Copy link
Owner

Key changes:

  • Added testing dependencies: MockK for mocking, Hilt for test dependency injection, and kotlinx-coroutines-test for coroutine testing.
  • Implemented unit tests for the data, domain, and presentation layers, covering ViewModels, UseCases, Repositories, and DataSources.
  • Added a lightweight unit test for the utility functions.
  • Updated the README.md with detailed instructions on how to run the unit tests.
  • Configured the build to support the new tests.
  • Updated build.yml to include pull request triggers for feature branches and added unit test execution step.
  • Removed build_pull_request.yml file.

Key changes:
- Added testing dependencies: MockK for mocking, Hilt for test dependency injection, and kotlinx-coroutines-test for coroutine testing.
- Implemented unit tests for the data, domain, and presentation layers, covering ViewModels, UseCases, Repositories, and DataSources.
- Added a lightweight unit test for the utility functions.
- Updated the README.md with detailed instructions on how to run the unit tests.
- Configured the build to support the new tests.
- Updated build.yml to include pull request triggers for feature branches and added unit test execution step.
- Removed build_pull_request.yml file.
@aunchagaonkar aunchagaonkar self-assigned this Sep 7, 2025
Copy link

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 refactors the CI/CD workflows and significantly enhances testing coverage by implementing comprehensive unit tests across all application layers. The changes modernize the build process and provide extensive test coverage for ViewModels, UseCases, Repositories, and DataSources.

  • Consolidated CI/CD workflows by removing the separate PR build workflow and extending the main build workflow to handle pull requests
  • Implemented comprehensive unit testing infrastructure with MockK, Hilt testing support, and coroutine testing utilities
  • Added unit tests covering data layer (repositories and data sources), domain layer (use cases), and presentation layer (view models)

Reviewed Changes

Copilot reviewed 15 out of 15 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
.github/workflows/build.yml Extended to support PR triggers and added unit test execution with artifact uploads
.github/workflows/build_pull_request.yml Removed redundant PR-specific workflow
.github/workflows/release.yml Cleaned up by removing Gradle cache configuration
gradle/libs.versions.toml Added testing dependencies (MockK, coroutines-test, Hilt testing)
app/build.gradle.kts Configured test options and added testing dependencies
hiddenapi/build.gradle.kts Removed targetSdk configuration
README.md Updated with testing instructions and corrected repository references
app/src/test/java/... Added comprehensive unit tests for all application layers

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@aunchagaonkar aunchagaonkar merged commit 1e3233a into main Sep 7, 2025
1 check passed
@aunchagaonkar aunchagaonkar deleted the feature/add-tests branch September 7, 2025 11:41
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.

2 participants