From 9653276ebcd1a5c24302cb8d06f905f90e158e45 Mon Sep 17 00:00:00 2001
From: Nathan Guse <nathaniel.guse@gmail.com>
Date: Fri, 4 Oct 2013 12:10:27 -0500
Subject: [ticket/11881] Better split the timezone conversion into chunks; add
 test

PHPBB3-11881
---
 tests/dbal/convert_timezones.php          | 94 +++++++++++++++++++++++++++++++
 tests/dbal/fixtures/convert_timezones.xml | 80 ++++++++++++++++++++++++++
 2 files changed, 174 insertions(+)
 create mode 100644 tests/dbal/convert_timezones.php
 create mode 100644 tests/dbal/fixtures/convert_timezones.xml

(limited to 'tests')

diff --git a/tests/dbal/convert_timezones.php b/tests/dbal/convert_timezones.php
new file mode 100644
index 0000000000..718b28b9ff
--- /dev/null
+++ b/tests/dbal/convert_timezones.php
@@ -0,0 +1,94 @@
+<?php
+/**
+*
+* @package testing
+* @copyright (c) 2013 phpBB Group
+* @license http://opensource.org/licenses/gpl-2.0.php GNU General Public License v2
+*
+*/
+
+class phpbb_dbal_convert_timezones_test extends phpbb_database_test_case
+{
+	protected $notifications, $db, $container, $user, $config, $auth, $cache;
+
+	public function getDataSet()
+	{
+		$this->db = $this->new_dbal();
+		$db_tools = new \phpbb\db\tools($this->db);
+
+		// user_dst doesn't exist anymore, must re-add it to test this
+		$db_tools->sql_column_add('phpbb_users', 'user_dst', array('BOOL', 1));
+
+		return $this->createXMLDataSet(dirname(__FILE__) . '/fixtures/convert_timezones.xml');
+	}
+
+	public function revert_schema()
+	{
+		return array(
+			'drop_columns'	=> array(
+				$this->table_prefix . 'users'			=> array(
+					'user_dst',
+				),
+			),
+		);
+	}
+
+	public function update_schema()
+	{
+		return array(
+			'add_columns'	=> array(
+				$this->table_prefix . 'users'			=> array(
+					'user_dst'		=> array('BOOL', 0),
+				),
+			),
+		);
+	}
+
+	protected function setUp()
+	{
+		parent::setUp();
+
+		global $phpbb_root_path, $phpEx;
+
+		$this->db = $this->new_dbal();
+
+		$this->migration = new \phpbb\db\migration\data\v310\timezone(
+			new \phpbb\config\config(array()),
+			$this->db,
+			new \phpbb\db\tools($this->db),
+			$phpbb_root_path,
+			$phpEx,
+			'phpbb_'
+		);
+	}
+
+	protected $expected_results = array(
+		//user_id => user_timezone
+		1 => 'Etc/GMT+12',
+		2 => 'Etc/GMT+11',
+		3 => 'Etc/GMT-3',
+		4 => 'Etc/GMT-4',
+		5 => 'America/St_Johns',
+		6 => 'Australia/Eucla',
+	);
+
+	public function test_convert()
+	{
+		$this->migration->update_timezones(0);
+
+		$sql = 'SELECT user_id, user_timezone
+			FROM phpbb_users
+			ORDER BY user_id ASC';
+		$result = $this->db->sql_query($sql);
+		while ($row = $this->db->sql_fetchrow($result))
+		{
+			$this->assertEquals($this->expected_results[$row['user_id']], $row['user_timezone']);
+		}
+		$this->db->sql_freeresult($result);
+
+		$db_tools = new \phpbb\db\tools($this->db);
+
+		// Remove the user_dst field again
+		$db_tools->sql_column_remove('phpbb_users', 'user_dst');
+	}
+}
diff --git a/tests/dbal/fixtures/convert_timezones.xml b/tests/dbal/fixtures/convert_timezones.xml
new file mode 100644
index 0000000000..ce941d8b74
--- /dev/null
+++ b/tests/dbal/fixtures/convert_timezones.xml
@@ -0,0 +1,80 @@
+<?xml version="1.0" encoding="UTF-8" ?>
+<dataset>
+	<table name="phpbb_users">
+		<column>user_id</column>
+		<column>username</column>
+		<column>username_clean</column>
+		<column>user_permissions</column>
+		<column>user_sig</column>
+		<column>user_occ</column>
+		<column>user_interests</column>
+		<column>user_timezone</column>
+		<column>user_dst</column>
+		<row>
+			<value>1</value>
+			<value>1</value>
+			<value>1</value>
+			<value></value>
+			<value></value>
+			<value></value>
+			<value></value>
+			<value>-12</value>
+			<value>0</value>
+		</row>
+		<row>
+			<value>2</value>
+			<value>2</value>
+			<value>2</value>
+			<value></value>
+			<value></value>
+			<value></value>
+			<value></value>
+			<value>-12</value>
+			<value>1</value>
+		</row>
+		<row>
+			<value>3</value>
+			<value>3</value>
+			<value>3</value>
+			<value></value>
+			<value></value>
+			<value></value>
+			<value></value>
+			<value>3</value>
+			<value>0</value>
+		</row>
+		<row>
+			<value>4</value>
+			<value>4</value>
+			<value>4</value>
+			<value></value>
+			<value></value>
+			<value></value>
+			<value></value>
+			<value>3</value>
+			<value>1</value>
+		</row>
+		<row>
+			<value>5</value>
+			<value>5</value>
+			<value>5</value>
+			<value></value>
+			<value></value>
+			<value></value>
+			<value></value>
+			<value>-3.5</value>
+			<value>0</value>
+		</row>
+		<row>
+			<value>6</value>
+			<value>6</value>
+			<value>6</value>
+			<value></value>
+			<value></value>
+			<value></value>
+			<value></value>
+			<value>8.75</value>
+			<value>0</value>
+		</row>
+	</table>
+</dataset>
-- 
cgit v1.2.1