use valgrind to test memory error on Linux,run example scripts

NeuroJSON · Oct 22, 2023 · c79f969 · c79f969
1 parent ff2a584
commit c79f969
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 7 deletions.
diff --git a/.github/workflows/run_test.yml b/.github/workflows/run_test.yml
@@ -77,7 +77,6 @@ jobs:
           which gcc
           gcc -v
           make -C src oct CC=gcc CXX=g++ USERLINKOPT="-static-libgcc -static-libstdc++ -fstack-protector -L/mingw64/lib -Wl,-Bstatic -lz -lwinpthread -Wl,-Bdynamic -v -s '$PWD/octave/octave-8.3.0-w64-64/mingw64/lib/octave/8.3.0/liboctinterp.dll.a'"
-          cp /mingw64/bin/zlib1.dll .
           objdump -p zipmat.mex | grep "DLL Name:"
           strip zipmat.mex
           echo "$PWD/octave/octave-8.3.0-w64-64/mingw64/bin" >> $GITHUB_PATH
@@ -99,18 +98,18 @@ jobs:
           strip -S zipmat.mex
           upx -9 zipmat.mex || true
       - name: Run octave test
-        if: ${{ matrix.os != 'ubuntu-20.04'}}
+        if: ${{ runner.os != 'Linux' }}
         run: |
           octave-cli --version
           if [ "$RUNNER_OS" == "Windows" ]; then
               export PATH="$PWD/octave/octave-8.3.0-w64-64/mingw64/bin":$PATH
           fi
-          octave-cli --eval "addpath(pwd);cd test;run_zmat_test"
+          octave-cli --eval "addpath(pwd);cd test;run_zmat_test;cd ../example;demo_zmat_basic;zmat_speedbench"
       - name: Run octave test
-        if: ${{ matrix.os == 'ubuntu-20.04'}}
+        if: ${{ runner.os != 'Linux' }}
         run: |
           octave-cli --version
-          valgrind octave-cli --eval "addpath(pwd);cd test;run_zmat_test"
+          valgrind octave-cli --eval "addpath(pwd);cd test;run_zmat_test;cd ../example;demo_zmat_basic;zmat_speedbench"
       - name: Create package folder
         run: |
           mkdir packages
@@ -126,7 +125,6 @@ jobs:
         run: |
           mv zipmat.mex zmat/octave/${{ env.OCTAVE_ARCH }}
           if [[ "$RUNNER_OS" == "Windows" ]]; then
-            cp /mingw64/bin/zlib1.dll zmat/octave/${{ env.OCTAVE_ARCH }}
             7z a -tzip packages/zmat-octave-${{ env.RELEASE_TAG }}.zip zmat
           else
             zip -FSr --symlink packages/zmat-octave-${{ env.RELEASE_TAG }}.zip zmat
@@ -200,7 +198,7 @@ jobs:
       - name: Run MATLAB examples
         uses: matlab-actions/run-command@v1
         with:
-          command: addpath(pwd);cd test;run_zmat_test
+          command: addpath(pwd);cd test;run_zmat_test;cd ../example;demo_zmat_basic;zmat_speedbench
       - name: Create package folder
         run: |
           mkdir packages

diff --git a/ChangeLog.txt b/ChangeLog.txt
@@ -2,6 +2,7 @@
 
 == ZMAT 0.9.9 (Foxy the Fantastic Mr. Fox - RC1), FangQ <q.fang (a) neu.edu> ==
 
+ 2023-10-22 [48ba900] fix memory error using valgrind, run full tests on all OS
  2023-10-13 [d5b2c40] revert blosc2 to v2.8.0 to remove new dependency libdl
  2023-10-13 [f2eadb5] fix broken test
  2023-10-13 [e6bbda1] update cmake file, update README