search

corona-warn-app / cwa-quick-test-backend

Apache License 2.0
13 stars 8 forks source link

Testresult timestamp is not set with SC parameter #115

Closed kerstin-oppermann-tsi closed 3 years ago

kerstin-oppermann-tsi commented 3 years ago

Configuration

WRU, URL: https://cff4f7147260.schnelltestportal.de/c19quicktest

Steps:

  1. Set quicktest result with parameter sc with postman:

POST https://testresult-5c253f79.coronawarn.app/api/v1/quicktest/results

Request: { "testResults": [ { "id": "f6f93810706dd586b84467950c8585f58bb928aa485aeeb556d66fb12ba55973", "result": 6, "sc": 1623058590 } ] {color:#4c9aff}}

1623058590 --> 07.06.2021 - 11:36:30

  1. Check parameter SC with postman:

POST https://testresult-5c253f79.coronawarn.app/api/v1/app/result

Response: {"testResult":6,"sc":1623069416}

1623069416 --> 07.06.2021 - 14:36:56 (Time when the quicktest result was set)

Expected:

SC has same timestamp as set in step1

Actual:

SC has timestamp when quicktest result was set in step1

a-trzewik commented 3 years ago

The ticket is placed in wrong project. The testreslt server is here https://github.com/corona-warn-app/cwa-testresult-server

bergmann-dierk commented 3 years ago

Following requests were sent:

`POST /api/v1/quicktest/results HTTP/1.1

{ "testResults": [ { "id": "f6f93810706dd586b84467950c8585f58bb928aa485aeeb556d66fb12ba55973", "result": 6, "sc": 1623154064 } ] } `

`POST /api/v1/app/result HTTP/1.1

{ "id": "f40f49d9e0961d71970455a42dd91d5f8e54e51550bbce3e6e457fc92c6d779a" }`

The ids did not match, therefore a new TestResult was created during the second request.

bergmann-dierk commented 3 years ago

@kerstin-oppermann-tsi The issuer of the bugreport will retest with different data. If the test runs successfully this issue is obsolete.

ascheibal commented 3 years ago

Issue was not reproduceable anymore. Will be closed