Simplify and fix tests

ThomasFarstrike · ThomasFarstrike · commit c3e0a1ceca37 · 2025-11-23T05:46:21.000+01:00
diff --git a/scripts/run_desktop.sh b/scripts/run_desktop.sh
@@ -61,9 +61,9 @@ pushd internal_filesystem/
 	elif [ ! -z "$script" ]; then # it's an app name
 		scriptdir="$script"
 		echo "Running app from $scriptdir"
-		"$binary" -X heapsize=$HEAPSIZE  -v -i -c "import sys ; sys.path.append('lib/') ; import mpos.main ; import mpos.apps; mpos.apps.start_app('$scriptdir')"
+		"$binary" -X heapsize=$HEAPSIZE  -v -i -c "$(cat boot.py) ; import mpos.apps; mpos.apps.start_app('$scriptdir')"
 	else
-		"$binary" -X heapsize=$HEAPSIZE -v -i -c "import sys ; sys.path.append('lib/') ; import mpos.main"
+		"$binary" -X heapsize=$HEAPSIZE -v -i -c "$(cat boot.py)"
 	fi
 		
 
diff --git a/tests/test_connectivity_manager.py b/tests/test_connectivity_manager.py
@@ -634,5 +634,3 @@ def test_multiple_apps_using_connectivity_manager(self):
         self.assertFalse(app3_state[0])
 
 
-if __name__ == '__main__':
-    unittest.main()
diff --git a/tests/test_graphical_abc_button_debug.py b/tests/test_graphical_abc_button_debug.py
@@ -103,5 +103,3 @@ def test_simulate_abc_button_click(self):
         print("="*70)
 
 
-if __name__ == "__main__":
-    unittest.main()
diff --git a/tests/test_graphical_keyboard_crash_reproduction.py b/tests/test_graphical_keyboard_crash_reproduction.py
@@ -145,5 +145,3 @@ def test_multiple_keyboards(self):
         print("SUCCESS: Multiple keyboards work")
 
 
-if __name__ == "__main__":
-    unittest.main()
diff --git a/tests/test_graphical_keyboard_default_vs_custom.py b/tests/test_graphical_keyboard_default_vs_custom.py
@@ -185,5 +185,3 @@ def _get_special_labels(self, keyboard):
         return labels
 
 
-if __name__ == "__main__":
-    unittest.main()
diff --git a/tests/test_graphical_keyboard_layout_switching.py b/tests/test_graphical_keyboard_layout_switching.py
@@ -284,5 +284,3 @@ def test_event_handler_switches_layout(self):
         print("\nSUCCESS: Event handler preserves custom layout!")
 
 
-if __name__ == "__main__":
-    unittest.main()
diff --git a/tests/test_graphical_keyboard_mode_switch.py b/tests/test_graphical_keyboard_mode_switch.py
@@ -153,5 +153,3 @@ def test_event_handler_exists(self):
         print("Note: The handler filters for VALUE_CHANGED events only")
 
 
-if __name__ == "__main__":
-    unittest.main()
diff --git a/tests/test_graphical_keyboard_q_button_bug.py b/tests/test_graphical_keyboard_q_button_bug.py
@@ -225,6 +225,3 @@ def test_keyboard_button_discovery(self):
         self.assertTrue(len(found_buttons) > 0,
                        "Should find at least some buttons on keyboard")
 
-
-if __name__ == "__main__":
-    unittest.main()
diff --git a/tests/test_graphical_keyboard_rapid_mode_switch.py b/tests/test_graphical_keyboard_rapid_mode_switch.py
@@ -150,5 +150,3 @@ def test_button_indices_after_mode_switch(self):
             except:
                 pass
 
-if __name__ == "__main__":
-    unittest.main()
diff --git a/tests/test_graphical_launch_all_apps.py b/tests/test_graphical_launch_all_apps.py
@@ -246,5 +246,3 @@ def test_about_app_loads(self):
             f"About app should load without errors: {error_msg}")
 
 
-if __name__ == '__main__':
-    unittest.main()
diff --git a/tests/test_graphical_wifi_keyboard.py b/tests/test_graphical_wifi_keyboard.py
@@ -229,5 +229,3 @@ def test_set_textarea_stores_reference(self):
         print("This prevents LVGL auto-insertion and fixes double-character bug!")
 
 
-if __name__ == "__main__":
-    unittest.main()
diff --git a/tests/test_wifi_service.py b/tests/test_wifi_service.py
@@ -455,5 +455,3 @@ def test_disconnect_desktop_mode(self):
         WifiService.disconnect(network_module=None)
 
 
-if __name__ == '__main__':
-    unittest.main()
diff --git a/tests/unittest.sh b/tests/unittest.sh
@@ -60,16 +60,17 @@ one_test() {
 		# Desktop execution
 		if [ $is_graphical -eq 1 ]; then
 			# Graphical test: include boot_unix.py and main.py
-			"$binary" -X heapsize=8M -c "import sys ; sys.path.append('lib/') ; import mpos.main ; import mpos.apps; sys.path.append(\"$tests_abs_path\")
+			"$binary" -X heapsize=8M -c "$(cat boot.py) ; import mpos.main ; import mpos.apps; sys.path.append(\"$tests_abs_path\")
 $(cat $file)
 result = unittest.main() ; sys.exit(0 if result.wasSuccessful() else 1) "
+            result=$?
 		else
 			# Regular test: no boot files
-			"$binary" -X heapsize=8M -c "import sys ; sys.path.append('lib/')
+			"$binary" -X heapsize=8M -c "$(cat boot.py)
 $(cat $file)
 result = unittest.main() ; sys.exit(0 if result.wasSuccessful() else 1) "
+            result=$?
 		fi
-		result=$?
 	else
 		if [ ! -z "$ondevice" ]; then
 			echo "Hack: reset the device to make sure no previous UnitTest classes have been registered..."
@@ -85,7 +86,7 @@ result = unittest.main() ; sys.exit(0 if result.wasSuccessful() else 1) "
 		echo "$test logging to $testlog"
 		if [ $is_graphical -eq 1 ]; then
 			# Graphical test: system already initialized, just add test paths
-			"$mpremote" exec "import sys ; sys.path.append('lib') ; sys.path.append('tests')
+			"$mpremote" exec "$(cat boot.py) ; sys.path.append('tests')
 $(cat $file)
 result = unittest.main()
 if result.wasSuccessful():
@@ -95,7 +96,7 @@ else:
 " | tee "$testlog"
 		else
 			# Regular test: no boot files
-			"$mpremote" exec "import sys ; sys.path.append('lib')
+			"$mpremote" exec "$(cat boot.py)
 $(cat $file)
 result = unittest.main()
 if result.wasSuccessful():

Original file line number	Diff line number	Diff line change
`@@ -634,5 +634,3 @@ def test_multiple_apps_using_connectivity_manager(self):`
`634`	`634`	`self.assertFalse(app3_state[0])`
`635`	`635`
`636`	`636`
`637`		`-if __name__ == '__main__':`
`638`		`- unittest.main()`
Original file line number	Diff line number	Diff line change
`@@ -103,5 +103,3 @@ def test_simulate_abc_button_click(self):`
`103`	`103`	`print("="*70)`
`104`	`104`
`105`	`105`
`106`		`-if __name__ == "__main__":`
`107`		`- unittest.main()`
Original file line number	Diff line number	Diff line change
`@@ -145,5 +145,3 @@ def test_multiple_keyboards(self):`
`145`	`145`	`print("SUCCESS: Multiple keyboards work")`
`146`	`146`
`147`	`147`
`148`		`-if __name__ == "__main__":`
`149`		`- unittest.main()`
Original file line number	Diff line number	Diff line change
`@@ -185,5 +185,3 @@ def _get_special_labels(self, keyboard):`
`185`	`185`	`return labels`
`186`	`186`
`187`	`187`
`188`		`-if __name__ == "__main__":`
`189`		`- unittest.main()`
Original file line number	Diff line number	Diff line change
`@@ -284,5 +284,3 @@ def test_event_handler_switches_layout(self):`
`284`	`284`	`print("\nSUCCESS: Event handler preserves custom layout!")`
`285`	`285`
`286`	`286`
`287`		`-if __name__ == "__main__":`
`288`		`- unittest.main()`
Original file line number	Diff line number	Diff line change
`@@ -153,5 +153,3 @@ def test_event_handler_exists(self):`
`153`	`153`	`print("Note: The handler filters for VALUE_CHANGED events only")`
`154`	`154`
`155`	`155`
`156`		`-if __name__ == "__main__":`
`157`		`- unittest.main()`
Original file line number	Diff line number	Diff line change
`@@ -246,5 +246,3 @@ def test_about_app_loads(self):`
`246`	`246`	`f"About app should load without errors: {error_msg}")`
`247`	`247`
`248`	`248`
`249`		`-if __name__ == '__main__':`
`250`		`- unittest.main()`