Skip to content
This repository has been archived by the owner on Jan 31, 2024. It is now read-only.

Unit Test Optimization - adding new test #279

Merged
merged 5 commits into from
May 12, 2022
Merged

Conversation

MaximeZmt
Copy link
Owner

@MaximeZmt MaximeZmt commented May 12, 2022

I've added new test in order to increase coverage

@MaximeZmt MaximeZmt self-assigned this May 12, 2022
@MaximeZmt MaximeZmt marked this pull request as ready for review May 12, 2022 21:46
@MaximeZmt MaximeZmt changed the title test adding new test Unit Test Optimization - adding new test May 12, 2022
@MaximeZmt MaximeZmt added the actual: 2h actual time to fix label May 12, 2022
@codeclimate
Copy link

codeclimate bot commented May 12, 2022

Code Climate has analyzed commit 8d06095 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (80% is the threshold).

This pull request will bring the total coverage in the repository to 83.4% (0.8% change).

View more on Code Climate.

Copy link
Collaborator

@Tsathogguaa Tsathogguaa left a comment

Choose a reason for hiding this comment

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

Looks good, thanks for the tests!

Copy link
Collaborator

@kamilababayeva kamilababayeva left a comment

Choose a reason for hiding this comment

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

Looks good! Thanks for improvements!

@kamilababayeva kamilababayeva merged commit 844fb46 into main May 12, 2022
@kamilababayeva kamilababayeva deleted the maximezmt/testcoverage branch May 12, 2022 23:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
actual: 2h actual time to fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants