vapor / fluent-kit

Swift ORM (queries, models, and relations) for NoSQL and SQL databases
MIT License
218 stars 116 forks source link

Add missing info to missingParent error, fix ID compare for composite IDs #514

Closed gwynne closed 2 years ago

gwynne commented 2 years ago
codecov-commenter commented 2 years ago

Codecov Report

Merging #514 (ab86f3e) into main (118870c) will increase coverage by 3.39%. The diff coverage is 78.37%.

@@            Coverage Diff             @@
##             main     #514      +/-   ##
==========================================
+ Coverage   43.06%   46.46%   +3.39%     
==========================================
  Files         109       98      -11     
  Lines        5995     5514     -481     
==========================================
- Hits         2582     2562      -20     
+ Misses       3413     2952     -461     
Flag Coverage Δ
unittests 46.46% <78.37%> (+3.39%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
Sources/FluentKit/FluentError.swift 31.57% <0.00%> (ø)
Sources/FluentKit/Properties/Parent.swift 27.65% <0.00%> (-1.56%) :arrow_down:
Sources/FluentKit/Operators/ValueOperators.swift 70.00% <100.00%> (+17.05%) :arrow_up:
...rces/FluentBenchmark/SolarSystem/SolarSystem.swift
...ntBenchmark/SolarSystem/GalacticJurisdiction.swift
...ces/FluentBenchmark/SolarSystem/Jurisdiction.swift
Sources/FluentBenchmark/SolarSystem/Governor.swift
Sources/FluentBenchmark/SolarSystem/Moon.swift
Sources/FluentBenchmark/SolarSystem/Star.swift
Sources/FluentBenchmark/SolarSystem/Galaxy.swift
... and 4 more
VaporBot commented 2 years ago

These changes are now available in 1.29.0