From 3696ef5b965e5d3e44197479eaf310e26288151c Mon Sep 17 00:00:00 2001 From: Joas Schilling Date: Wed, 11 Nov 2020 14:34:24 +0100 Subject: Don't allow Notnull for boolean columns Signed-off-by: Joas Schilling --- tests/lib/DB/MigrationsTest.php | 41 +++++++++++++++++++++++++++++++++++++++++ 1 file changed, 41 insertions(+) (limited to 'tests') diff --git a/tests/lib/DB/MigrationsTest.php b/tests/lib/DB/MigrationsTest.php index 9b6dec006d6..c8b1af3e08e 100644 --- a/tests/lib/DB/MigrationsTest.php +++ b/tests/lib/DB/MigrationsTest.php @@ -18,6 +18,7 @@ use Doctrine\DBAL\Schema\Schema; use Doctrine\DBAL\Schema\SchemaException; use Doctrine\DBAL\Schema\Sequence; use Doctrine\DBAL\Schema\Table; +use Doctrine\DBAL\Types\Type; use OC\DB\Connection; use OC\DB\MigrationService; use OC\DB\SchemaWrapper; @@ -632,4 +633,44 @@ class MigrationsTest extends \Test\TestCase { self::invokePrivate($this->migrationService, 'ensureOracleIdentifierLengthLimit', [$sourceSchema, $schema, 3]); } + + + public function testEnsureOracleIdentifierLengthLimitBooleanNotNull() { + $this->expectException(\InvalidArgumentException::class); + + $column = $this->createMock(Column::class); + $column->expects($this->any()) + ->method('getName') + ->willReturn('aaaa'); + $column->expects($this->any()) + ->method('getType') + ->willReturn(Type::getType('boolean')); + $column->expects($this->any()) + ->method('getNotnull') + ->willReturn(true); + + $table = $this->createMock(Table::class); + $table->expects($this->any()) + ->method('getName') + ->willReturn(\str_repeat('a', 30)); + + $table->expects($this->once()) + ->method('getColumns') + ->willReturn([$column]); + + $schema = $this->createMock(Schema::class); + $schema->expects($this->once()) + ->method('getTables') + ->willReturn([$table]); + + $sourceSchema = $this->createMock(Schema::class); + $sourceSchema->expects($this->any()) + ->method('getTable') + ->willThrowException(new SchemaException()); + $sourceSchema->expects($this->any()) + ->method('hasSequence') + ->willReturn(false); + + self::invokePrivate($this->migrationService, 'ensureOracleIdentifierLengthLimit', [$sourceSchema, $schema, 3]); + } } -- cgit v1.2.3 From f9d4fa2d38d6693d87b62ad448f38441b1876e9f Mon Sep 17 00:00:00 2001 From: Joas Schilling Date: Wed, 11 Nov 2020 14:40:26 +0100 Subject: Rename the method to match what it does Signed-off-by: Joas Schilling --- lib/private/DB/MigrationService.php | 6 ++--- tests/lib/DB/MigrationsTest.php | 44 ++++++++++++++++++------------------- 2 files changed, 25 insertions(+), 25 deletions(-) (limited to 'tests') diff --git a/lib/private/DB/MigrationService.php b/lib/private/DB/MigrationService.php index 4957706bb1d..ee64b45be64 100644 --- a/lib/private/DB/MigrationService.php +++ b/lib/private/DB/MigrationService.php @@ -459,7 +459,7 @@ class MigrationService { $targetSchema = $toSchema->getWrappedSchema(); if ($this->checkOracle) { $beforeSchema = $this->connection->createSchema(); - $this->ensureOracleIdentifierLengthLimit($beforeSchema, $targetSchema, strlen($this->connection->getPrefix())); + $this->ensureOracleConstraints($beforeSchema, $targetSchema, strlen($this->connection->getPrefix())); } $this->connection->migrateToSchema($targetSchema); $toSchema->performDropTableCalls(); @@ -536,7 +536,7 @@ class MigrationService { $targetSchema = $toSchema->getWrappedSchema(); if ($this->checkOracle) { $sourceSchema = $this->connection->createSchema(); - $this->ensureOracleIdentifierLengthLimit($sourceSchema, $targetSchema, strlen($this->connection->getPrefix())); + $this->ensureOracleConstraints($sourceSchema, $targetSchema, strlen($this->connection->getPrefix())); } $this->connection->migrateToSchema($targetSchema); $toSchema->performDropTableCalls(); @@ -569,7 +569,7 @@ class MigrationService { * @param int $prefixLength * @throws \Doctrine\DBAL\Exception */ - public function ensureOracleIdentifierLengthLimit(Schema $sourceSchema, Schema $targetSchema, int $prefixLength) { + public function ensureOracleConstraints(Schema $sourceSchema, Schema $targetSchema, int $prefixLength) { $sequences = $targetSchema->getSequences(); foreach ($targetSchema->getTables() as $table) { diff --git a/tests/lib/DB/MigrationsTest.php b/tests/lib/DB/MigrationsTest.php index c8b1af3e08e..507f3d0e228 100644 --- a/tests/lib/DB/MigrationsTest.php +++ b/tests/lib/DB/MigrationsTest.php @@ -220,7 +220,7 @@ class MigrationsTest extends \Test\TestCase { $this->migrationService->migrate(); } - public function testEnsureOracleIdentifierLengthLimitValid() { + public function testEnsureOracleConstraintsValid() { $column = $this->createMock(Column::class); $column->expects($this->once()) ->method('getName') @@ -275,10 +275,10 @@ class MigrationsTest extends \Test\TestCase { ->method('hasSequence') ->willReturn(false); - self::invokePrivate($this->migrationService, 'ensureOracleIdentifierLengthLimit', [$sourceSchema, $schema, 3]); + self::invokePrivate($this->migrationService, 'ensureOracleConstraints', [$sourceSchema, $schema, 3]); } - public function testEnsureOracleIdentifierLengthLimitValidWithPrimaryKey() { + public function testEnsureOracleConstraintsValidWithPrimaryKey() { $index = $this->createMock(Index::class); $index->expects($this->any()) ->method('getName') @@ -318,10 +318,10 @@ class MigrationsTest extends \Test\TestCase { ->method('hasSequence') ->willReturn(false); - self::invokePrivate($this->migrationService, 'ensureOracleIdentifierLengthLimit', [$sourceSchema, $schema, 3]); + self::invokePrivate($this->migrationService, 'ensureOracleConstraints', [$sourceSchema, $schema, 3]); } - public function testEnsureOracleIdentifierLengthLimitValidWithPrimaryKeyDefault() { + public function testEnsureOracleConstraintsValidWithPrimaryKeyDefault() { $defaultName = 'PRIMARY'; if ($this->db->getDatabasePlatform() instanceof PostgreSqlPlatform) { $defaultName = \str_repeat('a', 26) . '_' . \str_repeat('b', 30) . '_seq'; @@ -371,11 +371,11 @@ class MigrationsTest extends \Test\TestCase { ->method('hasSequence') ->willReturn(false); - self::invokePrivate($this->migrationService, 'ensureOracleIdentifierLengthLimit', [$sourceSchema, $schema, 3]); + self::invokePrivate($this->migrationService, 'ensureOracleConstraints', [$sourceSchema, $schema, 3]); } - public function testEnsureOracleIdentifierLengthLimitTooLongTableName() { + public function testEnsureOracleConstraintsTooLongTableName() { $this->expectException(\InvalidArgumentException::class); $table = $this->createMock(Table::class); @@ -396,11 +396,11 @@ class MigrationsTest extends \Test\TestCase { ->method('hasSequence') ->willReturn(false); - self::invokePrivate($this->migrationService, 'ensureOracleIdentifierLengthLimit', [$sourceSchema, $schema, 3]); + self::invokePrivate($this->migrationService, 'ensureOracleConstraints', [$sourceSchema, $schema, 3]); } - public function testEnsureOracleIdentifierLengthLimitTooLongPrimaryWithDefault() { + public function testEnsureOracleConstraintsTooLongPrimaryWithDefault() { $this->expectException(\InvalidArgumentException::class); $defaultName = 'PRIMARY'; @@ -449,11 +449,11 @@ class MigrationsTest extends \Test\TestCase { ->method('hasSequence') ->willReturn(false); - self::invokePrivate($this->migrationService, 'ensureOracleIdentifierLengthLimit', [$sourceSchema, $schema, 3]); + self::invokePrivate($this->migrationService, 'ensureOracleConstraints', [$sourceSchema, $schema, 3]); } - public function testEnsureOracleIdentifierLengthLimitTooLongPrimaryWithName() { + public function testEnsureOracleConstraintsTooLongPrimaryWithName() { $this->expectException(\InvalidArgumentException::class); $index = $this->createMock(Index::class); @@ -492,11 +492,11 @@ class MigrationsTest extends \Test\TestCase { ->method('hasSequence') ->willReturn(false); - self::invokePrivate($this->migrationService, 'ensureOracleIdentifierLengthLimit', [$sourceSchema, $schema, 3]); + self::invokePrivate($this->migrationService, 'ensureOracleConstraints', [$sourceSchema, $schema, 3]); } - public function testEnsureOracleIdentifierLengthLimitTooLongColumnName() { + public function testEnsureOracleConstraintsTooLongColumnName() { $this->expectException(\InvalidArgumentException::class); $column = $this->createMock(Column::class); @@ -526,11 +526,11 @@ class MigrationsTest extends \Test\TestCase { ->method('hasSequence') ->willReturn(false); - self::invokePrivate($this->migrationService, 'ensureOracleIdentifierLengthLimit', [$sourceSchema, $schema, 3]); + self::invokePrivate($this->migrationService, 'ensureOracleConstraints', [$sourceSchema, $schema, 3]); } - public function testEnsureOracleIdentifierLengthLimitTooLongIndexName() { + public function testEnsureOracleConstraintsTooLongIndexName() { $this->expectException(\InvalidArgumentException::class); $index = $this->createMock(Index::class); @@ -563,11 +563,11 @@ class MigrationsTest extends \Test\TestCase { ->method('hasSequence') ->willReturn(false); - self::invokePrivate($this->migrationService, 'ensureOracleIdentifierLengthLimit', [$sourceSchema, $schema, 3]); + self::invokePrivate($this->migrationService, 'ensureOracleConstraints', [$sourceSchema, $schema, 3]); } - public function testEnsureOracleIdentifierLengthLimitTooLongForeignKeyName() { + public function testEnsureOracleConstraintsTooLongForeignKeyName() { $this->expectException(\InvalidArgumentException::class); $foreignKey = $this->createMock(ForeignKeyConstraint::class); @@ -603,11 +603,11 @@ class MigrationsTest extends \Test\TestCase { ->method('hasSequence') ->willReturn(false); - self::invokePrivate($this->migrationService, 'ensureOracleIdentifierLengthLimit', [$sourceSchema, $schema, 3]); + self::invokePrivate($this->migrationService, 'ensureOracleConstraints', [$sourceSchema, $schema, 3]); } - public function testEnsureOracleIdentifierLengthLimitTooLongSequenceName() { + public function testEnsureOracleConstraintsTooLongSequenceName() { $this->expectException(\InvalidArgumentException::class); $sequence = $this->createMock(Sequence::class); @@ -631,11 +631,11 @@ class MigrationsTest extends \Test\TestCase { ->method('hasSequence') ->willReturn(false); - self::invokePrivate($this->migrationService, 'ensureOracleIdentifierLengthLimit', [$sourceSchema, $schema, 3]); + self::invokePrivate($this->migrationService, 'ensureOracleConstraints', [$sourceSchema, $schema, 3]); } - public function testEnsureOracleIdentifierLengthLimitBooleanNotNull() { + public function testEnsureOracleConstraintsBooleanNotNull() { $this->expectException(\InvalidArgumentException::class); $column = $this->createMock(Column::class); @@ -671,6 +671,6 @@ class MigrationsTest extends \Test\TestCase { ->method('hasSequence') ->willReturn(false); - self::invokePrivate($this->migrationService, 'ensureOracleIdentifierLengthLimit', [$sourceSchema, $schema, 3]); + self::invokePrivate($this->migrationService, 'ensureOracleConstraints', [$sourceSchema, $schema, 3]); } } -- cgit v1.2.3