#38 Test Results for External Link (NOT WIKI TC) is not submitted
Closed: Fixed 6 months ago by jskladan. Opened a year ago by sumantrom.

When you have a test case which point to test case which is not a QA:Test Case, but a Doc or any external link, the app wont show the submitted result.

Ex: https://testdays.fedoraproject.org/events/179 --> Add results for "tutorials" --> The results gets submited but it doesnt show up.


This was a supported feature till https://testdays.fedoraproject.org/events/169. Watch out for the AWS column.

@frantisekz @jskladan ^^ please can this be fixed?

This is a regression and we are about to start Test Day season soon... it will be awesome to have this fixed!!

I agree this should be fixed, but at the same time I don't think this has Critical priority, because it can be very easily worked around by creating a wiki page which points to the external document. Sure, not ideal, but very simple.

@sumantrom I've finally had time to look into this. Nothing changed between the events, so the "external testcase urls" are supported still the same as before - which can be checked by submitting a result in the /169 AWS column (which gets successfully submitted and shown).

I tested this in here: https://testdays.fedoraproject.org/events/2 adding the "working" AWS test and the "now" not working Tutorials url (see metadata at: https://fedoraproject.org/w/index.php?title=User:Jskladan/Sandbox:TestdayAppMockTestday&oldid=723590 ) Once again, the AWS works.

Testdays was not implemented with external urls in mind (way back when we only used Wiki), so it is basically an accident (read - oversight/bug in the original code :D) that some work.

The "not working" testcase is caused by the trailing slash in the url - there is some weird interaction between testdays and resultsdb, that ends up eating the data (I don't have access to the resutlsdb instance in question, so can't really debut).

Changing the url to not have trailing slashes makes the external url "work".

I've pushed a rudimentary "hotfix": https://pagure.io/fedora-qa/testdays-web/c/a88f16b1ac301882ba4e865e16352e26fbb3f3b4

Tested with this metadata: https://fedoraproject.org/wiki/User:Jskladan/Sandbox:TestdayAppMockTestday (see "external 4") and the result submission works as expected. So this should be OK for any new Test Days.

Metadata Update from @jskladan:
- Issue assigned to jskladan

6 months ago

Metadata Update from @jskladan:
- Custom field story_points adjusted to 1

6 months ago

Metadata Update from @jskladan:
- Issue close_status updated to: Fixed
- Issue status updated to: Closed (was: Open)

6 months ago

Log in to comment on this ticket.

Metadata