Enterprise-CMCS / eAPD

CMS (Centers for Medicare and Medicaid Services) eAPD - Modernizing the APD experience
https://eapd.cms.gov
Other
58 stars 25 forks source link

changes needed for setting up dev Okta account #4676

Closed thetif closed 1 year ago

thetif commented 1 year ago

Description

Due to changes in the project, we want to make sure the code is as open source as possible. One of those current blockers is needing an Okta account. This PR is updating things needed to follow the new instructions for creating a dev Okta account, which is free for devs trying to learn Okta or using Okta for personal development reasons.

The main change is adding an new environment variable for OKTA_AUDIENCE and updating it in all the places that need to change to add an env.

I also added a few more test users and changed the seed to use Pending Admin instead of State Admin for the Federal Admin panel tests.

Steps to manually verify this change

  1. Login and log out as a couple of the shared accounts

This pull request is ready to code review when

This pull request is ready to test when

This pull request is ready to review when QA has

This pull request can be merged when

codecov[bot] commented 1 year ago

Codecov Report

Merging #4676 (8668609) into main (7e92d4e) will increase coverage by 0.00%. The diff coverage is 37.50%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #4676   +/-   ##
=======================================
  Coverage   91.76%   91.77%           
=======================================
  Files         286      286           
  Lines        9290     9288    -2     
  Branches     1894     1894           
=======================================
- Hits         8525     8524    -1     
+ Misses        736      735    -1     
  Partials       29       29           
Flag Coverage Δ
api ∅ <ø> (∅)
common 98.75% <ø> (ø)
web 91.39% <37.50%> (+<0.01%) :arrow_up:
Impacted Files Coverage Δ
web/src/pages/login/SelectAffiliation.js 97.29% <ø> (ø)
e2e/cypress/support/commands.js 73.38% <37.50%> (+0.17%) :arrow_up:

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 7e92d4e...8668609. Read the comment docs.

cms-eapd-bot commented 1 year ago

This deploy was cleaned up.