Closed superchilled closed 2 years ago
Merging #246 (fe1b55d) into dev (0c4059b) will not change coverage. The diff coverage is
n/a
.:exclamation: Current head fe1b55d differs from pull request most recent head 00aca9d. Consider uploading reports for the commit 00aca9d to get more accurate results
@@ Coverage Diff @@
## dev #246 +/- ##
=======================================
Coverage 97.71% 97.71%
=======================================
Files 70 70
Lines 1400 1400
=======================================
Hits 1368 1368
Misses 32 32
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
Reason for this PR
The Verify API has a new possible error response for the
https://api.nexmo.com/verify/json
, which can return a response with and without certain properties in the response body. This PR is to ensure that the existing implementation of Verify in the Ruby SDK handles the various permutations of this response correctly. See this JIRA ticket for additional context.What this PR does
verify
endpoint already added in #241, this PR adds three more tests toverify_test.rb
for the forpsd2
endpoint:psd2
endpoint withnetwork
but notrequest_id
psd2
endpoinwithrequest_id
but notnetwork
psd2
endpoinwithrequest_id
andnetwork