Skip to content

Commit 10fca47

Browse files
committed
CI: add --error-exitcode=1 to valgrind command
Exit when some kind of memory error is detected: now errors are just printed to stdout. Signed-off-by: Davide Bettio <davide@uninstall.it>
1 parent 9c30f60 commit 10fca47

File tree

2 files changed

+10
-10
lines changed

2 files changed

+10
-10
lines changed

.github/workflows/build-and-test.yaml

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -333,60 +333,60 @@ jobs:
333333
run: |
334334
ulimit -c unlimited
335335
./tests/test-erlang -s prime_smp
336-
valgrind ./tests/test-erlang -s prime_smp
336+
valgrind --error-exitcode=1 ./tests/test-erlang -s prime_smp
337337
338338
- name: "Test: test-enif"
339339
working-directory: build
340340
run: |
341341
ulimit -c unlimited
342342
./tests/test-enif
343-
valgrind ./tests/test-enif
343+
valgrind --error-exitcode=1 ./tests/test-enif
344344
345345
- name: "Test: test-heap"
346346
working-directory: build
347347
run: |
348348
ulimit -c unlimited
349349
./tests/test-heap
350-
valgrind ./tests/test-heap
350+
valgrind --error-exitcode=1 ./tests/test-heap
351351
352352
- name: "Test: test-mailbox"
353353
working-directory: build
354354
run: |
355355
ulimit -c unlimited
356356
./tests/test-mailbox
357-
valgrind ./tests/test-mailbox
357+
valgrind --error-exitcode=1 ./tests/test-mailbox
358358
359359
- name: "Test: test-structs"
360360
timeout-minutes: 10
361361
working-directory: build
362362
run: |
363363
ulimit -c unlimited
364364
./tests/test-structs
365-
valgrind ./tests/test-structs
365+
valgrind --error-exitcode=1 ./tests/test-structs
366366
367367
- name: "Test: test_estdlib.avm"
368368
timeout-minutes: 5
369369
working-directory: build
370370
run: |
371371
ulimit -c unlimited
372372
./src/AtomVM ./tests/libs/estdlib/test_estdlib.avm
373-
valgrind ./src/AtomVM ./tests/libs/estdlib/test_estdlib.avm
373+
valgrind --error-exitcode=1 ./src/AtomVM ./tests/libs/estdlib/test_estdlib.avm
374374
375375
- name: "Test: test_eavmlib.avm"
376376
timeout-minutes: 10
377377
working-directory: build
378378
run: |
379379
ulimit -c unlimited
380380
./src/AtomVM ./tests/libs/eavmlib/test_eavmlib.avm
381-
valgrind ./src/AtomVM ./tests/libs/eavmlib/test_eavmlib.avm
381+
valgrind --error-exitcode=1 ./src/AtomVM ./tests/libs/eavmlib/test_eavmlib.avm
382382
383383
- name: "Test: test_alisp.avm"
384384
timeout-minutes: 10
385385
working-directory: build
386386
run: |
387387
ulimit -c unlimited
388388
./src/AtomVM ./tests/libs/alisp/test_alisp.avm
389-
valgrind ./src/AtomVM ./tests/libs/alisp/test_alisp.avm
389+
valgrind --error-exitcode=1 ./src/AtomVM ./tests/libs/alisp/test_alisp.avm
390390
391391
- name: "Test: Tests.avm (Elixir)"
392392
timeout-minutes: 10
@@ -396,7 +396,7 @@ jobs:
396396
if command -v elixirc &> /dev/null
397397
then
398398
./src/AtomVM ./tests/libs/exavmlib/Tests.avm
399-
valgrind ./src/AtomVM ./tests/libs/exavmlib/Tests.avm
399+
valgrind --error-exitcode=1 ./src/AtomVM ./tests/libs/exavmlib/Tests.avm
400400
fi
401401
402402
- name: "Install and smoke test"

.github/workflows/build-libraries.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ jobs:
7676
working-directory: build
7777
run: |
7878
./src/AtomVM ./tests/libs/estdlib/test_estdlib.avm
79-
valgrind ./src/AtomVM ./tests/libs/estdlib/test_estdlib.avm
79+
valgrind --error-exitcode=1 ./src/AtomVM ./tests/libs/estdlib/test_estdlib.avm
8080
8181
- name: "Rename and write sha256sum"
8282
working-directory: build

0 commit comments

Comments
 (0)