Some problems with tests are there.

If we go this way, we could probably use 301 redirects instead of 302

I think redirects in APIs are a bad idea and likely to cause confusion.

I do think this one's a good idea. Maybe also add it for /pull-request/ ?

So you would favor the status-quo or the discrepancy between UI and API ?

Correct. I think that's reasonable, due to the differences in clients using them.
The API tends to be automated clients, often with preconfigured URLs, whereas users sometimes try to enter the url themselves, and I think we can help them along to find what they want there.

I think it is better and nicer to override the setUp() method of the class instead of calling a test.
you have an example here https://pagure.io/pagure/blob/master/f/tests/test_pagure_lib.py#_152

Correct. I think that's reasonable, due to the differences in clients using them. The API tends to be automated clients, often with preconfigured URLs, whereas users sometimes try to enter the url themselves, and I think we can help them along to find what they want there.

@puiterwijk but, there shouldn't be any client which are currently using this api endpoint, no?

I am tempted to close this PR, it has not been updated in 10 months and just seats in the PR queue.

What do you think?

yeah +1 for closing. we can always reopen if work starts on it again.