Skip to content

Commit

Permalink
test: merge TestAPIProxy into TestAPI
Browse files Browse the repository at this point in the history
Now that the SDK‒proxy connection is itself instrumented, there's only
one path to test, with no special error-handling logic required.

I considered merging TestAPI and TestShared as well, now that (without
TestAPIProxy) TestAPI is the only subclass of TestShared.  But
reorganizing the alphabetized helpers in TestShared versus the
strictly-sequenced TestAPI methods can wait.
  • Loading branch information
cfm committed Feb 6, 2024
1 parent 4ccc325 commit e717d85
Show file tree
Hide file tree
Showing 97 changed files with 14,003 additions and 5,474 deletions.
1 change: 0 additions & 1 deletion client/tests/sdk/data/setup_method.json

This file was deleted.

22 changes: 22 additions & 0 deletions client/tests/sdk/data/setup_method.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
interactions:
- request:
body: '{"username": "journalist", "passphrase": "correct horse battery staple
profanity oil chewy", "one_time_code": "253705"}'
headers: {}
method: POST
uri: api/v1/token
response: !!python/object:securedrop_client.sdk.JSONResponse
data:
expiration: '2024-02-02T04:32:07.217091Z'
journalist_first_name: null
journalist_last_name: null
journalist_uuid: 8248d25d-f6f6-4ad3-a6d4-a287641d533c
token: IldNSTRhZFdmaDRQb0I3MzNYNmR0NkR4OFFrNDhsZEl0U05teVFyZEs4UTQi.ZbxUJw.JbZXOTTX32PYHCLs5X8sr7s10KI
headers:
connection: close
content-length: '265'
content-type: application/json
date: Fri, 02 Feb 2024 02:32:07 GMT
server: Werkzeug/2.2.3 Python/3.8.10
status: 200
version: 1
34 changes: 0 additions & 34 deletions client/tests/sdk/data/test-badotp.yml

This file was deleted.

33 changes: 0 additions & 33 deletions client/tests/sdk/data/test-badpassword.yml

This file was deleted.

34 changes: 0 additions & 34 deletions client/tests/sdk/data/test-baduser.yml

This file was deleted.

Loading

0 comments on commit e717d85

Please sign in to comment.