add a test workflow

mrdcvlsc · May 23, 2024 · 9a2b035 · 9a2b035
1 parent bb2a4d8
commit 9a2b035
Show file tree

Hide file tree

Showing 3 changed files with 30 additions and 14 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -2,20 +2,36 @@ name: ci
 
 on:
   push:
-    branches: [ "main" ]
+    branches: [ "main", "add-ci-test" ]
   pull_request:
-    branches: [ "main" ]
+    branches: [ "main", "add-ci-test" ]
 
 jobs:
   build:
-    runs-on: ${{ matrix.os }}
+    name: ${{matrix.platform.name}}
+    runs-on: ${{matrix.platform.os}}
 
     strategy:
+      fail-fast: false
+
       matrix:
-        os: [ubuntu-latest, windows-latest]
+        platform:
+        # - { name: Windows VS2019, os: windows-2019,   disp: type, testpgn: .\\pgn_samples\\first.pgn, analyzedpgn: .\\pgn_samples\\first.analyzed.pgn, bin: apgn.exe }
+        # - { name: Windows VS2022, os: windows-2022,   disp: type, testpgn: .\\pgn_samples\\first.pgn, analyzedpgn: .\\pgn_samples\\first.analyzed.pgn, bin: apgn.exe }
+        - { name: Windows Clang,  os: windows-latest, disp: type, testpgn: .\\pgn_samples\\first.pgn, analyzedpgn: .\\pgn_samples\\first.analyzed.pgn, bin: apgn.exe,   flags: CC=clang CXX=clang++ }
+        - { name: Windows GCC,    os: windows-latest, disp: type, testpgn: .\\pgn_samples\\first.pgn, analyzedpgn: .\\pgn_samples\\first.analyzed.pgn, bin: apgn.exe,   flags: CC=gcc CXX=g++ }
+        - { name: MacOS XCode,    os: macos-latest,   disp: cat,  testpgn: ./pgn_samples/first.pgn,   analyzedpgn: ./pgn_samples/first.analyzed.pgn,   bin: ./apgn }
+        - { name: Linux GCC,      os: ubuntu-latest,  disp: cat,  testpgn: ./pgn_samples/first.pgn,   analyzedpgn: ./pgn_samples/first.analyzed.pgn,   bin: ./apgn }
+        - { name: Linux Clang,    os: ubuntu-latest,  disp: cat,  testpgn: ./pgn_samples/first.pgn,   analyzedpgn: ./pgn_samples/first.analyzed.pgn,   bin: ./apgn, flags: CC=clang CXX=clang++ }
 
     steps:
     - uses: actions/checkout@v3
 
-    - name: compile and build
-      run: make -j4
+    - name: build for -> ${{matrix.platform.os}}
+      run: make ${{matrix.platform.flags}} -j4
+
+    - name: run test  
+      run: ${{matrix.platform.bin}} -depth 5 ${{matrix.platform.testpgn}}
+
+    - name: display results
+      run: ${{matrix.platform.disp}} ${{matrix.platform.analyzedpgn}}
diff --git a/Makefile b/Makefile
@@ -25,10 +25,10 @@ ifeq ($(OS), Linux)
 else
 	chmod +x bin/engines/stockfish.exe
 endif
-	${CXX} ${CXX_FLAGS} main.cpp -o ${EXECUTABLE}
+	$(CXX) $(CXX_FLAGS) main.cpp -o $(EXECUTABLE)
 
 test:
-	./${EXECUTABLE} ./pgn_samples/first.pgn W
+	./$(EXECUTABLE) ./pgn_samples/first.pgn W
 
 test_clean:
 ifeq ($(OS), Linux)
@@ -39,15 +39,15 @@ endif
 
 install:
 ifeq ($(OS), Linux)
-	@ln -s $(dir $(abspath $(lastword $(MAKEFILE_LIST))))${EXECUTABLE} ${INSTALLPATH}
+	@ln -s $(dir $(abspath $(lastword $(MAKEFILE_LIST))))$(EXECUTABLE) $(INSTALLPATH)
 else
 	@echo "make install is not supported for windows"
 # SETX /M PATH "%PATH%;$(dir $(abspath $(lastword $(MAKEFILE_LIST))))"
 endif
 
 uninstall:
 ifeq ($(OS), Linux)
-	@rm ${INSTALLPATH}/${EXECUTABLE}
+	@rm $(INSTALLPATH)/$(EXECUTABLE)
 else
 	@echo "make uninstall is not supported for windows"
 endif
@@ -58,4 +58,4 @@ clean:
 	@echo "removing uci-analyse object files"
 	@$(MAKE) -C dependencies/uci-analyser clean
 	@echo "removing analyse-pgn binaries files"
-	@rm ./bin/analyse${EXTENSION} ./bin/pgn-extract${EXTENSION} ./${EXECUTABLE}
+	@rm ./bin/analyse$(EXTENSION) ./bin/pgn-extract$(EXTENSION) ./$(EXECUTABLE)
diff --git a/dependencies/uci-analyser/Makefile b/dependencies/uci-analyser/Makefile
@@ -22,7 +22,7 @@
 
 OS := $(shell uname)
 
-CC=g++
+CXX=g++
 
 ifeq ($(OS), Linux)
 # LINUX
@@ -46,10 +46,10 @@ OBJS=analyse.o evaluation.o engine.o utils.o interpret.o
 TARGET=analyse
 
 .cpp.o:
-	$(CC) $(CFLAGS) $< -o $@
+	$(CXX) $(CFLAGS) $< -o $@
 
 $(TARGET) : $(OBJS)
-	$(CC) -o $@ $(OBJS)
+	$(CXX) -o $@ $(OBJS)
 
 clean:
 	rm -f $(OBJS) $(TARGET)