[ticket/11405] Add some basic tests for notifications in submit_post()

Poster, should NOT receive a notification Topic subscribed, should receive a notification Topic subscribed, but unauthed to read, should NOT receive a notification Topic subscribed, but already notified, should NOT receive a new notification Topic and forum subscribed, should receive ONE notification Forum subscribed, should receive a notification Forum subscribed, but already notified, should NOT receive a new notification PHPBB3-11405
author: Joas Schilling <nickvergessen@gmx.de> 2013-03-23 13:20:32 +0100
committer: Joas Schilling <nickvergessen@gmx.de> 2013-03-23 13:20:32 +0100
commit: bc5f8e30d09be1ded463b7d7a0e0d9b001de6b3b (patch)
tree: 292f64d65587e0b3be41de1b112aee1fc23e502d /tests/notification
parent: b51a66b60bffd0e4e07a7bd8db6532762c52678d (diff)
download: forums-bc5f8e30d09be1ded463b7d7a0e0d9b001de6b3b.tar
forums-bc5f8e30d09be1ded463b7d7a0e0d9b001de6b3b.tar.gz
forums-bc5f8e30d09be1ded463b7d7a0e0d9b001de6b3b.tar.bz2
forums-bc5f8e30d09be1ded463b7d7a0e0d9b001de6b3b.tar.xz
forums-bc5f8e30d09be1ded463b7d7a0e0d9b001de6b3b.zip
2 files changed, 327 insertions, 0 deletions
diff --git a/tests/notification/fixtures/submit_post_notification.xml b/tests/notification/fixtures/submit_post_notification.xml
new file mode 100644
index 0000000000..c082402056
--- /dev/null
+++ b/tests/notification/fixtures/submit_post_notification.xml
@@ -0,0 +1,132 @@
+<?xml version="1.0" encoding="UTF-8" ?>
+<dataset>
+	<table name="phpbb_forums_watch">
+		<column>forum_id</column>
+		<column>user_id</column>
+		<column>notify_status</column>
+		<row>
+			<value>1</value>
+			<value>6</value>
+			<value>0</value>
+		</row>
+		<row>
+			<value>1</value>
+			<value>7</value>
+			<value>0</value>
+		</row>
+		<row>
+			<value>1</value>
+			<value>8</value>
+			<value>0</value>
+		</row>
+	</table>
+	<table name="phpbb_notifications">
+		<column>item_type</column>
+		<column>user_id</column>
+		<column>item_id</column>
+		<column>item_parent_id</column>
+		<column>notification_data</column>
+		<row>
+			<value>post</value>
+			<value>5</value>
+			<value>1</value>
+			<value>1</value>
+			<value></value>
+		</row>
+		<row>
+			<value>post</value>
+			<value>8</value>
+			<value>1</value>
+			<value>1</value>
+			<value></value>
+		</row>
+	</table>
+	<table name="phpbb_posts">
+	</table>
+	<table name="phpbb_topics_watch">
+		<column>topic_id</column>
+		<column>user_id</column>
+		<column>notify_status</column>
+		<row>
+			<value>1</value>
+			<value>2</value>
+			<value>0</value>
+		</row>
+		<row>
+			<value>1</value>
+			<value>3</value>
+			<value>0</value>
+		</row>
+		<row>
+			<value>1</value>
+			<value>4</value>
+			<value>0</value>
+		</row>
+		<row>
+			<value>1</value>
+			<value>5</value>
+			<value>0</value>
+		</row>
+		<row>
+			<value>1</value>
+			<value>6</value>
+			<value>0</value>
+		</row>
+	</table>
+	<table name="phpbb_user_notifications">
+		<column>item_type</column>
+		<column>item_id</column>
+		<column>user_id</column>
+		<column>method</column>
+		<column>notify</column>
+		<row>
+			<value>post</value>
+			<value>0</value>
+			<value>2</value>
+			<value></value>
+			<value>1</value>
+		</row>
+		<row>
+			<value>post</value>
+			<value>0</value>
+			<value>3</value>
+			<value></value>
+			<value>1</value>
+		</row>
+		<row>
+			<value>post</value>
+			<value>0</value>
+			<value>4</value>
+			<value></value>
+			<value>1</value>
+		</row>
+		<row>
+			<value>post</value>
+			<value>0</value>
+			<value>5</value>
+			<value></value>
+			<value>1</value>
+		</row>
+		<row>
+			<value>post</value>
+			<value>0</value>
+			<value>6</value>
+			<value></value>
+			<value>1</value>
+		</row>
+		<row>
+			<value>post</value>
+			<value>0</value>
+			<value>7</value>
+			<value></value>
+			<value>1</value>
+		</row>
+		<row>
+			<value>post</value>
+			<value>0</value>
+			<value>8</value>
+			<value></value>
+			<value>1</value>
+		</row>
+	</table>
+</dataset>
diff --git a/tests/notification/submit_post_notifications_test.php b/tests/notification/submit_post_notifications_test.php
new file mode 100644
index 0000000000..e08d7a439a
--- /dev/null
+++ b/tests/notification/submit_post_notifications_test.php
@@ -0,0 +1,195 @@
+<?php
+/**
+*
+* @package testing
+* @copyright (c) 2013 phpBB Group
+* @license http://opensource.org/licenses/gpl-2.0.php GNU General Public License v2
+*
+*/
+
+require_once dirname(__FILE__) . '/../../phpBB/includes/functions.php';
+require_once dirname(__FILE__) . '/../../phpBB/includes/functions_content.php';
+require_once dirname(__FILE__) . '/../../phpBB/includes/functions_posting.php';
+require_once dirname(__FILE__) . '/../../phpBB/includes/utf/utf_tools.php';
+
+class phpbb_notification_submit_post_notifications_test extends phpbb_database_test_case
+{
+	protected $notifications, $db, $container, $user, $config, $auth, $cache;
+
+	public function getDataSet()
+	{
+		return $this->createXMLDataSet(dirname(__FILE__) . '/fixtures/submit_post_notification.xml');
+	}
+
+	public function setUp()
+	{
+		parent::setUp();
+
+		global $auth, $cache, $config, $db, $phpbb_container, $phpbb_dispatcher, $user, $request, $phpEx, $phpbb_root_path;
+
+		// Database
+		$this->db = $this->new_dbal();
+		$db = $this->db;
+
+		// Cache
+		$cache = new phpbb_mock_cache();
+
+		// Auth
+		$auth = $this->getMock('phpbb_auth');
+		$auth->expects($this->any())
+			->method('acl_get')
+			->with($this->stringContains('_'),
+				$this->anything())
+			->will($this->returnValueMap(array(
+				array('f_noapprove', 1, true),
+				array('f_postcount', 1, true),
+				array('m_edit', 1, false),
+			)));
+
+		$auth->expects($this->any())
+			->method('acl_get_list')
+			->with($this->anything(),
+				$this->stringContains('_'),
+				$this->greaterThan(0))
+			->will($this->returnValueMap(array(
+				array(
+					array(
+						0 => '3',
+						1 => '4',
+						2 => '5',
+						3 => '6',
+						5 => '7',
+						6 => '8',
+					),
+					'f_read',
+					1,
+					array(
+						1 => array(
+							'f_read' => array(3, 5, 6, 7, 8,),
+						),
+					),
+				),
+				array(
+					array(
+						0 => '3',
+						1 => '4',
+						2 => '5',
+						3 => '6',
+					),
+					'f_read',
+					1,
+					array(
+						1 => array(
+							'f_read' => array(3, 5, 6,),
+						),
+					),
+				),
+			)));
+
+		// Config
+		$config = new phpbb_config(array('num_topics' => 1,'num_posts' => 1,));
+		set_config(null, null, null, $config);
+		set_config_count(null, null, null, $config);
+
+		// Event dispatcher
+		$phpbb_dispatcher = new phpbb_mock_event_dispatcher();
+
+		// User
+		$user = $this->getMock('phpbb_user');
+		$user->ip = '';
+		$user->data = array(
+			'user_id'		=> 2,
+			'username'		=> 'user-name',
+			'is_registered'	=> true,
+			'user_colour'	=> '',
+		);
+
+		// Request
+		$type_cast_helper = $this->getMock('phpbb_request_type_cast_helper_interface');
+		$request = $this->getMock('phpbb_request');
+
+		// Container
+		$phpbb_container = new phpbb_mock_container_builder();
+
+		$user_loader = new phpbb_user_loader($db, $phpbb_root_path, '.' . $phpEx, USERS_TABLE);
+
+		// Notification Manager
+		$phpbb_notifications = new phpbb_notification_manager(array(), array(),
+			$phpbb_container, $user_loader, $db, $user,
+			$phpbb_root_path, '.' . $phpEx,
+			NOTIFICATION_TYPES_TABLE, NOTIFICATIONS_TABLE, USER_NOTIFICATIONS_TABLE);
+		$phpbb_container->set('notification_manager', $phpbb_notifications);
+
+		// Notification Types
+		$notification_types = array('quote', 'bookmark', 'post');
+		foreach ($notification_types as $type)
+		{
+			$class_name = 'phpbb_notification_type_' . $type;
+			$phpbb_container->set('notification.type.' . $type, new $class_name(
+				$user_loader, $db, $cache, $user, $auth, $config,
+				$phpbb_root_path, '.' . $phpEx,
+				NOTIFICATION_TYPES_TABLE, NOTIFICATIONS_TABLE, USER_NOTIFICATIONS_TABLE));
+		}
+	}
+
+	/**
+	* submit_post() Notifications test
+	*
+	* User => State description
+	*	2	=> Poster, should NOT receive a notification
+	*	3	=> Topic subscribed, should receive a notification
+	*	4	=> Topic subscribed, but unauthed to read, should NOT receive a notification
+	*	5	=> Topic subscribed, but already notified, should NOT receive a new notification
+	*	6	=> Topic and forum subscribed, should receive ONE notification
+	*	7	=> Forum subscribed, should receive a notification
+	*	8	=> Forum subscribed, but already notified, should NOT receive a new notification
+	*/
+	public function test_type_post()
+	{
+		$sql = 'SELECT user_id, item_id, item_parent_id
+			FROM ' . NOTIFICATIONS_TABLE . "
+			WHERE item_type = 'post'
+			ORDER BY user_id, item_id ASC";
+		$result = $this->db->sql_query($sql);
+		$this->assertEquals(array(
+			array('user_id' => 5, 'item_id' => 1, 'item_parent_id' => 1),
+			array('user_id' => 8, 'item_id' => 1, 'item_parent_id' => 1),
+		), $this->db->sql_fetchrowset($result));
+		$this->db->sql_freeresult($result);
+
+		$poll = array();
+		$data = array(
+			'forum_id'		=> 1,
+			'topic_id'		=> 1,
+			'topic_title'	=> 'topic_title',
+			'icon_id'		=> 0,
+			'enable_bbcode'		=> 0,
+			'enable_smilies'	=> 0,
+			'enable_urls'		=> 0,
+			'enable_sig'		=> 0,
+			'message'			=> '',
+			'message_md5'		=> '',
+			'attachment_data'	=> array(),
+			'bbcode_bitfield'	=> '',
+			'bbcode_uid'		=> '',
+			'post_edit_locked'	=> false,
+			//'force_approved_state'	=> 1,
+		);
+
+		submit_post('reply', '', 'poster-name', POST_NORMAL, $poll, $data, false, false);
+
+		$sql = 'SELECT user_id, item_id, item_parent_id
+			FROM ' . NOTIFICATIONS_TABLE . "
+			WHERE item_type = 'post'
+			ORDER BY user_id ASC, item_id ASC";
+		$result = $this->db->sql_query($sql);
+		$this->assertEquals(array(
+			array('user_id' => 3, 'item_id' => 1, 'item_parent_id' => 1,),
+			array('user_id' => 5, 'item_id' => 1, 'item_parent_id' => 1,),
+			array('user_id' => 6, 'item_id' => 1, 'item_parent_id' => 1,),
+			array('user_id' => 7, 'item_id' => 1, 'item_parent_id' => 1,),
+			array('user_id' => 8, 'item_id' => 1, 'item_parent_id' => 1,),
+		), $this->db->sql_fetchrowset($result));
+		$this->db->sql_freeresult($result);
+	}
+}
author	Joas Schilling <nickvergessen@gmx.de>	2013-03-23 13:20:32 +0100
committer	Joas Schilling <nickvergessen@gmx.de>	2013-03-23 13:20:32 +0100
commit	bc5f8e30d09be1ded463b7d7a0e0d9b001de6b3b (patch)
tree	292f64d65587e0b3be41de1b112aee1fc23e502d /tests/notification
parent	b51a66b60bffd0e4e07a7bd8db6532762c52678d (diff)
download	forums-bc5f8e30d09be1ded463b7d7a0e0d9b001de6b3b.tar forums-bc5f8e30d09be1ded463b7d7a0e0d9b001de6b3b.tar.gz forums-bc5f8e30d09be1ded463b7d7a0e0d9b001de6b3b.tar.bz2 forums-bc5f8e30d09be1ded463b7d7a0e0d9b001de6b3b.tar.xz forums-bc5f8e30d09be1ded463b7d7a0e0d9b001de6b3b.zip