-
Notifications
You must be signed in to change notification settings - Fork 4
Description
Hi,
ersteinmal danke für die Inspiration die mir dieses kleine Projekt von euch geschenkt hat.
Aktuell schreibe ich tests für die assert.sh Funktionen, da ich gerne Teile des Frameworks weiter verwenden möchte.
Dabei ist mir aufgefallen, das assert_range nicht so funktioniert, wie es vielleicht sein sollte.
Testet man assertRange 5 1 3 kommt dabei true raus.
Um dies zu lösen habe ich alle IF-Abfragen in die C-Style Notation (( )) - inkl Änderung von -ge auf equivalente >= - und die assert Abfrage, in der die Negierung stattfindet, auf (( !($1) )) gesetzt.
Danach funktioniert die Funktion wie erwartet. Schien mir die einfachste Lösung hier zu sein und mir persönlich sagt die Verwendung von >= oder <= eher zu als -eq ;)
assertRange würde dann "${1} >= ${2} && ${1} <= ${3}" verwenden.
(Und nicht gesehen, das es hier noch ein testFile gibt und somit bash test benötigt wird.)
Als fix müsste hier die Abfrage in assert_range auf "( ${1} -ge ${2} -a ${1} -le ${3} )" geändert werden.
Beste Grüße,
Lycano