From e177462c901cd29c3e423640a6f538a930c29536 Mon Sep 17 00:00:00 2001
From: Klaus Weidenbach <Klaus.Weidenbach@gmx.net>
Date: Sat, 9 Dec 2017 00:31:09 +0100
Subject: :white_check_mark: Unit Test for \DBA factory.

---
 tests/unit/includes/dba/DBATest.php | 67 +++++++++++++++++++++++++++++++++++++
 1 file changed, 67 insertions(+)
 create mode 100644 tests/unit/includes/dba/DBATest.php

(limited to 'tests/unit')

diff --git a/tests/unit/includes/dba/DBATest.php b/tests/unit/includes/dba/DBATest.php
new file mode 100644
index 000000000..900d13083
--- /dev/null
+++ b/tests/unit/includes/dba/DBATest.php
@@ -0,0 +1,67 @@
+<?php
+/*
+ * Copyright (c) 2017 Hubzilla
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a copy
+ * of this software and associated documentation files (the "Software"), to deal
+ * in the Software without restriction, including without limitation the rights
+ * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+ * copies of the Software, and to permit persons to whom the Software is
+ * furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice shall be included in
+ * all copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+ * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+ * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+ * SOFTWARE.
+ */
+
+namespace Zotlabs\Tests\Unit\includes;
+
+use Zotlabs\Tests\Unit\UnitTestCase;
+
+// required because of process isolation and no autoloading
+require_once 'include/dba/dba_driver.php';
+
+/**
+ * @brief Unit Test case for include/dba/DBA.php file.
+ *
+ * This test needs process isolation because of static \DBA.
+ * @runTestsInSeparateProcesses
+ */
+class DBATest extends UnitTestCase {
+
+	public function testDbaFactoryMysql() {
+		$this->assertNull(\DBA::$dba);
+
+		$ret = \DBA::dba_factory('server', 'port', 'user', 'pass', 'db', '0');
+		$this->assertInstanceOf('dba_pdo', $ret);
+		$this->assertFalse($ret->connected);
+
+		$this->assertSame('mysql', \DBA::$scheme);
+		$this->assertSame('schema_mysql.sql', \DBA::$install_script);
+		$this->assertSame('0001-01-01 00:00:00', \DBA::$null_date);
+		$this->assertSame('UTC_TIMESTAMP()', \DBA::$utc_now);
+		$this->assertSame('`', \DBA::$tquot);
+	}
+
+	public function testDbaFactoryPostgresql() {
+		$this->assertNull(\DBA::$dba);
+
+		$ret = \DBA::dba_factory('server', 'port', 'user', 'pass', 'db', '1');
+		$this->assertInstanceOf('dba_pdo', $ret);
+		$this->assertFalse($ret->connected);
+
+		$this->assertSame('pgsql', \DBA::$scheme);
+		$this->assertSame('schema_postgres.sql', \DBA::$install_script);
+		$this->assertSame('0001-01-01 00:00:00', \DBA::$null_date);
+		$this->assertSame("now() at time zone 'UTC'", \DBA::$utc_now);
+		$this->assertSame('"', \DBA::$tquot);
+	}
+
+}
-- 
cgit v1.2.3


From fb111e6d958fdf56352db66e144134241abbacf8 Mon Sep 17 00:00:00 2001
From: Klaus Weidenbach <Klaus.Weidenbach@gmx.net>
Date: Sun, 10 Dec 2017 01:29:49 +0100
Subject: :white_check_mark: Unit Test for dba_pdo driver class.

---
 tests/unit/DatabaseTestCase.php            |  68 ++++++++++++
 tests/unit/includes/dba/_files/account.yml |   9 ++
 tests/unit/includes/dba/dba_pdoTest.php    | 167 +++++++++++++++++++++++++++++
 3 files changed, 244 insertions(+)
 create mode 100644 tests/unit/DatabaseTestCase.php
 create mode 100644 tests/unit/includes/dba/_files/account.yml
 create mode 100644 tests/unit/includes/dba/dba_pdoTest.php

(limited to 'tests/unit')

diff --git a/tests/unit/DatabaseTestCase.php b/tests/unit/DatabaseTestCase.php
new file mode 100644
index 000000000..18c1cfb17
--- /dev/null
+++ b/tests/unit/DatabaseTestCase.php
@@ -0,0 +1,68 @@
+<?php
+/* Copyright (c) 2017 Hubzilla
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a copy
+ * of this software and associated documentation files (the "Software"), to deal
+ * in the Software without restriction, including without limitation the rights
+ * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+ * copies of the Software, and to permit persons to whom the Software is
+ * furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice shall be included in
+ * all copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+ * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+ * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+ * SOFTWARE.
+ */
+
+namespace Zotlabs\Tests\Unit;
+
+use PHPUnit\DbUnit\TestCaseTrait;
+use PHPUnit\Framework\TestCase;
+
+/**
+ * @brief Base class for our Database Unit Tests.
+ *
+ * @warning Never run these tests against a production database, because all
+ * tables will get truncated and there is no way to recover without a backup.
+ *
+ * @author Klaus Weidenbach
+ */
+abstract class DatabaseTestCase extends TestCase {
+
+	use TestCaseTrait;
+
+	/**
+	 * Only instantiate PDO once for test clean-up/fixture load.
+	 *
+	 * @var \PDO
+	 */
+	static private $pdo = null;
+
+	/**
+	 * Only instantiate \PHPUnit\DbUnit\Database\Connection once per test.
+	 *
+	 * @var \PHPUnit\DbUnit\Database\Connection
+	 */
+	private $conn = null;
+
+
+	final public function getConnection() {
+		if ($this->conn === null) {
+			if (self::$pdo === null) {
+				$dsn = \getenv('hz_db_scheme') . ':host=' . \getenv('hz_db_server')
+					. ';port=' . \getenv('hz_db_port') . ';dbname=' . \getenv('hz_db_database');
+
+				self::$pdo = new \PDO($dsn, \getenv('hz_db_user'), \getenv('hz_db_pass'));
+			}
+			$this->conn = $this->createDefaultDBConnection(self::$pdo, \getenv('hz_db_database'));
+		}
+
+		return $this->conn;
+	}
+}
diff --git a/tests/unit/includes/dba/_files/account.yml b/tests/unit/includes/dba/_files/account.yml
new file mode 100644
index 000000000..344bdb799
--- /dev/null
+++ b/tests/unit/includes/dba/_files/account.yml
@@ -0,0 +1,9 @@
+account:
+  -
+    account_id: 42
+    account_email: "hubzilla@example.com"
+    account_language: "no"
+  -
+    account_id: 43
+    account_email: "hubzilla@example.org"
+    account_language: "de"
diff --git a/tests/unit/includes/dba/dba_pdoTest.php b/tests/unit/includes/dba/dba_pdoTest.php
new file mode 100644
index 000000000..ce6e1ffd6
--- /dev/null
+++ b/tests/unit/includes/dba/dba_pdoTest.php
@@ -0,0 +1,167 @@
+<?php
+/*
+ * Copyright (c) 2017 Hubzilla
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a copy
+ * of this software and associated documentation files (the "Software"), to deal
+ * in the Software without restriction, including without limitation the rights
+ * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+ * copies of the Software, and to permit persons to whom the Software is
+ * furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice shall be included in
+ * all copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+ * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+ * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+ * SOFTWARE.
+ */
+
+namespace Zotlabs\Tests\Unit\includes;
+
+use Zotlabs\Tests\Unit\DatabaseTestCase;
+use PHPUnit\DbUnit\TestCaseTrait;
+use PHPUnit\DbUnit\DataSet\YamlDataSet;
+
+require_once 'include/dba/dba_pdo.php';
+
+/**
+ * @brief Unit Test case for include/dba/dba_pdo.php file.
+ *
+ * Some very basic tests to see if our database layer can connect to a real
+ * database.
+ */
+class dba_pdoTest extends DatabaseTestCase {
+
+	use TestCaseTrait;
+
+	/**
+	 * @var \dba_driver
+	 */
+	protected $dba;
+
+
+	/**
+	 * Set initial state of the database before each test is executed.
+	 * Load database fixtures.
+	 *
+	 * @return \PHPUnit\DbUnit\DataSet\IDataSet
+	 */
+	public function getDataSet() {
+		return new YamlDataSet(dirname(__FILE__) . '/_files/account.yml');
+	}
+
+	protected function setUp() {
+		// Will invoke getDataSet() to load fixtures into DB
+		parent::setUp();
+
+		$this->dba = new \dba_pdo(
+				\getenv('hz_db_server'),
+				\getenv('hz_db_scheme'),
+				\getenv('hz_db_port'),
+				\getenv('hz_db_user'),
+				\getenv('hz_db_pass'),
+				\getenv('hz_db_database')
+		);
+	}
+	protected function assertPreConditions() {
+		$this->assertSame('pdo', $this->dba->getdriver(), "Driver is expected to be 'pdo'.");
+		$this->assertInstanceOf('dba_driver', $this->dba);
+		$this->assertTrue($this->dba->connected, 'Pre condition failed, DB is not connected.');
+		$this->assertInstanceOf('PDO', $this->dba->db);
+	}
+	protected function tearDown() {
+		$this->dba = null;
+	}
+
+
+	/**
+	 * @group mysql
+	 */
+	public function testQuoteintervalOnMysql() {
+		$this->assertSame('value', $this->dba->quote_interval('value'));
+	}
+	/**
+	 * @group postgresql
+	 */
+	public function testQuoteintervalOnPostgresql() {
+		$this->assertSame("'value'", $this->dba->quote_interval('value'));
+	}
+
+	/**
+	 * @group mysql
+	 */
+	public function testGenerateMysqlConcatSql() {
+		$this->assertSame('GROUP_CONCAT(DISTINCT field SEPARATOR \';\')', $this->dba->concat('field', ';'));
+		$this->assertSame('GROUP_CONCAT(DISTINCT field2 SEPARATOR \' \')', $this->dba->concat('field2', ' '));
+	}
+	/**
+	 * @group postgresql
+	 */
+	public function testGeneratePostgresqlConcatSql() {
+		$this->assertSame('string_agg(field,\';\')', $this->dba->concat('field', ';'));
+		$this->assertSame('string_agg(field2,\' \')', $this->dba->concat('field2', ' '));
+	}
+
+
+	public function testConnectToSqlServer() {
+		// connect() is done in dba_pdo constructor which is called in setUp()
+		$this->assertTrue($this->dba->connected);
+	}
+
+	/**
+	 * @depends testConnectToSqlServer
+	 */
+	public function testCloseSqlServerConnection() {
+		$this->dba->close();
+
+		$this->assertNull($this->dba->db);
+		$this->assertFalse($this->dba->connected);
+	}
+
+	/**
+	 * @depends testConnectToSqlServer
+	 */
+	public function testSelectQueryShouldReturnArray() {
+		$ret = $this->dba->q('SELECT * FROM account');
+
+		$this->assertTrue(is_array($ret));
+	}
+
+	/**
+	 * @depends testConnectToSqlServer
+	 */
+	public function testInsertQueryShouldReturnPdostatement() {
+		// Fixture account.yml adds two entries to account table
+		$this->assertEquals(2, $this->getConnection()->getRowCount('account'), 'Pre-Condition');
+
+		$ret = $this->dba->q('INSERT INTO account
+				(account_id, account_email, account_language)
+				 VALUES (100, \'insert@example.com\', \'de\')
+		');
+		$this->assertInstanceOf('PDOStatement', $ret);
+
+		$this->assertEquals(3, $this->getConnection()->getRowCount('account'), 'Inserting failed');
+	}
+
+
+	public function testConnectToWrongSqlServer() {
+		$nodba = new \dba_pdo('wrongserver',
+				\getenv('hz_db_scheme'), \getenv('hz_db_port'),
+				\getenv('hz_db_user'), \getenv('hz_db_pass'),
+				\getenv('hz_db_database')
+		);
+
+		$this->assertSame('pdo', $nodba->getdriver());
+		$this->assertInstanceOf('dba_pdo', $nodba);
+		$this->assertFalse($nodba->connected);
+		$this->assertNull($nodba->db);
+
+		$this->assertFalse($nodba->q('SELECT * FROM account'));
+	}
+
+}
-- 
cgit v1.2.3


From 4bf0c9e36a4a89d6b84c6bf8a74124603add9d9e Mon Sep 17 00:00:00 2001
From: Klaus Weidenbach <Klaus.Weidenbach@gmx.net>
Date: Mon, 22 Jan 2018 22:59:13 +0100
Subject: :white_check_mark: Add tests for non existent tables.

Prevent PHP warnings "Undefined variable" in dba_pdo::q();
---
 tests/unit/includes/dba/dba_pdoTest.php | 22 ++++++++++++++++++++++
 1 file changed, 22 insertions(+)

(limited to 'tests/unit')

diff --git a/tests/unit/includes/dba/dba_pdoTest.php b/tests/unit/includes/dba/dba_pdoTest.php
index ce6e1ffd6..12e574d42 100644
--- a/tests/unit/includes/dba/dba_pdoTest.php
+++ b/tests/unit/includes/dba/dba_pdoTest.php
@@ -164,4 +164,26 @@ class dba_pdoTest extends DatabaseTestCase {
 		$this->assertFalse($nodba->q('SELECT * FROM account'));
 	}
 
+	/**
+	 * @depends testConnectToSqlServer
+	 */
+	public function testSelectQueryToNonExistentTableShouldReturnFalse() {
+		$ret = $this->dba->q('SELECT * FROM non_existent_table');
+
+		$this->assertFalse($ret);
+	}
+
+	/**
+	 * @depends testConnectToSqlServer
+	 */
+	public function testInsertQueryToNonExistentTableShouldReturnEmptyArray() {
+		$ret = $this->dba->q('INSERT INTO non_existent_table
+				(account_email, account_language)
+				 VALUES (\'email@example.com\', \'en\')
+		');
+
+		$this->assertNotInstanceOf('PDOStatement', $ret);
+		$this->isEmpty($ret);
+	}
+
 }
-- 
cgit v1.2.3