Skip to content

Commit 9625a44

Browse files
committed
Fix NPE in UnifiedHostGroupsIT by calling hasNext() before next()
The Dedupe iterator wrapping the Groups list only populates its current element inside hasNext(). Calling next() without hasNext() returns null. Co-authored-by: Isaac
1 parent 2622f1b commit 9625a44

1 file changed

Lines changed: 4 additions & 1 deletion

File tree

databricks-sdk-java/src/test/java/com/databricks/sdk/integration/UnifiedHostGroupsIT.java

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@
1010
import com.databricks.sdk.integration.framework.EnvTest;
1111
import com.databricks.sdk.service.iam.Group;
1212
import com.databricks.sdk.service.iam.ListGroupsRequest;
13+
import java.util.Iterator;
1314
import org.junit.jupiter.api.Test;
1415
import org.junit.jupiter.api.condition.DisabledIfEnvironmentVariable;
1516
import org.junit.jupiter.api.condition.EnabledIfEnvironmentVariable;
@@ -36,7 +37,9 @@ void listWorkspaceGroupsViaUnifiedHost(
3637
WorkspaceClient ws = new WorkspaceClient(config);
3738

3839
Iterable<Group> groups = ws.groups().list(new ListGroupsRequest().setAttributes("displayName"));
39-
Group first = groups.iterator().next();
40+
Iterator<Group> it = groups.iterator();
41+
assertTrue(it.hasNext(), "Expected at least one group");
42+
Group first = it.next();
4043
assertNotNull(first.getDisplayName());
4144
}
4245
}

0 commit comments

Comments
 (0)