Skip to content

Commit b89b7ff

Browse files
committed
ext/dom: applied fixers to improve test robustness
1 parent ba43f3c commit b89b7ff

192 files changed

Lines changed: 1119 additions & 1120 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

ext/dom/tests/DOM4_ChildNode_wrong_document.phpt

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -18,8 +18,8 @@ function test($method) {
1818
try {
1919
$element->$method($dom2->documentElement->firstChild);
2020
echo "FAIL";
21-
} catch (DOMException $e) {
22-
echo $e->getMessage() . "\n";
21+
} catch (Throwable $e) {
22+
echo $e::class, ': ', $e->getMessage(), "\n";
2323
}
2424
}
2525

@@ -29,6 +29,6 @@ test("replaceWith");
2929

3030
?>
3131
--EXPECT--
32-
Wrong Document Error
33-
Wrong Document Error
34-
Wrong Document Error
32+
DOMException: Wrong Document Error
33+
DOMException: Wrong Document Error
34+
DOMException: Wrong Document Error

ext/dom/tests/DOM4_DOMNode_removeDanglingElement.phpt

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,9 +11,9 @@ $element = $dom->createElement('test');
1111

1212
try {
1313
$element->remove();
14-
} catch (DOMException $e) {
15-
echo $e->getMessage();
14+
} catch (Throwable $e) {
15+
echo $e::class, ': ', $e->getMessage(), "\n";
1616
}
1717
?>
1818
--EXPECT--
19-
Not Found Error
19+
DOMException: Not Found Error

ext/dom/tests/DOM4_ParentNode_append_invalidtypes.phpt

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,9 +11,9 @@ $dom->loadXML('<test />');
1111

1212
try {
1313
$dom->documentElement->append(array());
14-
} catch(TypeError $e) {
15-
echo "OK! {$e->getMessage()}";
14+
} catch(Throwable $e) {
15+
echo $e::class, ': ', $e->getMessage(), "\n";
1616
}
1717
?>
1818
--EXPECT--
19-
OK! DOMElement::append(): Argument #1 must be of type DOMNode|string, array given
19+
TypeError: DOMElement::append(): Argument #1 must be of type DOMNode|string, array given

ext/dom/tests/DOM4_ParentNode_append_with_attributes.phpt

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -18,13 +18,13 @@ $element = $dom->documentElement;
1818

1919
try {
2020
$element->append($replacement, $addition);
21-
} catch (DOMException $e) {
22-
echo $e->getMessage() . "\n";
21+
} catch (Throwable $e) {
22+
echo $e::class, ': ', $e->getMessage(), "\n";
2323
}
2424

2525
echo $dom->saveXML();
2626
?>
2727
--EXPECT--
28-
Hierarchy Request Error
28+
DOMException: Hierarchy Request Error
2929
<?xml version="1.0"?>
3030
<test attr-one="21"/>

ext/dom/tests/DOM4_ParentNode_append_wrong_document.phpt

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -18,14 +18,14 @@ function test($method) {
1818
try {
1919
$element->$method($dom2->documentElement->firstChild);
2020
echo "FAIL";
21-
} catch (DOMException $e) {
22-
echo $e->getMessage() . "\n";
21+
} catch (Throwable $e) {
22+
echo $e::class, ': ', $e->getMessage(), "\n";
2323
}
2424
}
2525

2626
test("append");
2727
test("prepend");
2828
?>
2929
--EXPECT--
30-
Wrong Document Error
31-
Wrong Document Error
30+
DOMException: Wrong Document Error
31+
DOMException: Wrong Document Error

ext/dom/tests/DOMAttr_construct_error_001.phpt

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,9 +9,9 @@ dom
99
<?php
1010
try {
1111
$attr = new DOMAttr();
12-
} catch (TypeError $e) {
13-
echo $e->getMessage(), "\n";
12+
} catch (Throwable $e) {
13+
echo $e::class, ': ', $e->getMessage(), "\n";
1414
}
1515
?>
1616
--EXPECT--
17-
DOMAttr::__construct() expects at least 1 argument, 0 given
17+
ArgumentCountError: DOMAttr::__construct() expects at least 1 argument, 0 given

ext/dom/tests/DOMCDATASection_construct_error_001.phpt

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,9 +9,9 @@ dom
99
<?php
1010
try {
1111
$section = new DOMCDataSection();
12-
} catch (TypeError $e) {
13-
echo $e->getMessage();
12+
} catch (Throwable $e) {
13+
echo $e::class, ': ', $e->getMessage(), "\n";
1414
}
1515
?>
1616
--EXPECT--
17-
DOMCdataSection::__construct() expects exactly 1 argument, 0 given
17+
ArgumentCountError: DOMCdataSection::__construct() expects exactly 1 argument, 0 given

ext/dom/tests/DOMCharacterData_data_error_002.phpt

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,9 +10,9 @@ dom
1010
$character_data = new DOMCharacterData();
1111
try {
1212
print $character_data->data;
13-
} catch (DOMException $exception) {
14-
echo $exception->getMessage() . "\n";
13+
} catch (Throwable $exception) {
14+
echo $exception::class, ': ', $exception->getMessage(), "\n";
1515
}
1616
?>
1717
--EXPECT--
18-
Invalid State Error
18+
DOMException: Invalid State Error

ext/dom/tests/DOMCharacterData_deleteData_error_002.phpt

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -15,9 +15,9 @@ $cdata = $document->createCDATASection('test');
1515
$root->appendChild($cdata);
1616
try {
1717
$cdata->deleteData(5, 1);
18-
} catch (DOMException $e) {
19-
echo $e->getMessage();
18+
} catch (Throwable $e) {
19+
echo $e::class, ': ', $e->getMessage(), "\n";
2020
}
2121
?>
2222
--EXPECT--
23-
Index Size Error
23+
DOMException: Index Size Error

ext/dom/tests/DOMCharacterData_length_error_001.phpt

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,9 +10,9 @@ dom
1010
$character_data = new DOMCharacterData();
1111
try {
1212
print $character_data->length;
13-
} catch (DOMException $exception) {
14-
echo $exception->getMessage() . "\n";
13+
} catch (Throwable $exception) {
14+
echo $exception::class, ': ', $exception->getMessage(), "\n";
1515
}
1616
?>
1717
--EXPECT--
18-
Invalid State Error
18+
DOMException: Invalid State Error

0 commit comments

Comments
 (0)