diff --git a/tests/HTMLPurifier/AttrDef/CSS/CompositeTest.php b/tests/HTMLPurifier/AttrDef/CSS/CompositeTest.php index 8b5dc20e..384d39c3 100644 --- a/tests/HTMLPurifier/AttrDef/CSS/CompositeTest.php +++ b/tests/HTMLPurifier/AttrDef/CSS/CompositeTest.php @@ -40,9 +40,6 @@ class HTMLPurifier_AttrDef_CSS_CompositeTest extends HTMLPurifier_AttrDefHarness $result = $def->validate($input, $config, $context); $this->assertIdentical($output, $result); - $def1->tally(); - $def2->tally(); - // second test, first def fails, second def works $def1 = new HTMLPurifier_AttrDefMock(); @@ -60,9 +57,6 @@ class HTMLPurifier_AttrDef_CSS_CompositeTest extends HTMLPurifier_AttrDefHarness $result = $def->validate($input, $config, $context); $this->assertIdentical($output, $result); - $def1->tally(); - $def2->tally(); - // third test, all fail, so composite faiils $def1 = new HTMLPurifier_AttrDefMock(); @@ -80,9 +74,6 @@ class HTMLPurifier_AttrDef_CSS_CompositeTest extends HTMLPurifier_AttrDefHarness $result = $def->validate($input, $config, $context); $this->assertIdentical($output, $result); - $def1->tally(); - $def2->tally(); - } } diff --git a/tests/HTMLPurifier/ChildDef/StrictBlockquoteTest.php b/tests/HTMLPurifier/ChildDef/StrictBlockquoteTest.php index 53462046..60cf50f3 100644 --- a/tests/HTMLPurifier/ChildDef/StrictBlockquoteTest.php +++ b/tests/HTMLPurifier/ChildDef/StrictBlockquoteTest.php @@ -75,7 +75,6 @@ extends HTMLPurifier_ChildDefHarness $this->obj = new HTMLPurifier_ChildDef_StrictBlockquote('div | p'); $this->config->set('HTML', 'BlockWrapper', 'dav'); $this->assertResult('Needs wrap', '
Needs wrap
'); - $this->swallowErrors(); } } diff --git a/tests/HTMLPurifier/EncoderTest.php b/tests/HTMLPurifier/EncoderTest.php index 2d648db8..ac35d223 100644 --- a/tests/HTMLPurifier/EncoderTest.php +++ b/tests/HTMLPurifier/EncoderTest.php @@ -36,7 +36,6 @@ class HTMLPurifier_EncoderTest extends HTMLPurifier_Harness "\xF6", // this is invalid 'Expected identical [Binary: F6]' ); - $this->assertNoErrors(); $config = HTMLPurifier_Config::create(array( 'Core.Encoding' => 'ISO-8859-1' diff --git a/tests/HTMLPurifier/ErrorCollectorEMock.php b/tests/HTMLPurifier/ErrorCollectorEMock.php index 8457b764..a2c89837 100644 --- a/tests/HTMLPurifier/ErrorCollectorEMock.php +++ b/tests/HTMLPurifier/ErrorCollectorEMock.php @@ -25,14 +25,14 @@ class HTMLPurifier_ErrorCollectorEMock extends HTMLPurifier_ErrorCollectorMock public function send($v1, $v2) { // test for context - $context =& SimpleTest::getContext(); - $test =& $context->getTest(); + $context = SimpleTest::getContext(); + $test = $context->getTest(); // compat if (empty($this->_mock)) { - $mock =& $this; + $mock = $this; } else { - $mock =& $this->_mock; + $mock = $this->_mock; } foreach ($this->_expected_context as $key => $value) { diff --git a/tests/HTMLPurifier/HTMLModule/TidyTest.php b/tests/HTMLPurifier/HTMLModule/TidyTest.php index 66824f69..b850962d 100644 --- a/tests/HTMLPurifier/HTMLModule/TidyTest.php +++ b/tests/HTMLPurifier/HTMLModule/TidyTest.php @@ -111,10 +111,6 @@ class HTMLPurifier_HTMLModule_TidyTest extends HTMLPurifier_Harness ))); $module->construct($config); - // done - - $module->tally(); - } function test_makeFixesForLevel() { diff --git a/tests/HTMLPurifier/Strategy/CompositeTest.php b/tests/HTMLPurifier/Strategy/CompositeTest.php index 34bb214c..df95297d 100644 --- a/tests/HTMLPurifier/Strategy/CompositeTest.php +++ b/tests/HTMLPurifier/Strategy/CompositeTest.php @@ -59,12 +59,6 @@ class HTMLPurifier_Strategy_CompositeTest extends HTMLPurifier_Harness $output = $composite->execute($input_1, $config, $context); $this->assertIdentical($input_4, $output); - // tally the calls - - $mock_1->tally(); - $mock_2->tally(); - $mock_3->tally(); - } } diff --git a/tests/HTMLPurifier/Strategy/FixNestingTest.php b/tests/HTMLPurifier/Strategy/FixNestingTest.php index a381ce22..908ee2b4 100644 --- a/tests/HTMLPurifier/Strategy/FixNestingTest.php +++ b/tests/HTMLPurifier/Strategy/FixNestingTest.php @@ -108,7 +108,6 @@ class HTMLPurifier_Strategy_FixNestingTest extends HTMLPurifier_StrategyHarness $this->config->set('HTML', 'Parent', 'obviously-impossible'); // $this->expectError('Cannot use unrecognized element as parent'); $this->assertResult('