☂️ Python Coverage

current status: ✅

Overall Coverage

New Files

No new covered files...

Modified Files

No covered modified files...

updated for commit: 0a5cce7 by action🐍

Codecov Report

Attention: Patch coverage is 96.70330% with 3 lines in your changes missing coverage. Please review.

Project coverage is 94.33%. Comparing base (67e886e) to head (0a5cce7).
Report is 4 commits behind head on main.

@@            Coverage Diff             @@
##             main     #385      +/-   ##
==========================================
+ Coverage   94.26%   94.33%   +0.06%     
==========================================
  Files         250      252       +2     
  Lines       14098    14461     +363     
==========================================
+ Hits        13290    13642     +352     
- Misses        808      819      +11     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@osundwajeff could you also add the export to csv/excel from the django admin page? maybe add it as another new ticket.

@danangmassandy This is the PR with the failing test that needs to be looked at

@osundwajeff it seems the error is because the crop-plan API tries to fetch all Farms, so your test farms might also get fetched. Could you add request parameters farm_ids in the request like below?

response = self.request(
    reverse('api:v1:crop-plan') +
    '?farm_ids=farm-1,farm-2'
)

Please add the request parameters in the failed test below:
https://github.com/kartoza/tomorrownow_gap/blob/main/django_project/gap_api/tests/test_crop_plan_api.py#L122

I'll do that thanks @danangmassandy

@mvmaltitz Just in case you are wondering about ticket #391, It has been merged already