Merge remote-tracking branch 's9e/ticket/11762' into develop-olympus

* s9e/ticket/11762:
  [ticket/11762] Added call to test class's parent::setUp().
  [ticket/11762] Fixed test's filename
  [ticket/11762] Use the === operator to distinguish "0" from ""
This commit is contained in:
Joas Schilling 2013-08-03 15:35:34 +02:00
commit 33eb9ad53a
2 changed files with 40 additions and 2 deletions

View file

@ -413,7 +413,7 @@ function generate_text_for_display($text, $uid, $bitfield, $flags)
{ {
static $bbcode; static $bbcode;
if (!$text) if ($text === '')
{ {
return ''; return '';
} }
@ -459,7 +459,7 @@ function generate_text_for_storage(&$text, &$uid, &$bitfield, &$flags, $allow_bb
$uid = $bitfield = ''; $uid = $bitfield = '';
$flags = (($allow_bbcode) ? OPTION_FLAG_BBCODE : 0) + (($allow_smilies) ? OPTION_FLAG_SMILIES : 0) + (($allow_urls) ? OPTION_FLAG_LINKS : 0); $flags = (($allow_bbcode) ? OPTION_FLAG_BBCODE : 0) + (($allow_smilies) ? OPTION_FLAG_SMILIES : 0) + (($allow_urls) ? OPTION_FLAG_LINKS : 0);
if (!$text) if ($text === '')
{ {
return; return;
} }

View file

@ -0,0 +1,38 @@
<?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__) . '/../mock/user.php';
require_once dirname(__FILE__) . '/../mock/cache.php';
class phpbb_text_processing_generate_text_for_display_test extends phpbb_test_case
{
public function setUp()
{
global $cache, $user;
parent::setUp();
$cache = new phpbb_mock_cache;
$user = new phpbb_mock_user;
$user->optionset('viewcensors', false);
}
public function test_empty_string()
{
$this->assertSame('', generate_text_for_display('', '', '', 0));
}
public function test_zero_string()
{
$this->assertSame('0', generate_text_for_display('0', '', '', 0));
}
}