Skip to content

Remove a bogus test that only pass on retry

Rémy Coutable requested to merge remove-bogus-test into master

What does this MR do?

2bc0f0cf introduced this test and it probably passed at the time due to the return if peek_request? || json_request? || !request.get? condition in the controller concern, but then this line was removed in 39d06df4 and the test continued to pass only after a retry on the CI (which is in itself problematic).

I think the good call is to just remove the test here since the controller concern doesn't have any Peek-specific code anymore. 🤷🏻

Does this MR meet the acceptance criteria?

Conformity

Edited by Rémy Coutable

Merge request reports

Loading