patchwork icon indicating copy to clipboard operation
patchwork copied to clipboard

Query test results through a new GET request

Open lsandov1 opened this issue 9 years ago • 3 comments

We can now post results through

POST /api/1.0/series/(int: series_id)/revisions/(int: version)/test-results/

but we cannot query these. We should have the same request but using GET

GET /api/1.0/series/(int: series_id)/revisions/(int: version)/test-results/

lsandov1 avatar Dec 09 '15 22:12 lsandov1

I can see why this would be useful.

I also would like to add a new field in the revision object that would provide the consolidated test results across all test results posted for that revision. Eg. if you have 2 test results, one passing, one with a warning, the revision would have a 'warning' test status. Would that be enough for what you need?

dlespiau avatar Dec 10 '15 12:12 dlespiau

@dlespiau yes, that also would be useful.

So there would be a query for the tests done for a particular series's revision and a query for a particular series, the latter proving all tests done for each serie's revision. Does this make sense?

lsandov1 avatar Dec 10 '15 15:12 lsandov1

huum, the information on the series is just the "test status of the series", the information that is useful to know if a series passes all the tests or not. So it's the consolidated result of tests on the latest revision. That's really the information we need to decide if a series is good to go (ie is the last revision passing all the tests?)

dlespiau avatar Dec 10 '15 15:12 dlespiau