Skip to content

Add tests for cli#474

Merged
MarcelGeo merged 20 commits intodevelopfrom
test_cli_commands
Aug 4, 2025
Merged

Add tests for cli#474
MarcelGeo merged 20 commits intodevelopfrom
test_cli_commands

Conversation

@varmar05
Copy link
Collaborator

@varmar05 varmar05 commented Jul 3, 2025

Example test (project create) for testing cli commands.

@varmar05 varmar05 requested a review from MarcelGeo July 3, 2025 09:59
@coveralls
Copy link

coveralls commented Jul 3, 2025

Pull Request Test Coverage Report for Build 16651130116

Details

  • 314 of 326 (96.32%) changed or added relevant lines in 7 files are covered.
  • 4 unchanged lines in 1 file lost coverage.
  • Overall coverage increased (+2.4%) to 94.107%

Changes Missing Coverage Covered Lines Changed/Added Lines %
server/mergin/sync/commands.py 28 30 93.33%
server/mergin/tests/test_cli.py 172 174 98.85%
server/mergin/commands.py 102 110 92.73%
Files with Coverage Reduction New Missed Lines %
server/mergin/tests/test_middleware.py 4 80.0%
Totals Coverage Status
Change from base Build 15999513920: 2.4%
Covered Lines: 7442
Relevant Lines: 7908

💛 - Coveralls

Copy link
Collaborator

@MarcelGeo MarcelGeo left a comment

Choose a reason for hiding this comment

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

I think we could add following framework to our private repo too. To unify it.

@MarcelGeo
Copy link
Collaborator

@harminius would you provide tests for more commands?

@harminius harminius requested a review from MarcelGeo July 30, 2025 12:25
@varmar05
Copy link
Collaborator Author

varmar05 commented Aug 4, 2025

Looks good to me.

@varmar05 varmar05 changed the title Add example test for cli Add tests for cli Aug 4, 2025
@MarcelGeo MarcelGeo merged commit c815094 into develop Aug 4, 2025
4 checks passed
@MarcelGeo MarcelGeo deleted the test_cli_commands branch August 4, 2025 08:35
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