From c79f969171a377067bb65d3eccc0aa37c131f4fe Mon Sep 17 00:00:00 2001
From: Qianqian Fang <fangqq@gmail.com>
Date: Sun, 22 Oct 2023 12:27:10 -0400
Subject: [PATCH] use valgrind to test memory error on Linux,run example
 scripts

---
 .github/workflows/run_test.yml | 12 +++++-------
 ChangeLog.txt                  |  1 +
 2 files changed, 6 insertions(+), 7 deletions(-)

diff --git a/.github/workflows/run_test.yml b/.github/workflows/run_test.yml
index 9c22ad4..971423e 100644
--- 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
index 823c349..697db3e 100644
--- 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