search

vapor / postgres-nio

🐘 Non-blocking, event-driven Swift client for PostgreSQL.
https://api.vapor.codes/postgresnio/documentation/postgresnio/
MIT License
305 stars 70 forks source link

Move PostgresFrontendMessage to tests #399

Closed fabianfett closed 11 months ago

fabianfett commented 11 months ago

We should remove everything that we don't need from PostgresNIO. PostgresFrontendMessage is only required in tests now. So let's move it to tests.

codecov-commenter commented 11 months ago

Codecov Report

Merging #399 (8e69f43) into main (12584c6) will decrease coverage by 0.05%. The diff coverage is 100.00%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #399 +/- ## ========================================== - Coverage 49.14% 49.10% -0.05% ========================================== Files 109 108 -1 Lines 8934 8845 -89 ========================================== - Hits 4391 4343 -48 + Misses 4543 4502 -41 ``` | [Files Changed](https://app.codecov.io/gh/vapor/postgres-nio/pull/399?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=vapor) | Coverage Δ | | |---|---|---| | [...s/PostgresNIO/New/Extensions/ByteBuffer+PSQL.swift](https://app.codecov.io/gh/vapor/postgres-nio/pull/399?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=vapor#diff-U291cmNlcy9Qb3N0Z3Jlc05JTy9OZXcvRXh0ZW5zaW9ucy9CeXRlQnVmZmVyK1BTUUwuc3dpZnQ=) | `100.00% <ø> (ø)` | | | [...stgresNIO/New/PostgresFrontendMessageEncoder.swift](https://app.codecov.io/gh/vapor/postgres-nio/pull/399?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=vapor#diff-U291cmNlcy9Qb3N0Z3Jlc05JTy9OZXcvUG9zdGdyZXNGcm9udGVuZE1lc3NhZ2VFbmNvZGVyLnN3aWZ0) | `100.00% <100.00%> (ø)` | | ... and [1 file with indirect coverage changes](https://app.codecov.io/gh/vapor/postgres-nio/pull/399/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=vapor)