Skip to content

Commit e2c1116

Browse files
committed
test(oidc): add eventually to reduce flakiness
1 parent aee96c3 commit e2c1116

1 file changed

Lines changed: 1 addition & 6 deletions

File tree

src/it/java/io/weaviate/integration/OidcSupportITest.java

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,6 @@
2929
* Additionally, {@code WCS_DUMMY_CI_PW} and {@code OKTA_CLIENT_SECRET}
3030
* environment variables must be set.
3131
*/
32-
@Ignore
3332
public class OidcSupportITest extends ConcurrentTest {
3433
private static final String WCS_DUMMY_CI_USERNAME = "oidc-test-user@weaviate.io";
3534
private static final String WCS_DUMMY_CI_PW = System.getenv("WCS_DUMMY_CI_PW");
@@ -56,7 +55,6 @@ public class OidcSupportITest extends ConcurrentTest {
5655
* and authenticate with it.
5756
*/
5857
@Test
59-
@Ignore("flaky")
6058
public void test_bearerToken() throws Exception {
6159
Assume.assumeTrue("WCS_DUMMY_CI_PW is not set", WCS_DUMMY_CI_PW != null && !WCS_DUMMY_CI_PW.isBlank());
6260
Assume.assumeTrue("no internet connection", hasInternetConnection());
@@ -69,10 +67,7 @@ public void test_bearerToken() throws Exception {
6967
var auth = SpyTokenProvider.spyOn(Authentication.bearerToken(t.accessToken(), t.refreshToken(), 0));
7068
pingWeaviate(wcsContainer, auth);
7169

72-
Token newT = auth.getToken();
73-
Assertions.assertThat(newT.accessToken())
74-
.as("expect access_token was refreshed")
75-
.isNotEqualTo(t.accessToken());
70+
eventually(() -> auth.getToken() != t, 100, 5, "expect access_token was refreshed");
7671

7772
// Check that the new token authenticates requests.
7873
pingWeaviate(wcsContainer, auth);

0 commit comments

Comments
 (0)