uc-cdis / fence / 10769

75%

push

travis-ci-com

Update fence to now include Okta as an OIDC Identity Provider (#890)

* added configurations for Okta and Generic IdPs

* Update .gitignore

* fixing conflict

* fixing blank line

* updates for testing framework

* fix okta URL

* Cleanup okta code and testing scripts

* cleanup of okta code

Removed some unessessary generic settings.

* revert testing config to default google

revert to how it is in source before okta tests.

* Delete build.sh

clean up of my build file.

* Update okta discovery URL

Adding the discovery URL for okta in test config to see if that solves issue in testing scripts.

* update discovery URL in test script

changed the discoveryURL to have {tenant} instaed of real value for tests.

* update to test_login_redirect.py scripts

Updates to the testing scripts to return a status code for the Okta discovery URL which is not valid. Okta doesn't have a general discover URL,  only specific to tenants, so can't use one in testing framework.

* removed unused modules from Okta files.

removed unused modules from fence/blueprints/login/okta.py and fence/resources/openid/okta_oauth2.py

* updated okta_oauth2.py to remve trailing white space

As reported by codacy, removed a trailing whitespace.

* fix merge conflict issues on test-fence-config.yaml

PR has fallen behind current main changes. Updated test-fence-config from main branch to get back in sync, now re-applying changes to test-fence-config.yaml for Okta configuration.

* removing blank spaces from test_login_redirect.py

Removing extra blank lines from test_login_redirect.py

* adding okta as an IDP to test_audit_service.py

Added okta as an idp to test_audit_services.py

* Update to testing code per requested changes

Update the mock response for username and also updated comments for okta IDP on test-fence-config

* Fixing formatting issues as reported by black output

per recommendations, ran "black ." to fix ... (continued)

36 of 36 new or added lines in 4 files covered. (100.0%)

5911 of 8393 relevant lines covered (70.43%)

0.7 hits per line