Merge pull request #34 from that-recsys-lab/schwartzadev/add-ci-matrix

feat(ci): run tests on windows and ubuntu

.coveragerc

@@ -2,3 +2,4 @@
 
 [run]
 relative_files = True
+omit = librec_auto/test

.github/workflows/build.yml

@@ -25,7 +25,10 @@ jobs:
         run: |
           yapf librec_auto --recursive --diff
   test:
-    runs-on: ubuntu-latest
+    runs-on: ${{ matrix.os }}
+    strategy:
+      matrix:
+        os: [ubuntu-latest, windows-latest]
     steps:
       - uses: actions/checkout@v2
       - name: Setup Python 3.8
@@ -44,6 +47,7 @@ jobs:
         run: |
           coverage run --source=librec_auto -m pytest librec_auto/test/
       - name: Coveralls
+        if: runner.os != 'Windows'
         uses: AndreMiras/coveralls-python-action@develop
         with:
           parallel: false

librec_auto/test/core/util/test_files.py

@@ -1,8 +1,9 @@
 import os
 import inspect
-import librec_auto
+from sys import platform
 from pathlib import Path
 
+import librec_auto
 from librec_auto.core.util import Files, ExpPaths
 
 
@@ -33,8 +34,13 @@ def test_get_librec_paths():
     assert files.get_jar_path() == Path(
         '/demo/path/to/librec_auto/librec_auto/jar')
 
-    assert files.get_classpath(
-    ) == '/demo/path/to/librec_auto/librec_auto/jar/auto.jar'
+    if platform == "win32":
+        # for windows
+        assert files.get_classpath(
+        ) == 'D:/demo/path/to/librec_auto/librec_auto/jar/auto.jar', "Windows path is correct"
+    else:
+        assert files.get_classpath(
+        ) == '/demo/path/to/librec_auto/librec_auto/jar/auto.jar', "Unix path is correct"
 
 
 def test_get_study_paths():