Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: Add unit tests for DE polling #3231

Merged
merged 3 commits into from
Oct 22, 2024

Conversation

arjunaj5
Copy link
Contributor

@arjunaj5 arjunaj5 commented Oct 16, 2024

Clickup

https://app.clickup.com/t/86cwgw4wr

Code Coverage

Please add code coverage here

UI Preview

Screenshot 2024-10-16 at 10 45 21 PM

Copy link

PR description must contain a link to a ClickUp (case-insensitive)

Copy link

PR description must contain a link to a ClickUp (case-insensitive)

@arjunaj5 arjunaj5 requested a review from mvaishnavi October 16, 2024 17:18
Copy link

PR description must contain a link to a ClickUp (case-insensitive)

Copy link

Unit Test Coverage % values
Statements 95.97% ( 19112 / 19913 )
Branches 91.21% ( 10564 / 11581 )
Functions 94.35% ( 5704 / 6045 )
Lines 96.02% ( 18242 / 18998 )

@arjunaj5 arjunaj5 merged commit c3deaff into Fyle-86cwh9db4-failing-tests Oct 22, 2024
8 of 11 checks passed
arjunaj5 added a commit that referenced this pull request Oct 22, 2024
* Fix failing tests

* test: Add unit tests for DE polling (#3231)

* Add unit tests for DE polling

* minor

* fix lint
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/L Large PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants