tonyair/silverstripe-postgresql
1
0
Fork 0
mirror of https://github.com/silverstripe/silverstripe-postgresql synced 2024-06-25 22:19:41 +02:00
Code Issues Projects Releases Wiki Activity

Fix liniting errors

Browse Source
This commit is contained in:
Maxime Rainville 2019-09-26 09:34:30 +12:00
parent 74a20c0cb0
commit 0d54ec6bda
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
tests/PostgreSQLSchemaManagerTest.php
View File

@ -27,7 +27,8 @@ class PostgreSQLSchemaManagerTest extends SapphireTest
try { try {
DB::query('INSERT INTO "ClassNamesUpgrade" ("ClassName") VALUES (\'App\MySite\FooBar\')'); DB::query('INSERT INTO "ClassNamesUpgrade" ("ClassName") VALUES (\'App\MySite\FooBar\')');
$this->assertFalse(true, 'SS3 Constaint should have blocked the previous insert.'); $this->assertFalse(true, 'SS3 Constaint should have blocked the previous insert.');
} catch (DatabaseException $ex) { } } catch (DatabaseException $ex) {
}
$dbSchema->schemaUpdate(function () use ($dbSchema) { $dbSchema->schemaUpdate(function () use ($dbSchema) {
$dbSchema->requireTable( $dbSchema->requireTable(
@ -48,7 +49,6 @@ class PostgreSQLSchemaManagerTest extends SapphireTest
DB::query('DROP TABLE IF EXISTS "ClassNamesUpgrade"'); DB::query('DROP TABLE IF EXISTS "ClassNamesUpgrade"');
DB::query('DROP SEQUENCE IF EXISTS "ClassNamesUpgrade_ID_seq"'); DB::query('DROP SEQUENCE IF EXISTS "ClassNamesUpgrade_ID_seq"');
} }
} }
private function createSS3Table() private function createSS3Table()
@ -91,16 +91,15 @@ SQL
$this->assertTableCount(1, 'ClassNamesUpgrade_Versioned'); $this->assertTableCount(1, 'ClassNamesUpgrade_Versioned');
$this->assertConstraintCount(0, 'ClassNamesUpgrade_versioned_ClassName_check'); $this->assertConstraintCount(0, 'ClassNamesUpgrade_versioned_ClassName_check');
$this->assertConstraintCount(1, 'ClassNamesUpgrade_Versioned_ClassName_check'); $this->assertConstraintCount(1, 'ClassNamesUpgrade_Versioned_ClassName_check');
} finally { } finally {
DB::query('DROP TABLE IF EXISTS "ClassNamesUpgrade_Versioned"'); DB::query('DROP TABLE IF EXISTS "ClassNamesUpgrade_Versioned"');
DB::query('DROP TABLE IF EXISTS "ClassNamesUpgrade_versioned"'); DB::query('DROP TABLE IF EXISTS "ClassNamesUpgrade_versioned"');
DB::query('DROP SEQUENCE IF EXISTS "ClassNamesUpgrade_versioned_ID_seq"'); DB::query('DROP SEQUENCE IF EXISTS "ClassNamesUpgrade_versioned_ID_seq"');
} }
} }
private function assertConstraintCount($expected, $constraintName) { private function assertConstraintCount($expected, $constraintName)
{
$count = DB::prepared_query( $count = DB::prepared_query(
'SELECT count(*) FROM pg_catalog.pg_constraint WHERE conname like ?', 'SELECT count(*) FROM pg_catalog.pg_constraint WHERE conname like ?',
[$constraintName] [$constraintName]
@ -109,7 +108,8 @@ SQL
$this->assertEquals($expected, $count); $this->assertEquals($expected, $count);
} }
private function assertTableCount($expected, $tableName) { private function assertTableCount($expected, $tableName)
{
$count = DB::prepared_query( $count = DB::prepared_query(
'SELECT count(*) FROM pg_catalog.pg_tables WHERE "tablename" like ?', 'SELECT count(*) FROM pg_catalog.pg_tables WHERE "tablename" like ?',
[$tableName] [$tableName]