From c5630d0567b17b93ffb8ea727ad69f6c76fb4975 Mon Sep 17 00:00:00 2001
From: Niels De Graef <nielsdegraef@gmail.com>
Date: Mon, 15 Dec 2025 01:30:59 +0100
Subject: [PATCH] ComposerPageStateTest: Don't use NBSP directly

In some WebKitGTK versions, we were getting the following error from
within WebKit when running the `replace_non_breaking_space` test:

> SyntaxError Invalid character: '\0'

Doing some further investigation, it seems like webkit was choking on
the raw non-breaking space character (NBSP) that we were using in the
test's string literals, specifically the first byte of its raw value
`U+00A0`.

Work around this for now by generating the NBSP in Javascript itself,
rather than passing on the raw value.
---
 test/js/composer-page-state-test.vala | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

diff --git a/test/js/composer-page-state-test.vala b/test/js/composer-page-state-test.vala
index cbb19354..c8e84752 100644
--- a/test/js/composer-page-state-test.vala
+++ b/test/js/composer-page-state-test.vala
@@ -388,18 +388,20 @@ I can send email through smtp.gmail.com:587 or through <a href="https://www.gmai
 
     public void replace_non_breaking_space() throws Error {
         load_body_fixture();
-        string single_nbsp = "a b";
-        string multiple_nbsp = "a b c";
+
+        string nbsp = "String.fromCharCode(160)";
+
+        string single_nbsp = @"'a' + $(nbsp) + 'b'";
+        string multiple_nbsp = @"'a' + $(nbsp) + 'b' + $(nbsp) + 'c'";
+
         try {
             assert(
                 Util.JS.to_string(
-                    run_javascript(@"ComposerPageState.replaceNonBreakingSpace('$(single_nbsp)');")
-
+                    run_javascript(@"ComposerPageState.replaceNonBreakingSpace($(single_nbsp));")
                 ) == "a b");
             assert(
                 Util.JS.to_string(
-                    run_javascript(@"ComposerPageState.replaceNonBreakingSpace('$(multiple_nbsp)');")
-
+                    run_javascript(@"ComposerPageState.replaceNonBreakingSpace($(multiple_nbsp));")
                 ) == "a b c");
         } catch (Util.JS.Error err) {
             print("Util.JS.Error: %s\n", err.message);