Fixed test of test_cmd_memory_watch: no need for scanf, which mak…

…es testing more hazardous

.github/workflows/run-tests.yml

@@ -66,4 +66,4 @@ jobs:
 
     - name: Run Tests
       run: |
-        bash -c 'make test'
+        make test

tests/binaries/memwatch.c

@@ -1,11 +1,12 @@
 #include<stdio.h>
+#include<stdlib.h>
 
 int myglobal = 1;
 
-int main()
+int main(int argc, char** argv)
 {
     // breakpoints hardcoded for convenience
-    scanf("%d", &myglobal);
+    myglobal = strtoll(argv[1], NULL, 16);
     asm("int3");
     return 0;
 }

tests/runtests.py

@@ -278,14 +278,14 @@ def test_cmd_memory_watch(self):
         self.assertNoException(res)
         target = "/tmp/memwatch.out"
         res = gdb_start_silent_cmd("memory watch &myglobal",
-                before=["set args <<<$((0xdeadbeef))"],
+                before=["set args 0xdeadbeef",],
                 after=["continue",],
                 target=target,
                 context='memory')
         self.assertIn("deadbeef", res)
         self.assertNotIn("cafebabe", res)
         res = gdb_start_silent_cmd("memory watch &myglobal",
-                before=["set args <<<$((0xcafebabe))",],
+                before=["set args 0xcafebabe",],
                 after=["continue", ],
                 target=target,
                 context="memory")