Throw IOException from writeHTML() when the file cannot be written

writeHTML() installed ParseException's error handler where the other
writers install IOException's. testFailWriteHTML() called writeXML(), so
the failure path went untested; it now calls writeHTML().

Fixes #92

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

Author: gravitypdf-bot

CHANGELOG.md

@@ -31,6 +31,7 @@ QueryPath Changelog
 - Fix `QueryPath\ParseException` and `QueryPath\IOException` losing the PHP warning they were raised from (issue #91). The message was the error number, as in `2 (/path/to/src/DOMQuery.php: 1343)`, and the code was always `0`. The message now carries the warning text and the code is the PHP error level (for example `E_WARNING`), so code that matches on the old message text will need updating
 - Fix `tag()` emitting an "Undefined property" warning and returning `NULL` when the first match is not an element, such as a text, comment or processing instruction node returned by `contents()`. It now returns an empty string, as it does for an empty match set (issue #93)
 - Fix QueryPath's error handler staying installed for the rest of the request when an exception other than a `QueryPath\Exception` (a `TypeError`, or one thrown by a stream wrapper, for example) escaped from parsing a document, inserting markup, or `writeXML()`/`writeHTML()` (issue #95)
+- Fix `writeHTML()` throwing `QueryPath\ParseException` instead of `QueryPath\IOException` when the file cannot be written, as `writeXML()` and `writeHTML5()` do (issue #92). `IOException` extends `ParseException`, so existing `catch (ParseException $e)` blocks still catch it
 
 # 4.1.0
 

src/DOMQuery.php

@@ -1376,7 +1376,7 @@ public function writeHTML($path = null)
 		if ($path === null) {
 			print $this->saveDocumentHTML();
 		} else {
-			self::withErrorHandler(ParseException::class, function () use ($path) {
+			self::withErrorHandler(IOException::class, function () use ($path) {
 				$this->saveDocumentHTML($path);
 			});
 		}

tests/QueryPath/DOMQueryTest.php

@@ -1497,12 +1497,7 @@ public function testFailWriteXML()
 	{
 		$this->expectException(IOException::class);
 
-		try {
-			qp()->writeXML('./test/no-writing.xml');
-		} catch (Exception $e) {
-			//print $e->getMessage();
-			throw $e;
-		}
+		qp()->writeXML('./test/no-writing.xml');
 	}
 
 	public function testFailWriteXMLReportsWarning()
@@ -1518,24 +1513,14 @@ public function testFailWriteXHTML()
 	{
 		$this->expectException(IOException::class);
 
-		try {
-			qp()->writeXHTML('./test/no-writing.xml');
-		} catch (IOException $e) {
-			//print $e->getMessage();
-			throw $e;
-		}
+		qp()->writeXHTML('./test/no-writing.xml');
 	}
 
 	public function testFailWriteHTML()
 	{
 		$this->expectException(IOException::class);
 
-		try {
-			qp('<?xml version="1.0"?><foo/>')->writeXML('./test/no-writing.xml');
-		} catch (IOException $e) {
-			// print $e->getMessage();
-			throw $e;
-		}
+		htmlqp('<html><body><p>bar</p></body></html>')->writeHTML('./test/no-writing.html');
 	}
 
 	public function testFailWriteHTML5()