Skip to content

Commit

Permalink
fix and rename tests again
Browse files Browse the repository at this point in the history
  • Loading branch information
bshaffer committed Sep 27, 2024
1 parent da55557 commit fa97870
Show file tree
Hide file tree
Showing 9 changed files with 14 additions and 13 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/backwards-compatibility-checks.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
name: Backward Compatibility
name: Backwards Compatibility
on:
pull_request:
types: [opened, synchronize, reopened, edited]
Expand Down Expand Up @@ -121,4 +121,4 @@ jobs:
echo "Add \"MAJOR_VERSION_ALLOWED=component1,component2\" to the PR description to allow "
echo "major version releases for those components"
exit 1
fi
fi
Original file line number Diff line number Diff line change
Expand Up @@ -4,12 +4,12 @@ on:
- main
paths:
- 'Bigtable/**'
- '.github/workflows/emulator-system-tests-bigtable.yaml'
- '.github/workflows/system-tests-bigtable-emulator.yaml'
- '.github/emulator/**'
pull_request:
paths:
- 'Bigtable/**'
- '.github/workflows/emulator-system-tests-bigtable.yaml'
- '.github/workflows/system-tests-bigtable-emulator.yaml'
- '.github/emulator/**'
name: Run Bigtable System Tests With Emulator
permissions:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,12 +5,12 @@ on:
- main
paths:
- 'Datastore/**'
- '.github/workflows/emulator-system-tests-datastore.yaml'
- '.github/workflows/system-tests-datastore-emulator.yaml'
- '.github/emulator/**'
pull_request:
paths:
- 'Datastore/**'
- '.github/workflows/emulator-system-tests-datastore.yaml'
- '.github/workflows/system-tests-datastore-emulator.yaml'
- '.github/emulator/**'
permissions:
contents: read
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,12 +4,12 @@ on:
- main
paths:
- 'Firestore/**'
- '.github/workflows/emulator-system-tests-firestore.yaml'
- '.github/workflows/system-tests-firestore-emulator.yaml'
- '.github/emulator/**'
pull_request:
paths:
- 'Firestore/**'
- '.github/workflows/emulator-system-tests-firestore.yaml'
- '.github/workflows/system-tests-firestore-emulator.yaml'
- '.github/emulator/**'
name: Run Firestore System Tests With Emulator
permissions:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,12 +4,12 @@ on:
- main
paths:
- 'PubSub/**'
- '.github/workflows/emulator-system-tests-pubsub.yaml'
- '.github/workflows/system-tests-pubsub-emulator.yaml'
- '.github/emulator/**'
pull_request:
paths:
- 'PubSub/**'
- '.github/workflows/emulator-system-tests-pubsub.yaml'
- '.github/workflows/system-tests-pubsub-emulator.yaml'
- '.github/emulator/**'
name: Run PubSub System Tests With Emulator
permissions:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,11 +4,11 @@ on:
- main
paths:
- 'Spanner/**'
- '.github/workflows/emulator-system-tests-spanner.yaml'
- '.github/workflows/system-tests-spanner-emulator.yaml'
pull_request:
paths:
- 'Spanner/**'
- '.github/workflows/emulator-system-tests-spanner.yaml'
- '.github/workflows/system-tests-spanner-emulator.yaml'
name: Run Spanner System Tests With Emulator
permissions:
contents: read
Expand Down
File renamed without changes.
3 changes: 2 additions & 1 deletion Redis/tests/System/V1/CloudRedisClientTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -105,9 +105,10 @@ public function testDeleteOperation()
$this->assertTrue($deleteOp->operationSucceeded());

// Ensure delete op succeeded
$instances = self::$client->listInstances(ListInstancesRequest::build(self::$parent));
$this->assertSame(0, count(array_map(
fn ($instance) => $instance->getName() === $instanceName,
iterator_to_array($client->listInstances($listInstancesRequest)->iterateAllElements())
iterator_to_array($instances->iterateAllElements())
)));
}
}

0 comments on commit fa97870

Please sign in to comment.