jenkins-bot has submitted this change and it was merged.
Change subject: Add tests for SQLiteSchemaSqlBuilder so far
......................................................................
Add tests for SQLiteSchemaSqlBuilder so far
Change-Id: Ibc71b857a943a642e22798287a3625edab75571b
---
M src/SQLite/SQLiteSchemaSqlBuilder.php
A tests/phpunit/SQLite/SQLiteSchemaSqlBuilderTest.php
2 files changed, 37 insertions(+), 1 deletion(-)
Approvals:
Jeroen De Dauw: Looks good to me, approved
jenkins-bot: Verified
diff --git a/src/SQLite/SQLiteSchemaSqlBuilder.php
b/src/SQLite/SQLiteSchemaSqlBuilder.php
index f7d8a38..5b04949 100644
--- a/src/SQLite/SQLiteSchemaSqlBuilder.php
+++ b/src/SQLite/SQLiteSchemaSqlBuilder.php
@@ -2,6 +2,7 @@
namespace Wikibase\Database\SQLite;
+use Wikibase\Database\Escaper;
use Wikibase\Database\Schema\Definitions\FieldDefinition;
use Wikibase\Database\Schema\SchemaModificationSqlBuilder;
@@ -41,7 +42,6 @@
* @return string
*/
public function getAddFieldSql( $tableName, FieldDefinition $field ) {
- //TODO add unittests
return "ALTER TABLE {$tableName} ADD COLUMN " .
$this->fieldSqlBuilder->getFieldSQL( $field );
}
diff --git a/tests/phpunit/SQLite/SQLiteSchemaSqlBuilderTest.php
b/tests/phpunit/SQLite/SQLiteSchemaSqlBuilderTest.php
new file mode 100644
index 0000000..7f30163
--- /dev/null
+++ b/tests/phpunit/SQLite/SQLiteSchemaSqlBuilderTest.php
@@ -0,0 +1,36 @@
+<?php
+
+namespace Wikibase\Database\Tests\SQLite;
+
+use Wikibase\Database\Schema\Definitions\FieldDefinition;
+use Wikibase\Database\SQLite\SQLiteSchemaSqlBuilder;
+
+/**
+ * @covers Wikibase\Database\SQLite\SQLiteSchemaSqlBuilder
+ *
+ * @group Wikibase
+ * @group WikibaseDatabase
+ * @group Database
+ *
+ * @licence GNU GPL v2+
+ * @author Adam Shorland
+ */
+class SQLiteSchemaSqlBuilderTest extends \PHPUnit_Framework_TestCase {
+
+ private function newInstance() {
+ $mockEscaper = $this->getMock( 'Wikibase\Database\Escaper' );
+ $mockEscaper->expects( $this->any() )
+ ->method( 'getEscapedValue' )
+ ->will( $this->returnArgument(0) );
+
+ return new SQLiteSchemaSqlBuilder( $mockEscaper );
+ }
+
+ public function testGetAddFieldSql(){
+ $instance = $this->newInstance();
+ $field = new FieldDefinition( 'intField',
FieldDefinition::TYPE_INTEGER, FieldDefinition::NOT_NULL, 42 );
+ $sql = $instance->getAddFieldSql( 'tableName', $field );
+ $this->assertEquals( 'ALTER TABLE tableName ADD COLUMN intField
INT DEFAULT 42 NOT NULL', $sql );
+ }
+
+}
\ No newline at end of file
--
To view, visit https://gerrit.wikimedia.org/r/87049
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ibc71b857a943a642e22798287a3625edab75571b
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/WikibaseDatabase
Gerrit-Branch: master
Gerrit-Owner: Addshore <[email protected]>
Gerrit-Reviewer: Jeroen De Dauw <[email protected]>
Gerrit-Reviewer: jenkins-bot
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
