mirror of
https://github.com/phpbb/phpbb.git
synced 2025-06-08 12:28:52 +00:00
[ticket/10899] Add unit tests
PHPBB3-10899
This commit is contained in:
Tristan Darricau
parent
ac1eb2d789
commit
05e76e55e1
3 changed files with 327 additions and 14 deletions
|
|
@ -334,19 +334,41 @@ class log implements \phpbb\log\log_interface
|
||||||
*/
|
*/
|
||||||
public function delete($mode, $conditions = array())
|
public function delete($mode, $conditions = array())
|
||||||
{
|
{
|
||||||
$sql_where = '';
|
switch ($mode)
|
||||||
$started = false;
|
{
|
||||||
|
case 'admin':
|
||||||
|
$log_type = LOG_ADMIN;
|
||||||
|
break;
|
||||||
|
|
||||||
|
case 'mod':
|
||||||
|
$log_type = LOG_MOD;
|
||||||
|
break;
|
||||||
|
|
||||||
|
case 'user':
|
||||||
|
$log_type = LOG_USERS;
|
||||||
|
break;
|
||||||
|
|
||||||
|
case 'users':
|
||||||
|
$log_type = LOG_USERS;
|
||||||
|
break;
|
||||||
|
|
||||||
|
case 'critical':
|
||||||
|
$log_type = LOG_CRITICAL;;
|
||||||
|
break;
|
||||||
|
|
||||||
|
default:
|
||||||
|
$log_type = false;
|
||||||
|
}
|
||||||
|
|
||||||
|
if ($log_type === false)
|
||||||
|
{
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
$sql_where = 'WHERE log_type = ' . $log_type;
|
||||||
foreach ($conditions as $field => $field_value)
|
foreach ($conditions as $field => $field_value)
|
||||||
{
|
|
||||||
if ($started)
|
|
||||||
{
|
{
|
||||||
$sql_where .= ' AND ';
|
$sql_where .= ' AND ';
|
||||||
}
|
|
||||||
else
|
|
||||||
{
|
|
||||||
$sql_where = 'WHERE ';
|
|
||||||
$started = true;
|
|
||||||
}
|
|
||||||
|
|
||||||
if ($field == 'keywords')
|
if ($field == 'keywords')
|
||||||
{
|
{
|
||||||
|
|
@ -354,11 +376,11 @@ class log implements \phpbb\log\log_interface
|
||||||
}
|
}
|
||||||
else
|
else
|
||||||
{
|
{
|
||||||
if (is_array($field_value) && sizeof($field_value) == 2)
|
if (is_array($field_value) && sizeof($field_value) == 2 && is_string($field_value[0]))
|
||||||
{
|
{
|
||||||
$sql_where .= $field . ' ' . $field_value[0] . ' ' . $field_value[1];
|
$sql_where .= $field . ' ' . $field_value[0] . ' ' . $field_value[1];
|
||||||
}
|
}
|
||||||
else if (is_array($field_value) && sizeof($field_value) > 2)
|
else if (is_array($field_value))
|
||||||
{
|
{
|
||||||
$sql_where .= $this->db->sql_in_set($field, $field_value);
|
$sql_where .= $this->db->sql_in_set($field, $field_value);
|
||||||
}
|
}
|
||||||
|
|
@ -366,7 +388,6 @@ class log implements \phpbb\log\log_interface
|
||||||
{
|
{
|
||||||
$sql_where .= $field . ' = ' . $field_value;
|
$sql_where .= $field . ' = ' . $field_value;
|
||||||
}
|
}
|
||||||
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
||||||
60
tests/log/delete_test.php
Normal file
60
tests/log/delete_test.php
Normal file
|
|
@ -0,0 +1,60 @@
|
||||||
|
<?php
|
||||||
|
/**
|
||||||
|
*
|
||||||
|
* @package testing
|
||||||
|
* @copyright (c) 2012 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/utf/utf_tools.php';
|
||||||
|
|
||||||
|
class phpbb_log_add_test extends phpbb_database_test_case
|
||||||
|
{
|
||||||
|
public function getDataSet()
|
||||||
|
{
|
||||||
|
return $this->createXMLDataSet(dirname(__FILE__) . '/fixtures/delete_log.xml');
|
||||||
|
}
|
||||||
|
|
||||||
|
public function test_log_delete()
|
||||||
|
{
|
||||||
|
global $phpbb_root_path, $phpEx, $db, $phpbb_dispatcher, $auth;
|
||||||
|
|
||||||
|
$db = $this->new_dbal();
|
||||||
|
$cache = new phpbb_mock_cache;
|
||||||
|
$phpbb_dispatcher = new phpbb_mock_event_dispatcher();
|
||||||
|
$user = $this->getMock('\phpbb\user');
|
||||||
|
$user->data['user_id'] = 1;
|
||||||
|
$auth = $this->getMock('\phpbb\auth\auth');
|
||||||
|
|
||||||
|
$log = new \phpbb\log\log($db, $user, $auth, $phpbb_dispatcher, $phpbb_root_path, 'adm/', $phpEx, LOG_TABLE);
|
||||||
|
|
||||||
|
// Delete all admin logs
|
||||||
|
$this->assertCount(2, $log->get_logs('admin'));
|
||||||
|
$log->delete('admin');
|
||||||
|
// One entry is added to the admin log when the logs are purged
|
||||||
|
$this->assertCount(1, $log->get_logs('admin'));
|
||||||
|
|
||||||
|
// Delete with keyword
|
||||||
|
$this->assertCount(1, $log->get_logs('mod', false, 0, 0, 0, 0, 0, 0, 'l.log_time DESC', 'guest'));
|
||||||
|
$log->delete('mod', array('keywords' => 'guest'));
|
||||||
|
$this->assertEmpty($log->get_logs('mod', false, 0, 0, 0, 0, 0, 0, 'l.log_time DESC', 'guest'));
|
||||||
|
|
||||||
|
// Delete with simples conditions
|
||||||
|
$this->assertCount(3, $log->get_logs('mod', false, 0, 0, 12, 0, 1, 0, 'l.log_time DESC'));
|
||||||
|
$log->delete('mod', array('forum_id' => 12, 'user_id' => 1));
|
||||||
|
$this->assertEmpty($log->get_logs('mod', false, 0, 0, 12, 0, 1, 0, 'l.log_time DESC'));
|
||||||
|
|
||||||
|
// Delete with IN condition
|
||||||
|
$this->assertCount(2, $log->get_logs('mod', false, 0, 0, array(13, 14), 0, 0, 0, 'l.log_time DESC'));
|
||||||
|
$log->delete('mod', array('forum_id' => array(14, 13)));
|
||||||
|
$this->assertEmpty($log->get_logs('mod', false, 0, 0, array(13, 14), 0, 0, 0, 'l.log_time DESC'));
|
||||||
|
|
||||||
|
// Delete with a custom condition (ie: WHERE x >= 10)
|
||||||
|
$this->assertCount(3, $log->get_logs('critical', false, 0, 0, 0, 0, 0, 0, 'l.log_time DESC'));
|
||||||
|
$log->delete('critical', array('user_id' => array('>', 1)));
|
||||||
|
$this->assertCount(1, $log->get_logs('critical', false, 0, 0, 0, 0, 0, 0, 'l.log_time DESC'));
|
||||||
|
}
|
||||||
|
}
|
||||||
232
tests/log/fixtures/delete_log.xml
Normal file
232
tests/log/fixtures/delete_log.xml
Normal file
|
|
@ -0,0 +1,232 @@
|
||||||
|
<?xml version="1.0" encoding="UTF-8" ?>
|
||||||
|
<dataset>
|
||||||
|
<table name="phpbb_log">
|
||||||
|
<column>log_id</column>
|
||||||
|
<column>log_type</column>
|
||||||
|
<column>user_id</column>
|
||||||
|
<column>forum_id</column>
|
||||||
|
<column>topic_id</column>
|
||||||
|
<column>reportee_id</column>
|
||||||
|
<column>log_ip</column>
|
||||||
|
<column>log_time</column>
|
||||||
|
<column>log_operation</column>
|
||||||
|
<column>log_data</column>
|
||||||
|
<row>
|
||||||
|
<value>1</value>
|
||||||
|
<value>0</value>
|
||||||
|
<value>1</value>
|
||||||
|
<value>0</value>
|
||||||
|
<value>0</value>
|
||||||
|
<value>0</value>
|
||||||
|
<value>127.0.0.1</value>
|
||||||
|
<value>1</value>
|
||||||
|
<value>LOG_INSTALL_INSTALLED</value>
|
||||||
|
<value>a:1:{i:0;s:9:"3.1.0-dev";}</value>
|
||||||
|
</row>
|
||||||
|
<row>
|
||||||
|
<value>2</value>
|
||||||
|
<value>0</value>
|
||||||
|
<value>1</value>
|
||||||
|
<value>0</value>
|
||||||
|
<value>0</value>
|
||||||
|
<value>0</value>
|
||||||
|
<value>127.0.0.1</value>
|
||||||
|
<value>1</value>
|
||||||
|
<value>LOG_KEY_NOT_EXISTS</value>
|
||||||
|
<value>a:1:{i:0;s:15:"additional_data";}</value>
|
||||||
|
</row>
|
||||||
|
<row>
|
||||||
|
<value>3</value>
|
||||||
|
<value>2</value>
|
||||||
|
<value>1</value>
|
||||||
|
<value>0</value>
|
||||||
|
<value>0</value>
|
||||||
|
<value>0</value>
|
||||||
|
<value>127.0.0.1</value>
|
||||||
|
<value>1</value>
|
||||||
|
<value>LOG_CRITICAL</value>
|
||||||
|
<value>a:1:{i:0;s:13:"critical data";}</value>
|
||||||
|
</row>
|
||||||
|
<row>
|
||||||
|
<value>4</value>
|
||||||
|
<value>1</value>
|
||||||
|
<value>1</value>
|
||||||
|
<value>12</value>
|
||||||
|
<value>34</value>
|
||||||
|
<value>0</value>
|
||||||
|
<value>127.0.0.1</value>
|
||||||
|
<value>1</value>
|
||||||
|
<value>LOG_MOD</value>
|
||||||
|
<value></value>
|
||||||
|
</row>
|
||||||
|
<row>
|
||||||
|
<value>5</value>
|
||||||
|
<value>1</value>
|
||||||
|
<value>1</value>
|
||||||
|
<value>12</value>
|
||||||
|
<value>45</value>
|
||||||
|
<value>0</value>
|
||||||
|
<value>127.0.0.1</value>
|
||||||
|
<value>1</value>
|
||||||
|
<value>LOG_MOD</value>
|
||||||
|
<value></value>
|
||||||
|
</row>
|
||||||
|
<row>
|
||||||
|
<value>6</value>
|
||||||
|
<value>1</value>
|
||||||
|
<value>1</value>
|
||||||
|
<value>23</value>
|
||||||
|
<value>56</value>
|
||||||
|
<value>0</value>
|
||||||
|
<value>127.0.0.1</value>
|
||||||
|
<value>1</value>
|
||||||
|
<value>LOG_MOD</value>
|
||||||
|
<value></value>
|
||||||
|
</row>
|
||||||
|
<row>
|
||||||
|
<value>7</value>
|
||||||
|
<value>1</value>
|
||||||
|
<value>1</value>
|
||||||
|
<value>12</value>
|
||||||
|
<value>45</value>
|
||||||
|
<value>0</value>
|
||||||
|
<value>127.0.0.1</value>
|
||||||
|
<value>1</value>
|
||||||
|
<value>LOG_MOD2</value>
|
||||||
|
<value></value>
|
||||||
|
</row>
|
||||||
|
<row>
|
||||||
|
<value>8</value>
|
||||||
|
<value>3</value>
|
||||||
|
<value>1</value>
|
||||||
|
<value>0</value>
|
||||||
|
<value>0</value>
|
||||||
|
<value>2</value>
|
||||||
|
<value>127.0.0.1</value>
|
||||||
|
<value>1</value>
|
||||||
|
<value>LOG_USER</value>
|
||||||
|
<value>a:1:{i:0;s:5:"admin";}</value>
|
||||||
|
</row>
|
||||||
|
<row>
|
||||||
|
<value>9</value>
|
||||||
|
<value>3</value>
|
||||||
|
<value>1</value>
|
||||||
|
<value>0</value>
|
||||||
|
<value>0</value>
|
||||||
|
<value>1</value>
|
||||||
|
<value>127.0.0.1</value>
|
||||||
|
<value>1</value>
|
||||||
|
<value>LOG_USER</value>
|
||||||
|
<value>a:1:{i:0;s:5:"guest";}</value>
|
||||||
|
</row>
|
||||||
|
<row>
|
||||||
|
<value>10</value>
|
||||||
|
<value>3</value>
|
||||||
|
<value>1</value>
|
||||||
|
<value>0</value>
|
||||||
|
<value>0</value>
|
||||||
|
<value>0</value>
|
||||||
|
<value>127.0.0.1</value>
|
||||||
|
<value>1</value>
|
||||||
|
<value>LOG_SINGULAR_PLURAL</value>
|
||||||
|
<value>a:1:{i:0;i:2;}</value>
|
||||||
|
</row>
|
||||||
|
<row>
|
||||||
|
<value>11</value>
|
||||||
|
<value>1</value>
|
||||||
|
<value>1</value>
|
||||||
|
<value>15</value>
|
||||||
|
<value>3</value>
|
||||||
|
<value>0</value>
|
||||||
|
<value>127.0.0.1</value>
|
||||||
|
<value>1</value>
|
||||||
|
<value>LOG_MOD3</value>
|
||||||
|
<value>a:1:{i:0;s:5:"guest";}</value>
|
||||||
|
</row>
|
||||||
|
<row>
|
||||||
|
<value>12</value>
|
||||||
|
<value>1</value>
|
||||||
|
<value>1</value>
|
||||||
|
<value>13</value>
|
||||||
|
<value>0</value>
|
||||||
|
<value>0</value>
|
||||||
|
<value>127.0.0.1</value>
|
||||||
|
<value>1</value>
|
||||||
|
<value></value>
|
||||||
|
<value></value>
|
||||||
|
</row>
|
||||||
|
<row>
|
||||||
|
<value>13</value>
|
||||||
|
<value>1</value>
|
||||||
|
<value>1</value>
|
||||||
|
<value>14</value>
|
||||||
|
<value>0</value>
|
||||||
|
<value>0</value>
|
||||||
|
<value>127.0.0.1</value>
|
||||||
|
<value>1</value>
|
||||||
|
<value></value>
|
||||||
|
<value></value>
|
||||||
|
</row>
|
||||||
|
<row>
|
||||||
|
<value>14</value>
|
||||||
|
<value>2</value>
|
||||||
|
<value>2</value>
|
||||||
|
<value>0</value>
|
||||||
|
<value>0</value>
|
||||||
|
<value>0</value>
|
||||||
|
<value>127.0.0.1</value>
|
||||||
|
<value>1</value>
|
||||||
|
<value></value>
|
||||||
|
<value></value>
|
||||||
|
</row>
|
||||||
|
<row>
|
||||||
|
<value>15</value>
|
||||||
|
<value>2</value>
|
||||||
|
<value>2</value>
|
||||||
|
<value>0</value>
|
||||||
|
<value>0</value>
|
||||||
|
<value>0</value>
|
||||||
|
<value>127.0.0.1</value>
|
||||||
|
<value>1</value>
|
||||||
|
<value></value>
|
||||||
|
<value></value>
|
||||||
|
</row>
|
||||||
|
</table>
|
||||||
|
<table name="phpbb_users">
|
||||||
|
<column>user_id</column>
|
||||||
|
<column>username</column>
|
||||||
|
<column>username_clean</column>
|
||||||
|
<column>user_permissions</column>
|
||||||
|
<column>user_sig</column>
|
||||||
|
<row>
|
||||||
|
<value>1</value>
|
||||||
|
<value>Anonymous</value>
|
||||||
|
<value>Anonymous</value>
|
||||||
|
<value></value>
|
||||||
|
<value></value>
|
||||||
|
</row>
|
||||||
|
<row>
|
||||||
|
<value>2</value>
|
||||||
|
<value>admin</value>
|
||||||
|
<value>admin</value>
|
||||||
|
<value></value>
|
||||||
|
<value></value>
|
||||||
|
</row>
|
||||||
|
</table>
|
||||||
|
<table name="phpbb_topics">
|
||||||
|
<column>topic_id</column>
|
||||||
|
<column>forum_id</column>
|
||||||
|
<row>
|
||||||
|
<value>34</value>
|
||||||
|
<value>12</value>
|
||||||
|
</row>
|
||||||
|
<row>
|
||||||
|
<value>45</value>
|
||||||
|
<value>12</value>
|
||||||
|
</row>
|
||||||
|
<row>
|
||||||
|
<value>56</value>
|
||||||
|
<value>23</value>
|
||||||
|
</row>
|
||||||
|
</table>
|
||||||
|
</dataset>
|
||||||
Loading…
Add table
Reference in a new issue