From ad89817c08e1d1ec884cdc0274c0f13fe02ac3b0 Mon Sep 17 00:00:00 2001
From: "Kelvin M. Klann" <kmk3.code@protonmail.com>
Date: Wed, 13 May 2026 14:14:31 +0000
Subject: [PATCH] test/fs: deduplicate xdg dirs in macro tests (#7163)

To make it easier to add new directories.

Relates to #7147.
---
 test/fs/fs.sh             |  58 ++++-----
 test/fs/macro-subpath.exp | 264 +++++++-------------------------------
 test/fs/macro.exp         | 244 +++++++----------------------------
 3 files changed, 115 insertions(+), 451 deletions(-)

diff --git a/test/fs/fs.sh b/test/fs/fs.sh
index 68d07b98a..7b5b926a8 100755
--- a/test/fs/fs.sh
+++ b/test/fs/fs.sh
@@ -7,14 +7,20 @@ export MALLOC_CHECK_=3
 export MALLOC_PERTURB_=$(($RANDOM % 255 + 1))
 export LC_ALL=C
 
+macro_paths=(
+	"$HOME/Desktop"
+	"$HOME/Documents"
+	"$HOME/Downloads"
+	"$HOME/Music"
+	"$HOME/Pictures"
+	"$HOME/Projects"
+	"$HOME/Videos"
+)
+
 # These directories are required by some tests:
-mkdir -p ~/Desktop
-mkdir -p ~/Documents
-mkdir -p ~/Downloads
-mkdir -p ~/Music
-mkdir -p ~/Pictures
-mkdir -p ~/Projects
-mkdir -p ~/Videos
+for path in "${macro_paths[@]}"; do
+	mkdir -p "$path"
+done
 
 echo "TESTING: tmpfs as regular user (test/fs/tmpfs.exp)"
 ./tmpfs.exp
@@ -104,37 +110,19 @@ echo "TESTING: private-cwd (test/fs/private-cwd.exp)"
 
 echo "TESTING: macros (test/fs/macro.exp)"
 ./macro.exp
-rm -f ~/Desktop/_firejail_test_file
-rm -f ~/Documents/_firejail_test_file
-rm -f ~/Downloads/_firejail_test_file
-rm -f ~/Music/_firejail_test_file
-rm -f ~/Pictures/_firejail_test_file
-rm -f ~/Projects/_firejail_test_file
-rm -f ~/Videos/_firejail_test_file
+for path in "${macro_paths[@]}"; do
+	rm -f "$path/_firejail_test_file"
+done
 
-mkdir -p ~/Desktop/_firejail_test_dir/a
-mkdir -p ~/Desktop/_firejail_test_dir/b
-mkdir -p ~/Documents/_firejail_test_dir/a
-mkdir -p ~/Documents/_firejail_test_dir/b
-mkdir -p ~/Downloads/_firejail_test_dir/a
-mkdir -p ~/Downloads/_firejail_test_dir/b
-mkdir -p ~/Music/_firejail_test_dir/a
-mkdir -p ~/Music/_firejail_test_dir/b
-mkdir -p ~/Pictures/_firejail_test_dir/a
-mkdir -p ~/Pictures/_firejail_test_dir/b
-mkdir -p ~/Projects/_firejail_test_dir/a
-mkdir -p ~/Projects/_firejail_test_dir/b
-mkdir -p ~/Videos/_firejail_test_dir/a
-mkdir -p ~/Videos/_firejail_test_dir/b
+for path in "${macro_paths[@]}"; do
+	mkdir -p "$path/_firejail_test_dir/a"
+	mkdir -p "$path/_firejail_test_dir/b"
+done
 echo "TESTING: macro subpaths (test/fs/macro-subpath.exp)"
 ./macro-subpath.exp
-rm -fr ~/Desktop/_firejail_test_dir
-rm -fr ~/Documents/_firejail_test_dir
-rm -fr ~/Downloads/_firejail_test_dir
-rm -fr ~/Music/_firejail_test_dir
-rm -fr ~/Pictures/_firejail_test_dir
-rm -fr ~/Projects/_firejail_test_dir
-rm -fr ~/Videos/_firejail_test_dir
+for path in "${macro_paths[@]}"; do
+	rm -fr "$path/_firejail_test_dir"
+done
 
 echo "TESTING: whitelist empty (test/fs/whitelist-empty.exp)"
 ./whitelist-empty.exp
diff --git a/test/fs/macro-subpath.exp b/test/fs/macro-subpath.exp
index d9f3280aa..6b5618aaf 100755
--- a/test/fs/macro-subpath.exp
+++ b/test/fs/macro-subpath.exp
@@ -7,227 +7,57 @@ set timeout 3
 spawn $env(SHELL)
 match_max 100000
 
+set macros [ dict create \
+	"DESKTOP"   "$::env(HOME)/Desktop" \
+	"DOCUMENTS" "$::env(HOME)/Documents" \
+	"DOWNLOADS" "$::env(HOME)/Downloads" \
+	"MUSIC"     "$::env(HOME)/Music" \
+	"PICTURES"  "$::env(HOME)/Pictures" \
+	"PROJECTS"  "$::env(HOME)/Projects" \
+	"VIDEOS"    "$::env(HOME)/Videos" \
+]
+
 # Test that macros work with subpaths (see #2359).
-send -- "firejail --profile=./macro-subpath-whitelist.profile find \
-	~/Desktop/_firejail_test_dir \
-	~/Documents/_firejail_test_dir \
-	~/Downloads/_firejail_test_dir \
-	~/Music/_firejail_test_dir \
-	~/Pictures/_firejail_test_dir \
-	~/Projects/_firejail_test_dir \
-	~/Videos/_firejail_test_dir \
-	| LC_ALL=C sort
-\r"
+dict for {name path} $macros {
+	send -- "firejail --profile=./macro-subpath-whitelist.profile find $path/_firejail_test_dir | LC_ALL=C sort\r"
+	expect {
+		timeout {puts "TESTING ERROR 1 $name\n";exit}
+		-re "Child process initialized in \[0-9\]+.\[0-9\]+ ms"
+	}
+	expect {
+		timeout {puts "TESTING ERROR 1.1 $name\n";exit}
+		"$path/_firejail_test_dir/a" {}
+		"$path/_firejail_test_dir/b" {puts "TESTING ERROR 1.2 $name\n";exit}
+	}
+	after 100
+}
 
-expect {
-	timeout {puts "TESTING ERROR 1\n";exit}
-	-re "Child process initialized in \[0-9\]+.\[0-9\]+ ms"
+dict for {name path} $macros {
+	send -- "firejail --profile=./macro-subpath-blacklist.profile ls $path/_firejail_test_dir; echo ret \$?\r"
+	expect {
+		timeout {puts "TESTING ERROR 2 $name\n";exit}
+		-re "Child process initialized in \[0-9\]+.\[0-9\]+ ms"
+	}
+	expect {
+		timeout {puts "TESTING ERROR 2.1 $name\n";exit}
+		"Permission denied" {}
+		-re {ret 0} {puts "TESTING ERROR 2.2 $name\n";exit}
+	}
+	after 100
 }
-expect {
-	timeout {puts "TESTING ERROR 1.1 Desktop\n";exit}
-	"Desktop/_firejail_test_dir/a" {}
-	"Desktop/_firejail_test_dir/b" {puts "TESTING ERROR 1.2 Desktop\n";exit}
-}
-expect {
-	timeout {puts "TESTING ERROR 1.1 Documents\n";exit}
-	"Documents/_firejail_test_dir/a" {}
-	"Documents/_firejail_test_dir/b" {puts "TESTING ERROR 1.2 Documents\n";exit}
-}
-expect {
-	timeout {puts "TESTING ERROR 1.1 Downloads\n";exit}
-	"Downloads/_firejail_test_dir/a" {}
-	"Downloads/_firejail_test_dir/b" {puts "TESTING ERROR 1.2 Downloads\n";exit}
-}
-expect {
-	timeout {puts "TESTING ERROR 1.1 Music\n";exit}
-	"Music/_firejail_test_dir/a" {}
-	"Music/_firejail_test_dir/b" {puts "TESTING ERROR 1.2 Music\n";exit}
-}
-expect {
-	timeout {puts "TESTING ERROR 1.1 Pictures\n";exit}
-	"Pictures/_firejail_test_dir/a" {}
-	"Pictures/_firejail_test_dir/b" {puts "TESTING ERROR 1.2 Pictures\n";exit}
-}
-expect {
-	timeout {puts "TESTING ERROR 1.1 Projects\n";exit}
-	"Projects/_firejail_test_dir/a" {}
-	"Projects/_firejail_test_dir/b" {puts "TESTING ERROR 1.2 Projects\n";exit}
-}
-expect {
-	timeout {puts "TESTING ERROR 1.1 Videos\n";exit}
-	"Videos/_firejail_test_dir/a" {}
-	"Videos/_firejail_test_dir/b" {puts "TESTING ERROR 1.2 Videos\n";exit}
-}
-after 100
 
-# blacklist
-send -- "firejail --profile=./macro-subpath-blacklist.profile ls ~/Desktop/_firejail_test_dir; echo ret \$?\r"
-expect {
-	timeout {puts "TESTING ERROR 2 Desktop\n";exit}
-	-re "Child process initialized in \[0-9\]+.\[0-9\]+ ms"
+dict for {name path} $macros {
+	send -- "firejail --profile=./macro-subpath-readonly.profile touch $path/_firejail_test_dir/_firejail_test_file; echo ret \$?\r"
+	expect {
+		timeout {puts "TESTING ERROR 3 $name\n";exit}
+		-re "Child process initialized in \[0-9\]+.\[0-9\]+ ms"
+	}
+	expect {
+		timeout {puts "TESTING ERROR 3.1 $name\n";exit}
+		"Read-only file system" {}
+		-re {ret 0} {puts "TESTING ERROR 3.2 $name\n";exit}
+	}
+	after 100
 }
-expect {
-	timeout {puts "TESTING ERROR 2.1 Desktop\n";exit}
-	"Permission denied" {}
-	-re {ret 0} {puts "TESTING ERROR 2.2 Desktop\n";exit}
-}
-after 100
-
-send -- "firejail --profile=./macro-subpath-blacklist.profile ls ~/Documents/_firejail_test_dir; echo ret \$?\r"
-expect {
-	timeout {puts "TESTING ERROR 2 Documents\n";exit}
-	-re "Child process initialized in \[0-9\]+.\[0-9\]+ ms"
-}
-expect {
-	timeout {puts "TESTING ERROR 2.1 Documents\n";exit}
-	"Permission denied" {}
-	-re {ret 0} {puts "TESTING ERROR 2.2 Documents\n";exit}
-}
-after 100
-
-send -- "firejail --profile=./macro-subpath-blacklist.profile ls ~/Downloads/_firejail_test_dir; echo ret \$?\r"
-expect {
-	timeout {puts "TESTING ERROR 2 Downloads\n";exit}
-	-re "Child process initialized in \[0-9\]+.\[0-9\]+ ms"
-}
-expect {
-	timeout {puts "TESTING ERROR 2.1 Downloads\n";exit}
-	"Permission denied" {}
-	-re {ret 0} {puts "TESTING ERROR 2.2 Downloads\n";exit}
-}
-after 100
-
-send -- "firejail --profile=./macro-subpath-blacklist.profile ls ~/Music/_firejail_test_dir; echo ret \$?\r"
-expect {
-	timeout {puts "TESTING ERROR 2 Music\n";exit}
-	-re "Child process initialized in \[0-9\]+.\[0-9\]+ ms"
-}
-expect {
-	timeout {puts "TESTING ERROR 2.1 Music\n";exit}
-	"Permission denied" {}
-	-re {ret 0} {puts "TESTING ERROR 2.2 Music\n";exit}
-}
-after 100
-
-send -- "firejail --profile=./macro-subpath-blacklist.profile ls ~/Pictures/_firejail_test_dir; echo ret \$?\r"
-expect {
-	timeout {puts "TESTING ERROR 2 Pictures\n";exit}
-	-re "Child process initialized in \[0-9\]+.\[0-9\]+ ms"
-}
-expect {
-	timeout {puts "TESTING ERROR 2.1 Pictures\n";exit}
-	"Permission denied" {}
-	-re {ret 0} {puts "TESTING ERROR 2.2 Pictures\n";exit}
-}
-after 100
-
-send -- "firejail --profile=./macro-subpath-blacklist.profile ls ~/Projects/_firejail_test_dir; echo ret \$?\r"
-expect {
-	timeout {puts "TESTING ERROR 2 Projects\n";exit}
-	-re "Child process initialized in \[0-9\]+.\[0-9\]+ ms"
-}
-expect {
-	timeout {puts "TESTING ERROR 2.1 Projects\n";exit}
-	"Permission denied" {}
-	-re {ret 0} {puts "TESTING ERROR 2.2 Projects\n";exit}
-}
-after 100
-
-send -- "firejail --profile=./macro-subpath-blacklist.profile ls ~/Videos/_firejail_test_dir; echo ret \$?\r"
-expect {
-	timeout {puts "TESTING ERROR 2 Videos\n";exit}
-	-re "Child process initialized in \[0-9\]+.\[0-9\]+ ms"
-}
-expect {
-	timeout {puts "TESTING ERROR 2.1 Videos\n";exit}
-	"Permission denied" {}
-	-re {ret 0} {puts "TESTING ERROR 2.2 Videos\n";exit}
-}
-after 100
-
-# read-only
-send -- "firejail --profile=./macro-subpath-readonly.profile touch ~/Desktop/_firejail_test_dir/_firejail_test_file; echo ret \$?\r"
-expect {
-	timeout {puts "TESTING ERROR 3 Desktop\n";exit}
-	-re "Child process initialized in \[0-9\]+.\[0-9\]+ ms"
-}
-expect {
-	timeout {puts "TESTING ERROR 3.1 Desktop\n";exit}
-	"Read-only file system" {}
-	-re {ret 0} {puts "TESTING ERROR 3.2 Desktop\n";exit}
-}
-after 100
-
-send -- "firejail --profile=./macro-subpath-readonly.profile touch ~/Documents/_firejail_test_dir/_firejail_test_file; echo ret \$?\r"
-expect {
-	timeout {puts "TESTING ERROR 3 Documents\n";exit}
-	-re "Child process initialized in \[0-9\]+.\[0-9\]+ ms"
-}
-expect {
-	timeout {puts "TESTING ERROR 3.1 Documents\n";exit}
-	"Read-only file system" {}
-	-re {ret 0} {puts "TESTING ERROR 3.2 Documents\n";exit}
-}
-after 100
-
-send -- "firejail --profile=./macro-subpath-readonly.profile touch ~/Downloads/_firejail_test_dir/_firejail_test_file; echo ret \$?\r"
-expect {
-	timeout {puts "TESTING ERROR 3 Downloads\n";exit}
-	-re "Child process initialized in \[0-9\]+.\[0-9\]+ ms"
-}
-expect {
-	timeout {puts "TESTING ERROR 3.1 Downloads\n";exit}
-	"Read-only file system" {}
-	-re {ret 0} {puts "TESTING ERROR 3.2 Downloads\n";exit}
-}
-after 100
-
-send -- "firejail --profile=./macro-subpath-readonly.profile touch ~/Music/_firejail_test_dir/_firejail_test_file; echo ret \$?\r"
-expect {
-	timeout {puts "TESTING ERROR 3 Music\n";exit}
-	-re "Child process initialized in \[0-9\]+.\[0-9\]+ ms"
-}
-expect {
-	timeout {puts "TESTING ERROR 3.1 Music\n";exit}
-	"Read-only file system" {}
-	-re {ret 0} {puts "TESTING ERROR 3.2 Music\n";exit}
-}
-after 100
-
-send -- "firejail --profile=./macro-subpath-readonly.profile touch ~/Pictures/_firejail_test_dir/_firejail_test_file; echo ret \$?\r"
-expect {
-	timeout {puts "TESTING ERROR 3 Pictures\n";exit}
-	-re "Child process initialized in \[0-9\]+.\[0-9\]+ ms"
-}
-expect {
-	timeout {puts "TESTING ERROR 3.1 Pictures\n";exit}
-	"Read-only file system" {}
-	-re {ret 0} {puts "TESTING ERROR 3.2 Pictures\n";exit}
-}
-after 100
-
-send -- "firejail --profile=./macro-subpath-readonly.profile touch ~/Projects/_firejail_test_dir/_firejail_test_file; echo ret \$?\r"
-expect {
-	timeout {puts "TESTING ERROR 3 Projects\n";exit}
-	-re "Child process initialized in \[0-9\]+.\[0-9\]+ ms"
-}
-expect {
-	timeout {puts "TESTING ERROR 3.1 Projects\n";exit}
-	"Read-only file system" {}
-	-re {ret 0} {puts "TESTING ERROR 3.2 Projects\n";exit}
-}
-after 100
-
-send -- "firejail --profile=./macro-subpath-readonly.profile touch ~/Videos/_firejail_test_dir/_firejail_test_file; echo ret \$?\r"
-expect {
-	timeout {puts "TESTING ERROR 3 Videos\n";exit}
-	-re "Child process initialized in \[0-9\]+.\[0-9\]+ ms"
-}
-expect {
-	timeout {puts "TESTING ERROR 3.1 Videos\n";exit}
-	"Read-only file system" {}
-	-re {ret 0} {puts "TESTING ERROR 3.2 Videos\n";exit}
-}
-after 100
 
 puts "\nall done\n"
diff --git a/test/fs/macro.exp b/test/fs/macro.exp
index 48deefeb2..d42dc070e 100755
--- a/test/fs/macro.exp
+++ b/test/fs/macro.exp
@@ -7,210 +7,56 @@ set timeout 3
 spawn $env(SHELL)
 match_max 100000
 
+set macros [ dict create \
+	"DESKTOP"   "$::env(HOME)/Desktop" \
+	"DOCUMENTS" "$::env(HOME)/Documents" \
+	"DOWNLOADS" "$::env(HOME)/Downloads" \
+	"MUSIC"     "$::env(HOME)/Music" \
+	"PICTURES"  "$::env(HOME)/Pictures" \
+	"PROJECTS"  "$::env(HOME)/Projects" \
+	"VIDEOS"    "$::env(HOME)/Videos" \
+]
 
-send -- "firejail --profile=./macro-whitelist.profile ls ~\r"
-expect {
-	timeout {puts "TESTING ERROR 1\n";exit}
-	-re "Child process initialized in \[0-9\]+.\[0-9\]+ ms"
+# Test that macros work.
+dict for {name path} $macros {
+	send -- "firejail --profile=./macro-whitelist.profile find $path -maxdepth 1 | LC_ALL=C sort\r"
+	expect {
+		timeout {puts "TESTING ERROR 1\n";exit}
+		-re "Child process initialized in \[0-9\]+.\[0-9\]+ ms"
+	}
+	expect {
+		timeout {puts "TESTING ERROR 1.1 $name\n";exit}
+		"$path"
+	}
+	after 100
 }
-expect {
-	timeout {puts "TESTING ERROR 1.1 Desktop\n";exit}
-	"Desktop"
-}
-expect {
-	timeout {puts "TESTING ERROR 1.1 Documents\n";exit}
-	"Documents"
-}
-expect {
-	timeout {puts "TESTING ERROR 1.1 Downloads\n";exit}
-	"Downloads"
-}
-expect {
-	timeout {puts "TESTING ERROR 1.1 Music\n";exit}
-	"Music"
-}
-expect {
-	timeout {puts "TESTING ERROR 1.1 Pictures\n";exit}
-	"Pictures"
-}
-expect {
-	timeout {puts "TESTING ERROR 1.1 Projects\n";exit}
-	"Projects"
-}
-expect {
-	timeout {puts "TESTING ERROR 1.1 Videos\n";exit}
-	"Videos"
-}
-after 100
 
-# blacklist
-send -- "firejail --profile=./macro-blacklist.profile ls ~/Desktop; echo ret \$?\r"
-expect {
-	timeout {puts "TESTING ERROR 2 Desktop\n";exit}
-	-re "Child process initialized in \[0-9\]+.\[0-9\]+ ms"
+dict for {name path} $macros {
+	send -- "firejail --profile=./macro-blacklist.profile ls $path; echo ret \$?\r"
+	expect {
+		timeout {puts "TESTING ERROR 2 $name\n";exit}
+		-re "Child process initialized in \[0-9\]+.\[0-9\]+ ms"
+	}
+	expect {
+		timeout {puts "TESTING ERROR 2.1 $name\n";exit}
+		"Permission denied" {}
+		-re {ret 0} {puts "TESTING ERROR 2.2 $name\n";exit}
+	}
+	after 100
 }
-expect {
-	timeout {puts "TESTING ERROR 2.1 Desktop\n";exit}
-	"Permission denied" {}
-	-re {ret 0} {puts "TESTING ERROR 2.2 Desktop\n";exit}
-}
-after 100
 
-send -- "firejail --profile=./macro-blacklist.profile ls ~/Documents; echo ret \$?\r"
-expect {
-	timeout {puts "TESTING ERROR 2 Documents\n";exit}
-	-re "Child process initialized in \[0-9\]+.\[0-9\]+ ms"
+dict for {name path} $macros {
+	send -- "firejail --profile=./macro-readonly.profile touch $path/_firejail_test_file; echo ret \$?\r"
+	expect {
+		timeout {puts "TESTING ERROR 3 $name\n";exit}
+		-re "Child process initialized in \[0-9\]+.\[0-9\]+ ms"
+	}
+	expect {
+		timeout {puts "TESTING ERROR 3.1 $name\n";exit}
+		"Read-only file system" {}
+		-re {ret 0} {puts "TESTING ERROR 3.2 $name\n";exit}
+	}
+	after 100
 }
-expect {
-	timeout {puts "TESTING ERROR 2.1 Documents\n";exit}
-	"Permission denied" {}
-	-re {ret 0} {puts "TESTING ERROR 2.2 Documents\n";exit}
-}
-after 100
-
-send -- "firejail --profile=./macro-blacklist.profile ls ~/Downloads; echo ret \$?\r"
-expect {
-	timeout {puts "TESTING ERROR 2 Downloads\n";exit}
-	-re "Child process initialized in \[0-9\]+.\[0-9\]+ ms"
-}
-expect {
-	timeout {puts "TESTING ERROR 2.1 Downloads\n";exit}
-	"Permission denied" {}
-	-re {ret 0} {puts "TESTING ERROR 2.2 Downloads\n";exit}
-}
-after 100
-
-send -- "firejail --profile=./macro-blacklist.profile ls ~/Music; echo ret \$?\r"
-expect {
-	timeout {puts "TESTING ERROR 2 Music\n";exit}
-	-re "Child process initialized in \[0-9\]+.\[0-9\]+ ms"
-}
-expect {
-	timeout {puts "TESTING ERROR 2.1 Music\n";exit}
-	"Permission denied" {}
-	-re {ret 0} {puts "TESTING ERROR 2.2 Music\n";exit}
-}
-after 100
-
-send -- "firejail --profile=./macro-blacklist.profile ls ~/Pictures; echo ret \$?\r"
-expect {
-	timeout {puts "TESTING ERROR 2 Pictures\n";exit}
-	-re "Child process initialized in \[0-9\]+.\[0-9\]+ ms"
-}
-expect {
-	timeout {puts "TESTING ERROR 2.1 Pictures\n";exit}
-	"Permission denied" {}
-	-re {ret 0} {puts "TESTING ERROR 2.2 Pictures\n";exit}
-}
-after 100
-
-send -- "firejail --profile=./macro-blacklist.profile ls ~/Projects; echo ret \$?\r"
-expect {
-	timeout {puts "TESTING ERROR 2 Projects\n";exit}
-	-re "Child process initialized in \[0-9\]+.\[0-9\]+ ms"
-}
-expect {
-	timeout {puts "TESTING ERROR 2.1 Projects\n";exit}
-	"Permission denied" {}
-	-re {ret 0} {puts "TESTING ERROR 2.2 Projects\n";exit}
-}
-after 100
-
-send -- "firejail --profile=./macro-blacklist.profile ls ~/Videos; echo ret \$?\r"
-expect {
-	timeout {puts "TESTING ERROR 2 Videos\n";exit}
-	-re "Child process initialized in \[0-9\]+.\[0-9\]+ ms"
-}
-expect {
-	timeout {puts "TESTING ERROR 2.1 Videos\n";exit}
-	"Permission denied" {}
-	-re {ret 0} {puts "TESTING ERROR 2.2 Videos\n";exit}
-}
-after 100
-
-# read-only
-send -- "firejail --profile=./macro-readonly.profile touch ~/Desktop/_firejail_test_file; echo ret \$?\r"
-expect {
-	timeout {puts "TESTING ERROR 3 Desktop\n";exit}
-	-re "Child process initialized in \[0-9\]+.\[0-9\]+ ms"
-}
-expect {
-	timeout {puts "TESTING ERROR 3.1 Desktop\n";exit}
-	"Read-only file system" {}
-	-re {ret 0} {puts "TESTING ERROR 3.2 Desktop\n";exit}
-}
-after 100
-
-send -- "firejail --profile=./macro-readonly.profile touch ~/Documents/_firejail_test_file; echo ret \$?\r"
-expect {
-	timeout {puts "TESTING ERROR 3 Documents\n";exit}
-	-re "Child process initialized in \[0-9\]+.\[0-9\]+ ms"
-}
-expect {
-	timeout {puts "TESTING ERROR 3.1 Documents\n";exit}
-	"Read-only file system" {}
-	-re {ret 0} {puts "TESTING ERROR 3.2 Documents\n";exit}
-}
-after 100
-
-send -- "firejail --profile=./macro-readonly.profile touch ~/Downloads/_firejail_test_file; echo ret \$?\r"
-expect {
-	timeout {puts "TESTING ERROR 3 Downloads\n";exit}
-	-re "Child process initialized in \[0-9\]+.\[0-9\]+ ms"
-}
-expect {
-	timeout {puts "TESTING ERROR 3.1 Downloads\n";exit}
-	"Read-only file system" {}
-	-re {ret 0} {puts "TESTING ERROR 3.2 Downloads\n";exit}
-}
-after 100
-
-send -- "firejail --profile=./macro-readonly.profile touch ~/Music/_firejail_test_file; echo ret \$?\r"
-expect {
-	timeout {puts "TESTING ERROR 3 Music\n";exit}
-	-re "Child process initialized in \[0-9\]+.\[0-9\]+ ms"
-}
-expect {
-	timeout {puts "TESTING ERROR 3.1 Music\n";exit}
-	"Read-only file system" {}
-	-re {ret 0} {puts "TESTING ERROR 3.2 Music\n";exit}
-}
-after 100
-
-send -- "firejail --profile=./macro-readonly.profile touch ~/Pictures/_firejail_test_file; echo ret \$?\r"
-expect {
-	timeout {puts "TESTING ERROR 3 Pictures\n";exit}
-	-re "Child process initialized in \[0-9\]+.\[0-9\]+ ms"
-}
-expect {
-	timeout {puts "TESTING ERROR 3.1 Pictures\n";exit}
-	"Read-only file system" {}
-	-re {ret 0} {puts "TESTING ERROR 3.2 Pictures\n";exit}
-}
-after 100
-
-send -- "firejail --profile=./macro-readonly.profile touch ~/Projects/_firejail_test_file; echo ret \$?\r"
-expect {
-	timeout {puts "TESTING ERROR 3 Projects\n";exit}
-	-re "Child process initialized in \[0-9\]+.\[0-9\]+ ms"
-}
-expect {
-	timeout {puts "TESTING ERROR 3.1 Projects\n";exit}
-	"Read-only file system" {}
-	-re {ret 0} {puts "TESTING ERROR 3.2 Projects\n";exit}
-}
-after 100
-
-send -- "firejail --profile=./macro-readonly.profile touch ~/Videos/_firejail_test_file; echo ret \$?\r"
-expect {
-	timeout {puts "TESTING ERROR 3 Videos\n";exit}
-	-re "Child process initialized in \[0-9\]+.\[0-9\]+ ms"
-}
-expect {
-	timeout {puts "TESTING ERROR 3.1 Videos\n";exit}
-	"Read-only file system" {}
-	-re {ret 0} {puts "TESTING ERROR 3.2 Videos\n";exit}
-}
-after 100
 
 puts "\nall done\n"