Skip to content

Commit 7745f8f

Browse files
committed
test(oidc): ignore flaky test and enable test suite
1 parent 8fe112b commit 7745f8f

1 file changed

Lines changed: 5 additions & 2 deletions

File tree

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

Lines changed: 5 additions & 2 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,6 +55,7 @@ public class OidcSupportITest extends ConcurrentTest {
5655
* and authenticate with it.
5756
*/
5857
@Test
58+
@Ignore("flaky")
5959
public void test_bearerToken() throws Exception {
6060
Assume.assumeTrue("WCS_DUMMY_CI_PW is not set", WCS_DUMMY_CI_PW != null && !WCS_DUMMY_CI_PW.isBlank());
6161
Assume.assumeTrue("no internet connection", hasInternetConnection());
@@ -68,7 +68,10 @@ public void test_bearerToken() throws Exception {
6868
var auth = SpyTokenProvider.spyOn(Authentication.bearerToken(t.accessToken(), t.refreshToken(), 0));
6969
pingWeaviate(wcsContainer, auth);
7070

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

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

0 commit comments

Comments
 (0)