Skip to content

Hotfix/identifier conversion coverage - #310

Merged
TomHAnderson merged 2 commits into
API-Skeletons:14.0.xfrom
TomHAnderson:hotfix/identifier-conversion-coverage
Oct 2, 2026
Merged

TomHAnderson merged 2 commits into
API-Skeletons:14.0.xfrom
TomHAnderson:hotfix/identifier-conversion-coverage

Conversation

@TomHAnderson

Copy link
Copy Markdown
Member

No description provided.

TomHAnderson and others added 2 commits October 2, 2026 17:24
- Test that a batched collection target removed after its identifier is
  selected, as by another request, is left off its page.  The test base
  can now run a function just before a statement executes.
- Test a to-one association to an entity whose identifier is an
  association, whose identifier the to-one loader binds as it is.
- Remove a null identifier check which converting the identifier makes
  redundant, and a check that DBAL's type names are strings.
- Bind an entity without an identifier as itself, for Doctrine to report,
  rather than reading an identifier it does not have.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Also name the coverage scripts composer.json has; composer coverage
does not exist.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@codecov

codecov Bot commented Oct 2, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (c944147) to head (d148c83).
⚠️ Report is 4 commits behind head on 14.0.x.

Additional details and impacted files
@@              Coverage Diff              @@
##             14.0.x      #310      +/-   ##
=============================================
+ Coverage     99.87%   100.00%   +0.12%     
+ Complexity     1143      1142       -1     
=============================================
  Files            79        79              
  Lines          3091      3088       -3     
=============================================
+ Hits           3087      3088       +1     
+ Misses            4         0       -4     

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

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

@TomHAnderson
TomHAnderson merged commit b2fe6b2 into API-Skeletons:14.0.x Oct 2, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant