search

andmarti1424 / sc-im

sc-im - Spreadsheet Calculator Improvised -- An ncurses spreadsheet program for terminal
Other
4.8k stars 203 forks source link

test7.sh is broken #889

Open mipmip opened 3 weeks ago

mipmip commented 3 weeks ago

it breaks with this output:

➜ pim@rodin tests (feature/setupSimpleCI) ✗ bash test7.sh
test #1 "echo -e 'LET A7=90\nGETNUM A20\nGETNUM G20\nUNDO\nREDO\nGETNUM A20\nGETNUM G20' | valgrind -v --log-file=${NAME}_vallog --tool=memcheck --track-origins=yes --leak-check=full --show-leak-kinds=all --show-reachable=no ../src/sc-im test7.sc --nocurses --nodebug --quit_afterload 2>&1 |grep -v '^$\|Interp\|Change'" failed:
        expected "103\n954958\n103\n954958"
        got "21\n953810\n21\n973818"
1 of 9 test7 tests failed in 1.145s.
andmarti1424 commented 3 weeks ago

Seems there is a bug I never fixed. It was even there when I added the test7 on this commit: https://github.com/andmarti1424/sc-im/commit/2d7722682a2188b1f60c56391800d574f37b8024