Skip to content

**WIP** fix: Support attribute assignment of cardinality-many relationships - #1154

Draft
PhillSimonds wants to merge 2 commits into
developfrom
sts/fix-1152-many-rel-assignment
Draft

**WIP** fix: Support attribute assignment of cardinality-many relationships#1154
PhillSimonds wants to merge 2 commits into
developfrom
sts/fix-1152-many-rel-assignment

test: cover save() path for cardinality-many relationship assignment

9beed45
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Jul 10, 2026 in 1s

75.51% (target 0.00%)

View this Pull Request on Codecov

75.51% (target 0.00%)

Details

Codecov Report

❌ Patch coverage is 77.77778% with 4 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
infrahub_sdk/node/node.py 77.77% 2 Missing and 2 partials ⚠️

❗ There is a different number of reports uploaded between BASE (17483a0) and HEAD (9beed45). Click for more details.

HEAD has 1 upload less than BASE
Flag BASE (17483a0) HEAD (9beed45)
integration-tests 1 0
@@             Coverage Diff             @@
##           develop    #1154      +/-   ##
===========================================
- Coverage    82.35%   75.51%   -6.84%     
===========================================
  Files          138      138              
  Lines        12065    12060       -5     
  Branches      1805     1809       +4     
===========================================
- Hits          9936     9107     -829     
- Misses        1573     2401     +828     
+ Partials       556      552       -4     
Files with missing lines Coverage Δ
infrahub_sdk/node/node.py 80.42% <77.77%> (-7.15%) ⬇️

... and 34 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.