Closed amiecorso closed 4 years ago
Merging #243 into master will increase coverage by
1.65%
. The diff coverage is94.92%
.
@@ Coverage Diff @@
## master #243 +/- ##
==========================================
+ Coverage 88.03% 89.69% +1.65%
==========================================
Files 42 42
Lines 861 941 +80
==========================================
+ Hits 758 844 +86
+ Misses 103 97 -6
Impacted Files | Coverage Δ | |
---|---|---|
...ingKit/PayID/JavaScript/JavaScriptPayIDUtils.swift | 100.00% <ø> (ø) |
|
XpringKit/PayID/XRPPayIDClient.swift | 85.00% <82.60%> (+3.18%) |
:arrow_up: |
XpringKit/PayID/PayIDClient.swift | 96.55% <95.65%> (+15.30%) |
:arrow_up: |
XpringKit/XRP/DefaultXRPClient.swift | 98.60% <100.00%> (+1.43%) |
:arrow_up: |
...XRP/Model/XRPPayment+Org_Xrpl_Rpc_V1_Payment.swift | 100.00% <100.00%> (ø) |
|
...l/XRPTransaction+Org_Xrpl_Rpc_V1_Transaction.swift | 96.00% <100.00%> (+0.44%) |
:arrow_up: |
XpringKit/XRP/ReliableSubmissionXRPClient.swift | 94.23% <100.00%> (+0.11%) |
:arrow_up: |
XpringKit/XRP/XRPClient.swift | 100.00% <100.00%> (ø) |
|
XpringKit/Xpring/XpringClient.swift | 100.00% <100.00%> (+5.55%) |
:arrow_up: |
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 72310f9...d473c54. Read the comment docs.
High Level Overview of Change
Update xpring-common-js to new release. Update PayIDUtilsTest to reflect new PayID parsing rules that allow any number of dollar signs to appear in the path. Updates the name of underlying javascript resources to reflect new idiomatic casing in xpring-common-js.
Type of Change
Test Plan
CI