add a test workflow

Author: mrdcvlsc

.clangd

@@ -0,0 +1,9 @@
+InlayHints:
+  Designators: No
+  Enabled: Yes
+  ParameterNames: No
+  DeducedTypes: No
+
+CompileFlags:
+  Add: [-std=c++17, -Wall, -Wextra, -Wpedantic, -Werror]
+  Compiler: clang++

.github/workflows/build.yml

@@ -2,20 +2,53 @@ 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, ls: dir, os: windows-2019,   disp: type, sample: pgn_samples, testpgn: pgn_samples\\first.pgn, analyzedpgn: .\\pgn_samples\\first.analyzed.pgn, bin: .\\apgn.exe }
+        # - { name: Windows VS2022, ls: dir, os: windows-2022,   disp: type, sample: pgn_samples, testpgn: pgn_samples\\first.pgn, analyzedpgn: .\\pgn_samples\\first.analyzed.pgn, bin: .\\apgn.exe }
+        # - { name: Windows-Clang,  ls: dir, os: windows-latest, disp: type, sample: pgn_samples, testpgn: pgn_samples\\first.pgn, analyzedpgn: .\\pgn_samples\\first.analyzed.pgn, bin: .\\apgn.exe, flags: CC=clang CXX=clang++ }
+        - { name: Windows-GCC,    ls: dir, os: windows-latest, disp: type, sample: pgn_samples,   testpgn: pgn_samples\\first.pgn,  analyzedpgn: .\\pgn_samples\\first.analyzed.pgn, bin: .\\apgn.exe, flags: CC=gcc CXX=g++ }
+        - { name: Linux-GCC,      ls: ls,  os: ubuntu-latest,  disp: cat,  sample: ./pgn_samples, testpgn: ./pgn_samples/first.pgn, analyzedpgn: ./pgn_samples/first.analyzed.pgn,   bin: ./apgn }
+        - { name: Linux-Clang,    ls: ls,  os: ubuntu-latest,  disp: cat,  sample: ./pgn_samples, testpgn: ./pgn_samples/first.pgn, analyzedpgn: ./pgn_samples/first.analyzed.pgn,   bin: ./apgn,      flags: CC=clang CXX=clang++ }
+        # - { name: MacOS-XCode,    ls: ls,  os: macos-latest,   disp: cat,  sample: ./pgn_samples, testpgn: ./pgn_samples/first.pgn, analyzedpgn: ./pgn_samples/first.analyzed.pgn,   bin: ./apgn }
 
     steps:
     - uses: actions/checkout@v3
 
-    - name: compile and build
-      run: make -j4
+    - run: echo "this is a test" > test.log
+
+    - name: build for -> ${{matrix.platform.os}}
+      run: make ${{matrix.platform.flags}} -j4
+
+    - name: ready executable for execution
+      run: chmod a+rx ${{matrix.platform.bin}}
+
+    - name: ready first.pgn for reading
+      run: chmod a+r ${{matrix.platform.testpgn}}
+
+    - name: run test
+      run: ${{matrix.platform.bin}} -depth 5 ${{matrix.platform.testpgn}}
+
+    - name: show files in current directory
+      run: ${{matrix.platform.ls}}
+
+    - name: show files in sample pgn folder
+      run: ${{matrix.platform.ls}} ${{matrix.platform.sample}}
+
+    - name: display pgn test subject
+      run: ${{matrix.platform.disp}} ${{matrix.platform.testpgn}}
+
+    - name: display results
+      run: ${{matrix.platform.disp}} ${{matrix.platform.analyzedpgn}}

Makefile

@@ -21,14 +21,14 @@ all:
 	mv dependencies/uci-analyser/analyse bin/analyse
 
 ifeq ($(OS), Linux)
-	chmod +x bin/engines/stockfish
+	chmod a+rx bin/engines/stockfish
 else
-	chmod +x bin/engines/stockfish.exe
+	chmod a+rx 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)

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)

dependencies/uci-analyser/interpret.cpp

@@ -1,5 +1,7 @@
+#include <iterator>
 #include <map>
 #include <cmath>
+#include <stdexcept>
 #include "interpret.hpp"
 
 using namespace std;
@@ -88,8 +90,14 @@ namespace interpret
 
         ofstream outfile;
         outfile.open(saveFile+".stats.txt",ios_base::out | ios_base::app);
-        outfile<<"==============================================================\n";
-        outfile<<statCounts;
+
+        if (outfile.good()) {
+            outfile<<"==============================================================\n";
+            outfile<<statCounts;
+        } else {
+            throw std::runtime_error("Error writing to the statistics file");
+        }
+
         outfile.close();
     }