Fix scenario tests (openwallet-foundation#3231)

Signed-off-by: Jamie Hale <[email protected]>
ff137 · Oct 22, 2024 · 6712367 · 6712367
1 parent c0b4715
commit 6712367
Show file tree

Hide file tree

Showing 6 changed files with 7 additions and 12 deletions.
diff --git a/scenarios/README.md b/scenarios/README.md
@@ -10,7 +10,7 @@ Every test example will have a docker-compose.yml file for all the agents and se
 - From the scenarios directory. 
 - Make sure the local acapy image is up to date. 
  - `cd ..`
- - `docker build -t acapy-test -f ../docker/Dockerfile.run .`
+ - `docker build -t acapy-test -f ./docker/Dockerfile.run .`
  - `cd scenarios`
 - Navigate to the base example. `cd examples/simple`
 - `docker compose up`
@@ -19,7 +19,7 @@ To run all the tests with pytest:
 - From the scenarios directory. 
 - Make sure the local acapy image is up to date. 
  - `cd ..`
- - `docker build -t acapy-test -f ../docker/Dockerfile.run .`
+ - `docker build -t acapy-test -f ./docker/Dockerfile.run .`
  - `cd scenarios`
 - `poetry run pytest -m examples`
 - TODO: easily run individual tests with pytest.
diff --git a/scenarios/examples/connectionless/example.py b/scenarios/examples/connectionless/example.py
@@ -59,8 +59,7 @@ async def icv2():
                         "role": "ENDORSER",
                     },
                 ) as resp:
-                    if resp.ok:
-                        return await resp.json()
+                    assert resp.ok
 
             await alice.post("/wallet/did/public", params=params(did=public_did.did))
 

diff --git a/scenarios/examples/json_ld/example.py b/scenarios/examples/json_ld/example.py
@@ -77,8 +77,7 @@ async def main():
                                 "role": "ENDORSER",
                             },
                         ) as resp:
-                            if resp.ok:
-                                return await resp.json()
+                            assert resp.ok
 
                     await alice.post(
                         "/wallet/did/public", params=params(did=public_did.did)

diff --git a/scenarios/examples/multitenancy/example.py b/scenarios/examples/multitenancy/example.py
@@ -72,8 +72,7 @@ async def main():
                         "role": "ENDORSER",
                     },
                 ) as resp:
-                    if resp.ok:
-                        return await resp.json()
+                    assert resp.ok
 
             await alice.post("/wallet/did/public", params=params(did=public_did.did))
         _, cred_def = await indy_anoncred_credential_artifacts(

diff --git a/scenarios/examples/presenting_revoked_credential/example.py b/scenarios/examples/presenting_revoked_credential/example.py
@@ -72,8 +72,7 @@ async def main():
                         "role": "ENDORSER",
                     },
                 ) as resp:
-                    if resp.ok:
-                        return await resp.json()
+                    assert resp.ok
 
             await alice.post("/wallet/did/public", params=params(did=public_did.did))
         schema, cred_def = await indy_anoncred_credential_artifacts(

diff --git a/scenarios/examples/self_attested/example.py b/scenarios/examples/self_attested/example.py
@@ -58,8 +58,7 @@ async def main():
                         "role": "ENDORSER",
                     },
                 ) as resp:
-                    if resp.ok:
-                        return await resp.json()
+                    assert resp.ok
 
             await alice.post("/wallet/did/public", params=params(did=public_did.did))
         schema, cred_def = await indy_anoncred_credential_artifacts(