Fix #4524 Add test cases for host models
This PR is to fix #4524.
It covers the task list as mentioned in the issue and ensures complete coverage of the hosts/models.py file.
Hi @RishabhJain2018 this PR is ready for review
Thanks @Zahed-Riyaz for the PR. Let me fix the codecov so that I can track the progress too. Once that is fixed, I'll merge this.
Sure! Thank you for the update.
Build wasn't successful because of code quality checks. Should be good now. (hopefully)
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 69.70%. Comparing base (
96968d6) to head (cc0218c). Report is 1183 commits behind head on master.
Additional details and impacted files
@@ Coverage Diff @@
## master #4525 +/- ##
==========================================
- Coverage 72.93% 69.70% -3.23%
==========================================
Files 83 21 -62
Lines 5368 3608 -1760
==========================================
- Hits 3915 2515 -1400
+ Misses 1453 1093 -360
see 65 files with indirect coverage changes
see 65 files with indirect coverage changes
Continue to review full report in Codecov by Sentry.
Legend - Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing dataPowered by Codecov. Last update 8d1444f...cc0218c. Read the comment docs.
:rocket: New features to boost your workflow:
- :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
- :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
Hi @RishabhJain2018 this can be merged now