Fixes #1229

Removing relation child while updating nullable fk as None in descriptors.py->RelationDescriptor

Ran the test cases locally using pytest -svv --cov=ormar --cov=tests --cov-fail-under=100 --cov-report=term-missing

All test cases passed

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (3a206dd) to head (57ab550).
⚠️ Report is 115 commits behind head on master.

@@            Coverage Diff            @@
##            master     #1230   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          201       201           
  Lines        16647     16659   +12     
=========================================
+ Hits         16647     16659   +12     

Hi,

Can you add a test that fails now and that this issue fixes?
Can be based on #1229 (without fastapi part, just ormar code is enough).
Please place it under tests/test_relations/test_foreign_keys.py as an additional test case.

Hi,

Can you add a test that fails now and that this issue fixes? Can be based on #1229 (without fastapi part, just ormar code is enough). Please place it under tests/test_relations/test_foreign_keys.py as an additional test case.

Hi @collerek,

I added the test-case. It fails without my changes and is passing with my changes. Please suggest if I can give a better name to the test-case.

Here are the screenshots-

Failing without my changes:

Passed with my changes:

@collerek please merge this if everything looks fine

CodSpeed Performance Report

Merging #1230 will degrade performances by 23.48%

_{Comparing amit12297:fix-fk-null-update (57ab550) with master (3a206dd)}

Summary

❌ 1 regressions
✅ 83 untouched benchmarks

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Benchmarks breakdown