commit 05945e45c07c6a786c985489729b70d9540245e5
author Kevin Cheng <kevcheng@google.com> Fri Nov 15 16:15:13 2019 -0800
committer Kevin Cheng <kevcheng@google.com> Fri Nov 15 23:21:40 2019 -0800
tree 5a8f39b73fa819bf382dcf0a16dd136e11c75e4a
parent f7f5c1e40b8b37063708ce23f648efa7e9d4b693

Update aggregatedList in ListInstances to use pageToken

aggregatedList returns max 500 instances, but in the infra, we can have much
more than that. So use the pageToken to make sure we get all of the instances.

Bug: 144589609
Test: acloud delete ...
atest acloud_test
Change-Id: Ide06ab58378c222510522f7480334aab229fa4f8

internal/lib/gcompute_client_test.py

diff --git a/internal/lib/gcompute_client_test.py b/internal/lib/gcompute_client_test.py
index d4f225d..690588a 100644
--- a/internal/lib/gcompute_client_test.py
+++ b/internal/lib/gcompute_client_test.py
@@ -497,7 +497,8 @@
         calls = [
             mock.call(
                 project=PROJECT,
-                filter=None),
+                filter=None,
+                pageToken=None),
         ]
         resource_mock.aggregatedList.assert_has_calls(calls)
         self.assertEqual(instances, [instance_1, instance_2])