fix: Handle null cancellation dates

[zachaysan]

Changes

Trials mean we don't always get cancellation dates in the response from Chargebee. This fixes this.

How did you test this code?

Edit: There is now testing in place. The one path where current_term_end is set was covered by an existing test, so I've added one new one for the case where it is unset.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

3 Skipped Deployments

Name	Status	Preview	Comments	Updated (UTC)
docs	⬜️ Ignored (Inspect)	Visit Preview		Sep 6, 2024 1:58pm
flagsmith-frontend-preview	⬜️ Ignored (Inspect)	Visit Preview		Sep 6, 2024 1:58pm
flagsmith-frontend-staging	⬜️ Ignored (Inspect)	Visit Preview		Sep 6, 2024 1:58pm

[sentry-io]

🔍 Existing Issues For Review

Your pull request is modifying functions with the following pre-existing issues:

📄 File: api/organisations/chargebee/webhook_handlers.py

Function	Unhandled Issue
process_subscription	TypeError: 'NoneType' object cannot be interpreted as an integer /ap... Event Count: 3

---

_{Did you find this useful? React with a 👍 or 👎}

[github-actions]

Docker builds report

Image	Build Status	Security report
ghcr.io/flagsmith/flagsmith:pr-4589	Finished ✅	Results ✅
ghcr.io/flagsmith/flagsmith-private-cloud:pr-4589	Finished ✅	Results ✅
ghcr.io/flagsmith/flagsmith-api-test:pr-4589	Finished ✅	Skipped
ghcr.io/flagsmith/flagsmith-e2e:pr-4589	Finished ✅	Skipped
ghcr.io/flagsmith/flagsmith-api:pr-4589	Finished ✅	Results ✅
ghcr.io/flagsmith/flagsmith-frontend:pr-4589	Finished ✅	Results ✅
ghcr.io/flagsmith/flagsmith:pr-4589	Finished ✅	Results ✅
ghcr.io/flagsmith/flagsmith-private-cloud:pr-4589	Finished ✅	Results ✅

[github-actions]

Uffizzi Preview deployment-55888 was deleted.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.16%. Comparing base (5c02c1e) to head (2100670).
Report is 14 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff            @@
##             main    #4589    +/-   ##
========================================
  Coverage   97.15%   97.16%            
========================================
  Files        1156     1157     +1     
  Lines       39934    40039   +105     
========================================
+ Hits        38797    38902   +105     
  Misses       1137     1137            

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

[matthewelwell]

@zachaysan I'm not sure why testing is N/A here? Let's make sure that we add some test coverage please.

[zachaysan]

@zachaysan I'm not sure why testing is N/A here? Let's make sure that we add some test coverage please.

This is done now @matthewelwell