-
Notifications
You must be signed in to change notification settings - Fork 21
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Numberverify verify Test definition #124
Merged
Merged
Changes from 1 commit
Commits
Show all changes
28 commits
Select commit
Hold shift + click to select a range
a03682d
initial commit
AxelNennker 849e20d
use Scenario Outline for common tests
AxelNennker 6e52ca9
new case access token expired
AxelNennker bcc87e9
rename NumberVerification.feature to NumberVerification_verify.feature
AxelNennker cfe09c0
add verify
AxelNennker 564359d
remove x-correlator format test
AxelNennker 9fe46ce
remove CIBA
AxelNennker 79181e6
some lines about testing environment and assets
AxelNennker da3c3bb
single cause of failure
AxelNennker d012be8
better text if phone number from access token and from parameter do n…
AxelNennker 82f02e9
And the request body does not have the field phoneNumber
AxelNennker ada4a18
NumberVerificationMatchResponse
AxelNennker 2c488bf
remove over a mobile connection
AxelNennker fc899f8
remove mobile connection
AxelNennker e0e2c58
initial commit of "share" test definitions
AxelNennker fcea400
add hashed numbers test definitions
AxelNennker 50949f4
re-added "And the connection the request is sent over originates from…
AxelNennker 796b9c6
Update NumberVerification_device_phone_number_share.feature
AxelNennker 414a57f
Update NumberVerification_verify.feature
AxelNennker 6d6d58f
added test NumberVerification_verify203_both_phone_number_and_hashed_…
AxelNennker f8513aa
missing scope error is 403
AxelNennker 4c775ca
missing scope is error code 403
AxelNennker 81072eb
Update code/Test_Definitions/NumberVerification_verify.feature
AxelNennker f3fbd39
added NumberVerification_phone_number_share203_no_phonenumber_associa…
AxelNennker fa334b2
addedd NumberVerification_phone_number_verify204_no_phonenumber_assoc…
AxelNennker 393d7da
added @NumberVerification_phone_number_verify205_must_have_used_netwo…
AxelNennker a61b1c4
fix indentation
AxelNennker 5b35bde
added @NumberVerification_phone_number_share205_must_have_used_networ…
AxelNennker File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
added test NumberVerification_verify203_both_phone_number_and_hashed_…
…in_request
- Loading branch information
commit 6d6d58f2611fbebb1eb1a89881e347b5a46dde3c
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -156,6 +156,21 @@ Feature: Camara Number Verification API verify | |
And the response property "$.code" is "AUTHENTICATION_REQUIRED" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. or UNAUTHENTICATED, still don't know which one should apply in this case |
||
And the response property "$.message" is "New authentication is required." | ||
AxelNennker marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
|
||
@NumberVerification_verify203_both_phone_number_and_hashed_in_request | ||
Scenario: verify phone number but no phonenumber in request | ||
Given they use the base url | ||
And the resource is "/verify" | ||
And one of the scopes associated with the access token is number-verification:verify | ||
When the HTTPS "POST" request is sent | ||
And the connection the request is sent over originates from a device with NUMBERVERIFY_VERIFY_MATCH_PHONENUMBER1 | ||
And the request body has the field phoneNumber with a value of NUMBERVERIFY_VERIFY_MATCH_PHONENUMBER1 | ||
And the request body has the field hashedPhoneNumber with a value of NUMBERVERIFY_VERIFY_MATCH_PHONENUMBER_HASHED1 | ||
And the response header "x-correlator" has same value as the request header "x-correlator" | ||
And the response header "Content-Type" is "application/json" | ||
And the response body complies with the OAS schema at "/components/schemas/NumberVerificationMatchResponse" | ||
Then the response status code is 400 | ||
And the response property "$.status" is 400 | ||
And the response property "$.code" is "INVALID_ARGUMENT" | ||
And the response property "$.message" contains a user friendly text | ||
|
||
|
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Test included for veryfy operation:
Pending test to be included:
cc: @bigludo7 @AxelNennker