Skip to content

Commit

Permalink
use cloud/core from branch for emulator tests
Browse files Browse the repository at this point in the history
  • Loading branch information
bshaffer committed May 5, 2022
1 parent b3144a0 commit a633765
Show file tree
Hide file tree
Showing 11 changed files with 17 additions and 16 deletions.
4 changes: 3 additions & 1 deletion .github/workflows/bigtable-emulator-system-tests.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,9 @@ jobs:

- name: Install dependencies
run: |
composer update --prefer-dist --no-interaction --no-suggest -d Bigtable/
composer config minimum-stability dev -d Bigtable/
composer config repositories.local path "../Core" -d Bigtable
composer require "google/cloud-core:*" -d Bigtable/
- name: Run system tests
run: |
Expand Down
4 changes: 3 additions & 1 deletion .github/workflows/firestore-emulator-system-tests.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,9 @@ jobs:

- name: Install dependencies
run: |
composer update --prefer-dist --no-interaction --no-suggest -d Firestore/
composer config minimum-stability dev -d Firestore/
composer config repositories.local path "../Core" -d Firestore
composer require "google/cloud-core:*" -d Firestore/
- name: Run system tests
run: |
Expand Down
2 changes: 1 addition & 1 deletion Datastore/tests/System/DatastoreTestCase.php
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,7 @@ public static function tearDownFixtures()

public function clientProvider()
{
self::setupBeforeClass();
self::set_up_before_class();

return [
'restClient' => [self::$restClient],
Expand Down
7 changes: 2 additions & 5 deletions Firestore/tests/System/QueryTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -18,16 +18,13 @@
namespace Google\Cloud\Firestore\Tests\System;

use Google\Cloud\Firestore\FieldPath;
use Yoast\PHPUnitPolyfills\Polyfills\AssertStringContains;

/**
* @group firestore
* @group firestore-query
*/
class QueryTest extends FirestoreTestCase
{
use AssertStringContains;

private $query;

public function set_up()
Expand Down Expand Up @@ -127,8 +124,8 @@ public function testWhereInArray()
$doc_ids = array_map(function ($doc) {
return $doc->id();
}, $docs);
$this->assertStringContainsString($doc1->id(), $doc_ids);
$this->assertStringContainsString($doc2->id(), $doc_ids);
$this->assertContains($doc1->id(), $doc_ids);
$this->assertContains($doc2->id(), $doc_ids);
}

public function testSnapshotCursors()
Expand Down
2 changes: 1 addition & 1 deletion Spanner/tests/System/BatchTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ class BatchTest extends SpannerTestCase

public static function set_up_before_class()
{
parent::setupBeforeClass();
parent::set_up_before_class();

self::$tableName = uniqid(self::TESTING_PREFIX);

Expand Down
2 changes: 1 addition & 1 deletion Spanner/tests/System/LargeReadTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ class LargeReadTest extends SpannerTestCase

public static function set_up_before_class()
{
parent::setupBeforeClass();
parent::set_up_before_class();

self::$tableName = uniqid(self::TESTING_PREFIX);

Expand Down
2 changes: 1 addition & 1 deletion Spanner/tests/System/PgQueryTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ class PgQueryTest extends SpannerPgTestCase

public static $timestampVal;

public static function setUpBeforeClass()
public static function set_up_before_class()
{
parent::set_up_before_class();

Expand Down
4 changes: 2 additions & 2 deletions Spanner/tests/System/PgReadTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -37,9 +37,9 @@ class PgReadTest extends SpannerPgTestCase
private static $indexes = [];
private static $dataset;

public static function setupBeforeClass()
public static function set_up_before_class()
{
parent::setupBeforeClass();
parent::set_up_before_class();

self::$readTableName = "read_table";
self::$rangeTableName = "range_table";
Expand Down
2 changes: 1 addition & 1 deletion Spanner/tests/System/PgTransactionTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ class PgTransactionTest extends SpannerPgTestCase
private static $tableName;
private static $table2;

public static function setUpBeforeClass()
public static function set_up_before_class()
{
parent::set_up_before_class();

Expand Down
2 changes: 1 addition & 1 deletion Spanner/tests/System/PgWriteTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ class PgWriteTest extends SpannerPgTestCase
const TABLE_NAME = 'Writes';
const COMMIT_TIMESTAMP_TABLE_NAME = 'CommitTimestamps';

public static function setupBeforeClass()
public static function set_up_before_class()
{
parent::set_up_before_class();

Expand Down
2 changes: 1 addition & 1 deletion Spanner/tests/System/SpannerPgTestCase.php
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ class SpannerPgTestCase extends SystemTestCase

private static $hasSetUp = false;

public static function setUpBeforeClass()
public static function set_up_before_class()
{
if (self::$hasSetUp) {
return;
Expand Down

0 comments on commit a633765

Please sign in to comment.