fix(records): raise TypeError instead of leaking AttributeError for typed arguments - #2749
Draft
andersfylling wants to merge 3 commits into
Draft
fix(records): raise TypeError instead of leaking AttributeError for typed arguments#2749andersfylling wants to merge 3 commits into
andersfylling wants to merge 3 commits into
Codecov / codecov/project
succeeded
Aug 11, 2026 in 1s
93.79% (+0.00%) compared to 629c506
View this Pull Request on Codecov
93.79% (+0.00%) compared to 629c506
Details
Codecov Report
❌ Patch coverage is 97.46835% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 93.79%. Comparing base (629c506) to head (58d988e).
| Files with missing lines | Patch % | Lines |
|---|---|---|
| cognite/client/_api/data_modeling/records.py | 94.28% | 2 Missing |
Additional details and impacted files
@@ Coverage Diff @@
## master #2749 +/- ##
=======================================
Coverage 93.79% 93.79%
=======================================
Files 513 513
Lines 52445 52515 +70
=======================================
+ Hits 49189 49256 +67
- Misses 3256 3259 +3 | Files with missing lines | Coverage Δ | |
|---|---|---|
| cognite/client/_sync_api/data_modeling/records.py | 100.00% <100.00%> (ø) |
|
| ...s_unit/test_api/test_data_modeling/test_records.py | 100.00% <100.00%> (ø) |
|
| cognite/client/_api/data_modeling/records.py | 98.13% <94.28%> (+0.59%) |
⬆️ |
🚀 New features to boost your workflow:
- ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
- 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
Loading