Why the change in the port?

I recommend splitting this into separate tests

I recommend splitting this into separate tests

I recommend splitting this into separate tests

I recommend splitting this into separate tests

Same as above, maybe consider creating a test class for every function.

Same as above

Same as above

Same as above

I don't want to comment on every method in this test, but you should consider creating test class for every function and test every case in separate method. It will be much more readable and you will quickly find the failing test.

It's the default port used by flask, fixing the doc here :)

I don't want to comment on every method in this test, but you should consider creating test class for every function and test every case in separate method. It will be much more readable and you will quickly find the failing test.

Doable but this will increase the diff and isn't quite in the scope of this PR :)

Rebased on the top of develop :)

I don't want to comment on every method in this test, but you should consider creating test class for every function and test every case in separate method. It will be much more readable and you will quickly find the failing test.

Doable but this will increase the diff and isn't quite in the scope of this PR :)

Let's create a issue to track this work.

Otherwise LGTM

I'm trying to fix the conflict, but +1 on a ticket to refactor the test suite :)

There are still some failing tests it seems (https://jenkins-fedora-apps.apps.ci.centos.org/blue/organizations/jenkins/elections/detail/oidc/1/pipeline#step-43-log-28)

The tests passed, but the run failed because of diff-cover: https://jenkins-fedora-apps.apps.ci.centos.org/blue/organizations/jenkins/elections/detail/oidc/7/pipeline

:+1:

Thanks! :)