Test: improve error logging in performance tests

privacybydesign · Oct 25, 2023 · 5484d91 · 5484d91
1 parent be5f600
commit 5484d91
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/testdata/performance/irma-server.js b/testdata/performance/irma-server.js
@@ -13,10 +13,11 @@ export const options = {
 
 function checkResponse(response, expectedOutput = '') {
   const checkOutput = check(response, {
+    'verify response': (r) => r.error === '',
     'verify status code': (r) => r.status === 200,
-    'verify body': (r) => r.body.includes(expectedOutput),
+    'verify body': (r) => r.body != null && r.body.includes(expectedOutput),
   });
-  if (!checkOutput) fail(`unexpected response: status ${response.status}`);
+  if (!checkOutput) fail(`unexpected response: status ${response.status}, error "${response.error}", body "${response.body}"`);
 }
 
 export default function () {

diff --git a/testdata/performance/keyshare-server.js b/testdata/performance/keyshare-server.js
@@ -16,10 +16,11 @@ export const options = {
 
 function checkResponse(response, expectedOutput = '') {
   const checkOutput = check(response, {
+    'verify response': (r) => r.error === '',
     'verify status code': (r) => r.status === 200,
-    'verify body': (r) => r.body.includes(expectedOutput),
+    'verify body': (r) => r.body != null && r.body.includes(expectedOutput),
   });
-  if (!checkOutput) fail(`unexpected response: status ${response.status}`);
+  if (!checkOutput) fail(`unexpected response: status ${response.status}, error "${response.error}", body "${response.body}"`);
 }
 
 export function setup() {
@@ -41,6 +42,7 @@ export function setup() {
         'Content-Type': 'application/json',
       },
     });
+    checkResponse(registerResp);
 
     const sessionResp = http.get(registerResp.json().u, {
       headers: {