Closed sroebert closed 7 years ago
Merging #40 into master will decrease coverage by
10.07%
. The diff coverage is74.84%
.
@@ Coverage Diff @@
## master #40 +/- ##
===========================================
- Coverage 90.05% 79.98% -10.08%
===========================================
Files 17 10 -7
Lines 1970 1004 -966
===========================================
- Hits 1774 803 -971
- Misses 196 201 +5
Impacted Files | Coverage Δ | |
---|---|---|
Sources/PostgreSQL/Database.swift | 100% <100%> (+15.38%) |
:arrow_up: |
Sources/PostgreSQL/Bind/BinaryUtils.swift | 98.1% <100%> (ø) |
|
Sources/PostgreSQL/ConnectionInfo.swift | 100% <100%> (+33.33%) |
:arrow_up: |
Sources/PostgreSQL/Bind/FieldType.swift | 25.24% <25.24%> (ø) |
|
Sources/PostgreSQL/Error.swift | 17.5% <37.5%> (+12.23%) |
:arrow_up: |
Sources/PostgreSQL/Result.swift | 65.62% <58.75%> (-26.05%) |
:arrow_down: |
Sources/PostgreSQL/Bind/Bind.swift | 77.24% <77.24%> (ø) |
|
Sources/PostgreSQL/Bind/Bind+Node.swift | 94.79% <94.79%> (ø) |
|
Sources/PostgreSQL/Connection.swift | 95.89% <96.55%> (+11.42%) |
:arrow_up: |
Tests/PostgreSQLTests/MiscTests.swift | ||
... and 3 more |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 82b28f6...11c76eb. Read the comment docs.
I approve!
This pull request contains more changes to more closely match Vapor MySQL. This will also be needed to perform further updates to PostgreSQL Driver.
Changes:
Database
toConnection
, leavingDatabase
to be a simple class containing all the information to connect to a database.PostgreSQLError
and replaced all uses ofDatabaseError
with it.listen
function to stop listening.listen
function from crashing with a fatal error. Now an error is returned instead.Date
, instead of parsing the date into a string.