OKTA-289662: Add SMS Templates API Integration Tests #395
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
arvindkrishnakumar-okta
requested review from
bdemers,
robertjd,
bretterer and
laura-rodriguez
arvindkrishnakumar-okta
changed the title
arvindkrishnakumar-okta
requested review from
bryanapellanes-okta,
shuowu and
aarongranick-okta
bdemers
reviewed
Jun 4, 2020
integration-tests/src/test/groovy/com/okta/sdk/tests/it/SmsTemplateIT.groovy
Outdated
Show resolved
Hide resolved
bdemers
reviewed
Jun 4, 2020
integration-tests/src/test/groovy/com/okta/sdk/tests/it/SmsTemplateIT.groovy
Outdated
Show resolved
Hide resolved
bdemers
reviewed
Jun 4, 2020
integration-tests/src/test/groovy/com/okta/sdk/tests/it/SmsTemplateIT.groovy
Outdated
Show resolved
Hide resolved
bdemers
reviewed
Jun 4, 2020
* Increase base client connection timeout Since we have lot of flaky ITs that break due to a laggy backend, we should increase the client connection timeout to prevent IT failures in Oktapreview test org. This timeout value could be reduced if we migrate to prod org (which is planned). * Update DefaultClientBuilderTest.groovy * Update DefaultClientBuilderTest.groovy
…template-integration-tests
Codecov Report
@@ Coverage Diff @@
## dev_openapi_v2.0.0_major_rel #395 +/- ##
==================================================================
- Coverage 71.09% 70.91% -0.18%
+ Complexity 1131 1127 -4
==================================================================
Files 120 120
Lines 3629 3624 -5
Branches 567 566 -1
==================================================================
- Hits 2580 2570 -10
- Misses 861 868 +7
+ Partials 188 186 -2
Continue to review full report at Codecov.
|
…template-integration-tests
arvindkrishnakumar-okta
force-pushed
the
okta-289662-add-sms-template-integration-tests
branch
from
e69e775
to
da4b31f
Compare
…template-integration-tests
arvindkrishnakumar-okta
force-pushed
the
okta-289662-add-sms-template-integration-tests
branch
from
e4ac87b
to
dc7fe16
Compare
bretterer
approved these changes
Jun 10, 2020
arvindkrishnakumar-okta
force-pushed
the
okta-289662-add-sms-template-integration-tests
branch
from
363c560
to
61fc6e6
Compare
arvindkrishnakumar-okta
deleted the
okta-289662-add-sms-template-integration-tests
branch
arvindkrishnakumar-okta
added a commit
that referenced
this pull request
Jul 7, 2020
* upgrade to openapi v2.0.0 (#380) * openapi spec update - remove 200 response from generateCsrForApplication * Update OktaOrgCleaner.groovy * Fix flaky GroupsIT#groupUserOperationsTest test * Update GroupsIT.groovy * OKTA-279039: Fix flaky ApplicationsIT associateUserWithApplication test * Update GroupsIT.groovy * Addressed review comment * OKTA-289656: Add Applications API Integration Tests * OKTA-289655: Add Admin role API Integration Tests * Open API Csr rename update (#394) * openapi csr rename update * Merge pull request #396 * revert csr model rename (#398) * revert csr model rename * open api spec update (#401) * open api spec update * Open API spec - fix binding enum value for ProtocolEndpoint model (#406) * OKTA-289662: Add SMS Templates API Integration Tests (#395) * OKTA-289665: Add Linked Objects Integration Tests (#404) * Updated Licence Header in LinkedObjectsIT (#410) * OKTA-289668: Add User Type Integration Tests (#405) * Open API Update - Sync with #238 #239 (#411) * sync with open api pr #240 (#413) * Open API Update - Sync with #241 (#412) * Open API Update - Sync with #242 (#414) * Open API Update - Sync with Open API PR 243 (#416) * OKTA-289657: Add Factors API Integration Tests (#391) * OKTA-289659: Add Event Hooks Integration Tests (#399) * OKTA-289659: Add Authorization Server Integration Tests (#402) * OKTA-289660: Add Inline Hooks Integration Tests (#400) * Open API Update - Sync with Open API PR 244 (#418) * fixed copy paste error * OKTA-289661: Add Feature API Integration Tests (#397) * Open API Update - Sync with Open API PR 249 (#420) * OKTA-289663: Add IdP Integration Tests (#409) * resolved merge conflicts with master changes * removed Default prefix from few Idp builders * added swagger code gen check for backward compatibility requirements per review comments * fixed findbugs error * updates per swagger regen code and one other update to Idp builder * fixed expiresAt param type in spec * deprecated old classes * refactored InlineHooksIT - deactivate created hooks so it can be cleaned up after test * Remove deprecated code (#426) * Remove deprecated code * fix typo * fixed PMD violation - unused inports * Update integration-tests/src/test/groovy/com/okta/sdk/tests/it/CrudTestSupport.groovy Co-authored-by: Brian Demers <bdemers@apache.org> * Update integration-tests/src/test/groovy/com/okta/sdk/tests/it/FactorsIT.groovy Co-authored-by: Brian Demers <bdemers@apache.org> * Update integration-tests/src/test/groovy/com/okta/sdk/tests/it/UsersIT.groovy Co-authored-by: Brian Demers <bdemers@apache.org> * Update integration-tests/src/test/groovy/com/okta/sdk/tests/it/GroupsIT.groovy Co-authored-by: Brian Demers <bdemers@apache.org> * minor improvements (#427) * OKTA-308935: Create Migration Guide for Java Mgmt SDK v2.0.0 (#423) Co-authored-by: Ivan Ezeigbo <56391095+ivanezeigbo-okta@users.noreply.github.com> Co-authored-by: Brian Demers <bdemers@apache.org>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Resolves:
OKTA-289662