Skip to content

[DEVX-606]: Add test coverage for packages with low or no coverage#409

Merged
srikanthbachala20 merged 4 commits intomasterfrom
DEVX-606-Add-Test-Coverage
Oct 7, 2024
Merged

[DEVX-606]: Add test coverage for packages with low or no coverage#409
srikanthbachala20 merged 4 commits intomasterfrom
DEVX-606-Add-Test-Coverage

Conversation

@srikanthbachala20
Copy link
Copy Markdown
Contributor

@srikanthbachala20 srikanthbachala20 commented Oct 7, 2024

Why

  • To improve the overall Test Coverage of SDK, started to improve the test coverage of dataloaders module clarifai/datasets/upload and model evaluation module clarifai/utils/evaluation

How

  • Added tests for dataloaders module and evaluation module

Test Coverage

  • Initially, the test coverage is 52%
  • Now, the test coverage is increased by 13% to 65%

Copy link
Copy Markdown
Contributor

@sanjaychelliah sanjaychelliah left a comment

Choose a reason for hiding this comment

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

LGTM. Can you please post the test coverage report after this PR in the comments!

@srikanthbachala20 srikanthbachala20 merged commit 11198f8 into master Oct 7, 2024
@srikanthbachala20 srikanthbachala20 deleted the DEVX-606-Add-Test-Coverage branch October 7, 2024 11:07
@srikanthbachala20 srikanthbachala20 mentioned this pull request Oct 7, 2024
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.

3 participants