Skip to content

Conversation

@lilgreenbird
Copy link
Contributor

@lilgreenbird lilgreenbird commented Feb 1, 2023

  • updated fedauth tests to get new access token once per test suite instead of every test
  • added checks and retry for throttling
  • shortened access token expiry for tests that tests token expiry
  • some cleanup

@lilgreenbird lilgreenbird changed the title Updated fedauth tests Updated fedauth tests to run on diff test server Feb 8, 2023
@lilgreenbird lilgreenbird marked this pull request as ready for review February 8, 2023 19:36
@lilgreenbird lilgreenbird added this to the 12.3.0 milestone Feb 8, 2023
@lilgreenbird lilgreenbird merged commit 2429110 into main Feb 9, 2023
Jeffery-Wasty added a commit that referenced this pull request Jan 29, 2025
Jeffery-Wasty added a commit that referenced this pull request Mar 19, 2025
…date (#2602)

* Update build.gradle

* Update pom.xml

* Update pom.xml

* Update pom.xml

* Update pom.xml

* Update pom.xml

* Update pom.xml

* Update pom.xml

* ManagedIdentity I

Signed-off-by: Jeff Wasty <[email protected]>

* Updates for running tests with managed identity (#2416)

Signed-off-by: Jeff Wasty <[email protected]>

# Conflicts:
#	src/test/java/com/microsoft/sqlserver/jdbc/SQLServerConnectionTest.java
#	src/test/java/com/microsoft/sqlserver/jdbc/TestResource.java
#	src/test/java/com/microsoft/sqlserver/jdbc/TestUtils.java
#	src/test/java/com/microsoft/sqlserver/jdbc/connection/TimeoutTest.java
#	src/test/java/com/microsoft/sqlserver/jdbc/databasemetadata/DatabaseMetaDataTest.java
#	src/test/java/com/microsoft/sqlserver/jdbc/unit/statement/BatchExecutionTest.java

* Delete mistakenly added files

* Delete mistakenly added files

* Revert "Delete mistakenly added files"

This reverts commit 19abc22.

* Revert "Delete mistakenly added files"

This reverts commit 04f8c82.

* Revert "Updates for running tests with managed identity (#2416)"

This reverts commit b727a96.

* Revert "ManagedIdentity I"

This reverts commit 6d5c519.

* Delete garbage

* Fixed Managed Identity tests (#1935)

Signed-off-by: Jeff Wasty <[email protected]>

# Conflicts:
#	src/test/java/com/microsoft/sqlserver/jdbc/AlwaysEncrypted/MSITest.java

* Managed Identity dev experience improvements #1936

Signed-off-by: Jeff Wasty <[email protected]>

* Updates for running tests with managed identity #2416

Signed-off-by: Jeff Wasty <[email protected]>

* cleanup

Signed-off-by: Jeff Wasty <[email protected]>

* Update tests for Kerberos

Signed-off-by: Jeff Wasty <[email protected]>

* Remove unused test

* tsc=true for ae

Signed-off-by: Jeff Wasty <[email protected]>

* Add tags for testDefaultRetry

* Remove unstable test

Signed-off-by: Jeff Wasty <[email protected]>

* Add throttling error message to tests (#2044)

Signed-off-by: Jeff Wasty <[email protected]>

* Updated fedauth tests to run on diff test server (#2062)

Signed-off-by: Jeff Wasty <[email protected]>

* Update fedauth tests to use cached tokens and retry if throttled (#2077)

Signed-off-by: Jeff Wasty <[email protected]>

* Add back missing import

Signed-off-by: Jeff Wasty <[email protected]>

* Resolve static issue with IOBuffer

Signed-off-by: Jeff Wasty <[email protected]>

* Add back a missing import

* Fixed TestUtils

* Remove mistakenly added code

* Cleanup

* Cleanup testutils

* More cleanup

* Add more time for 'testDefaultRetry' &
Fix tests for testing encrypt options (#2215)

* Update setAEConnectionString

* Updated fedauth error tests (#2538)

Signed-off-by: Jeff Wasty <[email protected]>

# Conflicts:
#	src/test/java/com/microsoft/sqlserver/jdbc/fedauth/ErrorMessageTest.java
#	src/test/java/com/microsoft/sqlserver/jdbc/fedauth/FedauthCommon.java

* Update fedAuth tests

* Revert "Updated fedauth error tests (#2538)"

This reverts commit dfb4b77.

* Revert "Update fedAuth tests"

This reverts commit da5c58b.

* Added ActiveDirectoryServicePrincipalCertificate for tests

* Add 'requireSecret' exclude tag for tests which require adding a secret to app registration

* Version update

* Add connection retry changes from 2513 to ensure existing timing tests pass

* Adjust timings on tests as the fix requires code changes we should not incorporate for this release

* Changelog

* Update dependency versions

---------

Signed-off-by: Jeff Wasty <[email protected]>
Co-authored-by: lilgreenbird <[email protected]>
Co-authored-by: Terry Chow <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants