Unit test for Support Number Challenge for Okta MFA (#877)

[silver886]

Unit test for #877.

#877 (comment)
#877 (comment)

[silver886]

This PR won't pass unit test unless #877 is in place.

[mapkon]

@silver886 The tests are failing on this one.

[silver886]

Yeah, because the actual code that implements the feature is #877.

Should I copy @muramasa64 's code into this PR?

[mapkon]

Yeah, because the actual code that implements the feature is #877.

Should I copy @muramasa64 's code into this PR?

Yeah - and probably close that PR since it will no longer be required.

[codecov-commenter]

Codecov Report

Merging #1116 (edf7c38) into master (b63aed1) will increase coverage by 0.66%.
The diff coverage is 100.00%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@            Coverage Diff             @@
##           master    #1116      +/-   ##
==========================================
+ Coverage   36.52%   37.19%   +0.66%     
==========================================
  Files          53       53              
  Lines        7954     7959       +5     
==========================================
+ Hits         2905     2960      +55     
+ Misses       4675     4619      -56     
- Partials      374      380       +6     

Flag	Coverage Δ
unittests	37.19% <100.00%> (+0.66%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed	Coverage Δ
pkg/provider/okta/okta.go	26.52% <100.00%> (+4.33%)	⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more