From 0d2afb6f709c899d3e53c6ec14a6e49c09caef55 Mon Sep 17 00:00:00 2001
From: Thomas Farstrike <thomasverstreken@protonmail.com>
Date: Sun, 2 Nov 2025 13:28:29 +0100
Subject: [PATCH] PackageManager: add unit tests

---
 .gitignore                                    |   1 +
 internal_filesystem/lib/README.md             |   1 +
 .../lib/mpos/content/package_manager.py       |   8 ++-
 internal_filesystem/lib/unittest/__init__.mpy | Bin 0 -> 5853 bytes
 tests/package_manager.py                      |  37 ++++++++++
 tests/simple.py                               |   6 ++
 tests/tests/check_syntax.sh                   |  20 ++++++
 tests/unittest.sh                             |  63 ++++++++++++++++++
 8 files changed, 134 insertions(+), 2 deletions(-)
 create mode 100644 internal_filesystem/lib/unittest/__init__.mpy
 create mode 100644 tests/package_manager.py
 create mode 100644 tests/simple.py
 create mode 100755 tests/tests/check_syntax.sh
 create mode 100755 tests/unittest.sh

diff --git a/.gitignore b/.gitignore
index 45013b9e..cab0b726 100644
--- a/.gitignore
+++ b/.gitignore
@@ -4,6 +4,7 @@ conf.json*
 
 # auto created when running on desktop:
 internal_filesystem/SDLPointer_2
+internal_filesystem/SDLPointer_3
 
 # config files etc:
 internal_filesystem/data
diff --git a/internal_filesystem/lib/README.md b/internal_filesystem/lib/README.md
index 1c149121..fe3eabff 100644
--- a/internal_filesystem/lib/README.md
+++ b/internal_filesystem/lib/README.md
@@ -7,4 +7,5 @@ This /lib folder contains:
 - mip.install("aiohttp") # easy websockets
 - mip.install("base64") # for nostr etc
 - mip.install("collections") # used by aiohttp
+- mip.install("unittest")
 
diff --git a/internal_filesystem/lib/mpos/content/package_manager.py b/internal_filesystem/lib/mpos/content/package_manager.py
index 114b5c19..60aa365e 100644
--- a/internal_filesystem/lib/mpos/content/package_manager.py
+++ b/internal_filesystem/lib/mpos/content/package_manager.py
@@ -166,8 +166,12 @@ class PackageManager:
         """Compare two version numbers (e.g., '1.2.3' vs '4.5.6').
         Returns True if ver1 is greater than ver2, False otherwise."""
         print(f"Comparing versions: {ver1} vs {ver2}")
-        v1_parts = [int(x) for x in ver1.split('.')]
-        v2_parts = [int(x) for x in ver2.split('.')]
+        try:
+            v1_parts = [int(x) for x in ver1.split('.')]
+            v2_parts = [int(x) for x in ver2.split('.')]
+        except ValueError as e:
+            print(f"Invalid input, got error: {e}")
+            return False
         print(f"Version 1 parts: {v1_parts}")
         print(f"Version 2 parts: {v2_parts}")
         for i in range(max(len(v1_parts), len(v2_parts))):
diff --git a/internal_filesystem/lib/unittest/__init__.mpy b/internal_filesystem/lib/unittest/__init__.mpy
new file mode 100644
index 0000000000000000000000000000000000000000..ce71760c6ae6314e0db15e51d9c1631170cda7bf
GIT binary patch
literal 5853
zcmeZeV~}rDm((cD%Pc8LEiTcIkI&4@EQycTE2w1PXUWWGV976LU@NXHW)LbVN=!~o
zN=(jX5D3oBEC@+0E@4n}EG|whDhW!=EKV(U&d)1JtterT2`)_n83yBv`IYA6z&HXR
zea?x+sSH9OR&Z%%Nh*T~h#6FxmzP=uWv3RG=9DlnYx5ZB85-!BFmM!SXBIH<fJo0Y
z1`!Y)nwOJWT+ASsT2YXiT#}mNmYA7ST9nEl5ucn`P*Pfy8lPH`%pee4Qk0pO?&;4U
zS5TChR{|1DEhx#%&tu?7%gjk-5J*ogDND>LO=ZxD&q&Nm$w`d|IUY$}HQ2g>)D*wc
zTn0Xn)di_343$C(P^J}wSQ^CcVvq^}7>9vN!OhXrhe0T{s3^Y(EXM<3GVmz41_k*C
zF-XMcrIte-Ud$j8UsReGUknZswxZHJ29e^_lF$O@oW$Z{1}-q0L7*fxvB)LAJdc4%
zkHM6$s5B20b_`AG93WRS=r<-y#OLHEro`vwr<CTTGAIkc!iPapS}3(58R~g^w%p=$
z2A+b%qQu-{1}=r@>e^U_MoktS1%}4JMqe%kjp|xW29d;+6z81O#Jti12BG+52&<Su
zBqbjzTFlT`$X8sN1a^b~C_un04p0~{h$VtkhihSJVh)20gyWZA0^!R+_&%w{#Sj4%
zh=6-hYGO%h5kyQ0BIcNrn_mporvVZ2%P&C|6@Z9(7Q-y{EcVMUVGvL)QphY;P%UDR
zfGP0H&r4;1S`JnO77&3bgs>q>LyAgMVWzny<`kzgz%0pQP=F}$EcVPRE=kNwPKB8d
z$q6uR;fY0g#SA?0DXGc%MGOKqIf=PRDT#IrQlPksPpv432Zs|6IK>q+aO5TCrZVs)
zrlf!aQ4E}5UBQV56z|0)#X+Tc3@rZM3{vHZ#lfY?$*IM~X{9*~G6<DEnZ+dxoC2VP
zXQfbG%V5qIAD@_#5+Bdds0vCg@gVOv%5s3CmrWr*n?WMEw5TXGuLSG}1~IVHiXiz<
zAhjYnJ~J;ZpFzkbJvA@2qM*o*fvdXKwz`&qCq5ILUwu6l;^UExh>vHGiw7GP4>2Y_
zo<SxgwW0*%DQG4ai;pi$Eh^5;&x?;|(B(+Z&r4xo17%k!kd26vrqP(MEVU>pzc`hl
z(YdjlDVae8oIT>fA=PBtsPEXM-zeG0&I3*EA|U5OEfI;2&&(~zFDi+TXJ`@z`=z9^
zAe8}>3*sSJo~0y-q0y5gtu!y0fu|@nu{b}EfvJLlsgi-GASW?7wU~h`B{iocks-IZ
ziI<~1u_%v$n=P$0kAW>Yrx>IITuia$=BF?;W^;j^#=w?ST*APWUy_kp1dV8r@!%4N
zl^+~YNM#EvFDySpaycunYLP--eu+X6s7Os=U=>y^QYb4{(17J)O;BQHm4m8+i7J%j
zD<q{Vq=Ir9Sb-EIJ)_Cn*y542gGeeA7o;X<rd29{J+6>nqyPyCu&a{tOEMT(B~*(P
z6hW#L$}>wcGV>Hviy*2QSS7JYLG(kk+2YoQE(O!Z1MvzdP*jT;SosiKaBgK4bA?s_
zAZI|r0i1PN#W5uy`52oF*cS|}LZzTo4z5oWiXj<{fz?fyDu98NHz+YrK{Z7Ilt_!Y
z7+B>&+0@lVK?76_loq8H+p4DMC_w56Th$az23B4L4N#$4keUMFyV+6&FtA$M!0LQG
zNG{a_XHRgxS12mYvsFz&I79)-eRd42+^WS28mh&b3VM2a480{fA&hEjZc6@X*$SP)
z%E_G)ogAIrE!-&yDG4n+onoEBDV+u_oE?&#Azi!;fznK3Obvn3%wo(9fzm8uEFK&U
zfzqsEtX{m~f~EIe+#&<TSiKlyLyQ;%*aXW=n8nyaj2H#jO_;>k1I5b&#VbOLm;^XX
zn8Y|jjF<&DO_;<uLyTAixJ;PDxQtlE1dZ6lgpAn5gpD}FL_&->1-MO^#JEF@xCD4i
zn8bJldBiG1jJO4OO_;=Zjd;XFgR~ffjd{hS1o%J<J~4Fx{y;6JU}JtUBLM-is=#D{
z3I;Z|4sJn1v1$P}G2OuA#vBGN_747(6tEDxn643nn28ain3)lin0a6_bAc3(Nh4ov
zgTTfO5~hL49-LfejeHHl8<LwNHn1p}H}W;|HwrWgHi)UHYJ00FZQ3AVu`%$Ti?@)a
zGSdcO(;p^`sX~oH8#f4>nlQPzfuw_y1CpC$ICwZZ)G`$y)^dpH27=v*YImc-;s!<m
zlSV!(>jrg|jT<DaHf@lwhC4z#Fu6_SU_*n0jCmtpgPx~?vZ|h{v5(irhS0!!mtBrD
z2$dh&AZ+F0w@JW8#CC(Ql^LTJ<I_VMgsolT1?)uZjTklvTZ=fDF~pfLnlT>UB;Y7w
zyBVTJNNST%#l`>=mdK!F#wLb~4qQczvJIxe8-jzq1q_%D9ug=PvE8`Ij43_RNiQ(D
z(S?D7v%?PJ8BTB*ni(;I!jB0WVD^E@6$@BIpy9>Z;1rnL$0HEH)!~+s;*gS%k&>C|
zkm-=&km1lR+AR!XBjFBSaO7}-4F*MylM$1cOCTtUnZ?|USj5~R@y#mc0b#I-c|sWM
zVqRcn9Ae&Jnp4atP=GH`i#gbsOUxI><QDUTDCZILhcI}>0w4@Nu^<S8Uo05H5D*K6
zFa*WIAPga~a0o+KECOtch*+eNs902BvX}=amq{aEgL`0dQ^cW$0}et?jeL#54PKh6
zLiHO2oK2XVU9}lDNH}j0b{3J_xM{N%vl(M#U~)5uqku~zUxUA<k`7d%ON>(}NR_J@
zg8}owLx(qR3f|mk+Qb;SLD<<@Eik#!U{|99k6R<3RYRx{hk(^a0qadBEey#OQhe@>
ze2pRv5khPmO&C)fIc72Pcz~2e3t55$0+X98Hn2)~Hu5!!Hi}s_#3^fRcJ@$a-ndDN
z>Br%Xn{GP030ZD7VN8W8^nxi&hA3o#DZGQE&|?Fugf~oKhN{Nqz<Vwp%B(P@x6qXO
zz?9~~l(NB;-bN_x$T-!?Xypr0+bC|;DA7=)s-)wgrlX|nt*T_Cr0ltI(@htrz~rr<
z^zXMBM0yA{N-DFPFs6dVVuTu{Hg5WC!npMoNZl4^Pi2mco3?5(|2X254sxVZ`(dF*
zX`x1`O&dR(Fm5(sY5*GuQpgF8pC3nrEVqC|qod$dE2EhovUA#XJk$)qjxkqK_5|CA
z%@v8DQ1yd5<p#_tTrj8PK%D|o7lP)VJ22JUXzl?=qd&wsRt>WtvBLuk$9sryY;xGZ
zDiHuvI3KJ~AP}zbGLlMx#f&0BAeB}POO#bNIlC(JZv1g*V<6bHP2gZ?l32_r5)4tZ
z60Alb1fm9P5L6MvVn&frh@y31Maq02MIft?RD?lPY=)@d2dQvz6S9P=0439KkP1*T
z-2ryCvH;B4cR@x$l4)bX23C;>n8Lk6jf@*NnJ^*bTQzPsA8?S1Y~-tL6mB@6>ZTN;
zspPdO@ScmC5c@_G#$*rfO`C(afOD6ZT)0r#eV6c~p1eYhj9WLFFoA-9Yii(84c1sM
z##m4>zY$c(2PPK?@kKTAHOe%c72-5uNN!{i<Q24PxY)|hut9-=@!-Z}#zr0~f#^or
zM!AN|sUWdKo01tD8Kn418f6>#8m<VjsWO0asE!n$P>of?9cASW&dx>*$wGo4M=^SE
za>X=?Hr(Bi+^8ePCtPFI@JLyCgA1mL#~Z+wix@P#Xyt%g-sG^i;edlcY@=+$OC?`t
z4<1#9jlr8Bo@9yi;sAL?5E7$490J@OUosU^QXIeqFSnR3q=17|PYFg$Vj4!wV#a~V
zER7no82L;Z`QjVCs)9<gxQ!bm;scW#6_ofA8u=RK8-8vOjuVmF2rkx|7=nZ}8u=O(
z8vdy3Y!Hr%_vSq$q`qn62H|)U=13>Qjmd2uK~lzze2t0|7_~Iy6*Uco8aX$bFvffP
z9};S014ZLup+?5dLXAo$OfGQ{CaVdPOR!L*&}Jbn6DAkW4I&Af%@`w{qBe=>3mBL&
zx&#U}a&0zYM94Tet*;MEZj@nQ<C(z)_8<?qX3+>tW^A{xQUN(pc>=epyPAfMUa+#d
zrn?ZY3B#d{A{v{_7$Y}{B$_cq3N<n*3z;y2Y2i)6Nk2>&BR2^vG~94@-z2Qua0BEb
zCS{RL!U~&&m9>~a(oRtV$%i)y%QW0@@!YH|x=C2($J4_?jZB+CT9cbu7$kUSD70{;
zq$IR(ws3&m#S3;9sK!e%ViHR<Virp^ViC(QVin5_OlE7A5R@=!<eQ+VqN1Xrs;b(s
zK|;A<gM>oE1__yl4H8M9_)q~6iXcL^VS@yyo=_0vN@?VqpuIs@x#6Z8>jq(khMUPv
z1(E`ZjeHaIwA3^=2&*`|ZV*;<i4baJGGU0^Ad<R4SjCJH!cYXqWJ^hwRB9t%qsjym
zWer^oU3F!lLmT70%?y|h9}!3cH7z!S8x==E;UoqLCxLVk+pT7d5Y8qDwZ()n(upH5
znXxfqC%BC<!3^BS;65nS$g$CcF+I}BI54@<Ls1~3k#B;XlCrY0_y*ym4Z_MBgcY=y
ze<Zh7D9L0t@-?bXaMgx@4dKGd8-<mhMoB1d2p3k^D6H@_N<v{nxUkGdVVS2<5;7aY
zg_AZ4Cq0dlNZKH*B9gTc;yGbO5EJCR4Z^Y@rYy+I$t@aQBBHf*69Uw=wABrSm_)?P
z7!C>53$Z9Om@v6`3$ciZZ`fqU6zLQ$#0si!0)$va!~-{M6p+|t*2Em?<giIZ(u^Uw
zTjB<*iBx_;VI#wYXcZN8Ej3LYO(iW&9c3>~9c7J8jiE<2HH97(65X=Zgz1=2BjXk!
zHWMb7v@OEY4WY+3h6~Fyg517QSXM-C!*(quGjP%qkl${?6d9NdR&Qy-<dU>8Tv!35
zQemU8qKMpvZD5s~T|9&unFN$Tft|21Tv!>TLV2UG3c3na6UNAm;lgSld9}dgNhRwA
zFE%o&H!>GAvP>w_)^yVGP|?xR)m2ssbMBT`QPR@XaMIDz(ok~ORC9M_U}WOp<l+`;
zWK?Ak;s%9|DualIDkC_>Ktvlsnbkv;!GuAb!^uUJL7hXWkxi9DK+}l9g!wR-t;}e|
zXu^C1%oflxVlrV?=Y(*yjhIcC)w#48j(YH^G70FYG6?H_daBOk#T_ox$bR3&Ins+U
zcEiR^J9h5ct-!*5Y_kbNq-Tmyy())*9;A%mP~|XTROjBN$|0<00%~XpHFAQ4;=R??
zxmCHL1x6&;I#q5HrsG15T-!|;ot>QIwrvN6p14pW7f6GPk1B_-A=o64Ia^E^B4f8E
zvsy5)#Z0IQOa{}{P`U<6*9Io{F|3)y5|rI2ivraqv?`dWs%WaKsA{>Wc?GM;r#dJr
zt11P8GLM8Rn<|qQ(+`0h0X~x^#z<8b6NX4tM)3-PToGGUCNstps%$&7n8eFH6FhVq
zR2e1IR2jpClRz<;v>mKeSk1)?9{u2ikY~ae>16<}*fLLg$_q&e<g2o&GH5Xg6sR%^
z6bd!6?lfWC8K}i<(#8-O7<f{H6I9g&28ve%ikGXhnlMCqaH=wkxR^0SqJ({HU~;R&
zss=_mqekvVo(acP)wSKU)iu;X9U^xj9utOxPJy7>SO}a9+%|#)9E2JfHwZOym@v7x
f3pMg?Fkw6(#Jxe-(1h{OCK2NeW{i=LayJ<O7udQ_

literal 0
HcmV?d00001

diff --git a/tests/package_manager.py b/tests/package_manager.py
new file mode 100644
index 00000000..6204e994
--- /dev/null
+++ b/tests/package_manager.py
@@ -0,0 +1,37 @@
+import unittest
+
+from mpos import PackageManager
+
+class TestCompareVersions(unittest.TestCase):
+
+    def test_lower_short(self):
+        self.assertFalse(PackageManager.compare_versions("1" , "4"))
+
+    def test_lower(self):
+        self.assertFalse(PackageManager.compare_versions("1.2.3" , "4.5.6"))
+
+    def test_equal(self):
+        self.assertFalse(PackageManager.compare_versions("1.2.3" , "1.2.3"))
+
+    def test_higher(self):
+        self.assertTrue(PackageManager.compare_versions("4.5.6", "1.2.3"))
+    
+    def test_higher_medium_and_long(self):
+        self.assertTrue(PackageManager.compare_versions("4.5", "1.2.3"))
+    
+    def test_words(self):
+        self.assertFalse(PackageManager.compare_versions("weird" , "input"))
+
+    def test_one_empty(self):
+        self.assertFalse(PackageManager.compare_versions("1.2.3" , ""))
+
+class TestPackageManager_is_installed_by_name(unittest.TestCase):
+
+    def test_installed_builtin(self):
+        self.assertTrue(PackageManager.is_installed_by_name("com.micropythonos.appstore"))
+
+    def test_installed_not_builtin(self):
+        self.assertTrue(PackageManager.is_installed_by_name("com.micropythonos.helloworld"))
+
+    def test_not_installed(self):
+        self.assertFalse(PackageManager.is_installed_by_name("com.micropythonos.badname"))
diff --git a/tests/simple.py b/tests/simple.py
new file mode 100644
index 00000000..35437d1e
--- /dev/null
+++ b/tests/simple.py
@@ -0,0 +1,6 @@
+import unittest
+
+class TestMath(unittest.TestCase):
+    def test_add(self):
+        self.assertEqual(1 + 1, 2)
+
diff --git a/tests/tests/check_syntax.sh b/tests/tests/check_syntax.sh
new file mode 100755
index 00000000..62c9623c
--- /dev/null
+++ b/tests/tests/check_syntax.sh
@@ -0,0 +1,20 @@
+#!/bin/bash
+mydir=$(readlink -f "$0")
+mydir=$(dirname "$mydir")
+fs="$mydir"/../../internal_filesystem/
+cross="$mydir"/../../lvgl_micropython/lib/micropython/mpy-cross/build/mpy-cross
+
+failed=0
+while read file; do
+	"$cross" -march=x64 -o /dev/null  "$file"
+	exitcode="$?"
+	if [ $exitcode -ne 0 ]; then
+		echo "$file got exitcode $exitcode"
+		failed=$(expr $failed \+ 1)
+	fi
+done < <(find "$fs" -iname "*.py")
+
+if [ $failed -ne 0 ]; then
+	echo "ERROR: $failed .py files have syntax errors"
+	exit 1
+fi
diff --git a/tests/unittest.sh b/tests/unittest.sh
new file mode 100755
index 00000000..d3a59f95
--- /dev/null
+++ b/tests/unittest.sh
@@ -0,0 +1,63 @@
+#!/bin/bash
+
+mydir=$(readlink -f "$0")
+mydir=$(dirname "$mydir")
+testdir="$mydir"
+scriptdir=$(readlink -f "$mydir"/../scripts/)
+fs="$mydir"/../internal_filesystem/
+onetest="$1"
+
+
+# print os and set binary
+os_name=$(uname -s)
+if [ "$os_name" = "Darwin" ]; then
+        echo "Running on macOS"
+        binary="$scriptdir"/../lvgl_micropython/build/lvgl_micropy_macOS
+else
+        # other cases can be added here
+        echo "Running on $os_name"
+        binary="$scriptdir"/../lvgl_micropython/build/lvgl_micropy_unix
+fi
+
+one_test() {
+	file="$1"
+	pushd "$fs"
+	echo "Testing $file"
+	"$binary" -X heapsize=8M -c "import sys ; sys.path.append('lib') 
+$(cat $file)
+result = unittest.main() ; sys.exit(0 if result.wasSuccessful() else 1) "
+	result=$?
+	popd
+	return "$result"
+}
+
+failed=0
+
+if [ -z "$onetest" ]; then
+	echo "Usage: $0 [one_test_to_run.py]"
+	echo "Example: $0 tests/simple.py"
+	echo
+	echo "If no test is specified: run all tests from $testdir"
+	while read file; do
+		one_test "$file"
+		result=$?
+		if [ $result -ne 0 ]; then
+			echo "test $file got error $result"
+			failed=$(expr $failed \+ 1)
+		fi
+
+	done < <( find "$testdir" -iname "*.py" )
+else
+	one_test $(readlink -f "$onetest")
+	[ $? -ne 0 ] && failed=1
+fi
+
+
+if [ $failed -ne 0 ]; then
+        echo "ERROR: $failed .py files have failing unit tests"
+        exit 1
+else
+	echo "GOOD: no .py files have failing unit tests"
+	exit 0
+fi
+