Closed dpDesignz closed 4 years ago
@techno-express ok, I finally got around to writing a phpunit test. Let me know if you're happy with this. It passes when I run it. 😄
Merging #180 into master will increase coverage by
1.34%
. The diff coverage is100%
.
@@ Coverage Diff @@
## master #180 +/- ##
============================================
+ Coverage 83.56% 84.91% +1.34%
- Complexity 788 793 +5
============================================
Files 14 14
Lines 2002 2002
============================================
+ Hits 1673 1700 +27
+ Misses 329 302 -27
Impacted Files | Coverage Δ | Complexity Δ | |
---|---|---|---|
lib/Database.php | 78.57% <100%> (ø) |
17 <13> (+5) |
:arrow_up: |
lib/Database/ez_sqlsrv.php | 74.35% <0%> (ø) |
65% <0%> (ø) |
:arrow_down: |
lib/Config.php | 92.42% <0%> (ø) |
34% <0%> (ø) |
:arrow_down: |
lib/ezFunctions.php | 77.12% <0%> (+2.61%) |
0% <0%> (ø) |
:arrow_down: |
lib/ezQuery.php | 75.15% <0%> (+4.88%) |
204% <0%> (ø) |
:arrow_down: |
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 e73a3de...6ece68e. Read the comment docs.
Fixes #174 Removed check for $columnFields being empty, and added dots between table and column names in the query builder. Also, fixed a couple of phpunit depreciation warnings.