Add the Coveralls step to the Github Actions workflow (#272)

BayesWitnesses · Jul 24, 2020 · 591ce95 · 591ce95
1 parent 02a453e
commit 591ce95
Show file tree

Hide file tree

Showing 5 changed files with 45 additions and 47 deletions.
diff --git a/.ci/setup.sh b/.ci/setup.sh
@@ -2,6 +2,9 @@
 
 set -e
 
+pip install Cython numpy
+pip install -r $BUILD_DIRECTORY/requirements-test.txt
+
 # Install .NET Core SDK.
 if [[ $LANG == *"c_sharp"* ]] || [[ $LANG == *"f_sharp"* ]] || [[ $LANG == *"visual_basic"* ]]; then
   wget -q https://packages.microsoft.com/config/ubuntu/$(lsb_release -rs)/packages-microsoft-prod.deb

diff --git a/.ci/test.sh b/.ci/test.sh
@@ -7,9 +7,7 @@ cd $BUILD_DIRECTORY
 if [[ $TEST == "API" ]]; then
   flake8 .
   pytest -v tests/ --cov=m2cgen/ --ignore=tests/e2e/
-  if [[ $TRAVIS == "true" ]]; then
-    coveralls
-  fi
+  coveralls
 fi
 
 if [[ $TEST == "E2E" ]]; then

diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
@@ -10,7 +10,36 @@ on:
 
 jobs:
   test:
-    name: "Python ${{ matrix.python }}, Test: ${{ matrix.lang }}"
+    name: "Python ${{ matrix.python }} API tests"
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        python:
+          - 3.6
+          - 3.7
+          - 3.8
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v1
+        with:
+          fetch-depth: 5
+      - name: Setup Python ${{ matrix.python }}
+        uses: actions/setup-python@v2
+        with:
+          python-version: ${{ matrix.python }}
+      - name: Setup dependencies and run API tests
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+          TEST: API
+          PYTHON: ${{ matrix.python }}
+          BUILD_DIRECTORY: ${{ github.workspace }}
+        run: |
+          bash $GITHUB_WORKSPACE/.ci/setup.sh
+          bash $GITHUB_WORKSPACE/.ci/test.sh
+
+  e2e-test:
+    name: "Python ${{ matrix.python }} E2E tests"
     runs-on: ubuntu-latest
     strategy:
       fail-fast: false
@@ -20,7 +49,6 @@ jobs:
           - 3.7
           - 3.8
         lang:
-          - "API"
           - "c_lang or python or java or go_lang or javascript or php or haskell or ruby"
           - "c_sharp or visual_basic or f_sharp"
           - "r_lang or dart"
@@ -34,20 +62,16 @@ jobs:
         uses: actions/setup-python@v2
         with:
           python-version: ${{ matrix.python }}
-      - name: Setup dependencies and run tests
+      - name: Setup dependencies and run E2E tests
+        env:
+          TEST: E2E
+          LANG: ${{ matrix.lang }}
+          PYTHON: ${{ matrix.python }}
+          BUILD_DIRECTORY: ${{ github.workspace }}
+          LC_ALL: en_US.UTF-8
         run: |
-          if [[ "${{ matrix.lang }}" == "API" ]]; then
-              export TEST="API";
-          else
-              export TEST="E2E";
-              export LC_ALL="en_US.UTF-8";
-              sudo locale-gen $LC_ALL;
-              sudo update-locale;
-          fi
-          export PYTHON="${{ matrix.python }}"
-          export LANG="${{ matrix.lang }}"
-          export BUILD_DIRECTORY=$GITHUB_WORKSPACE
+          sudo locale-gen $LC_ALL
+          sudo update-locale
+
           bash $GITHUB_WORKSPACE/.ci/setup.sh
-          pip install Cython numpy
-          pip install -r $GITHUB_WORKSPACE/requirements-test.txt
           bash $GITHUB_WORKSPACE/.ci/test.sh
diff --git a/.travis.yml b/.travis.yml
diff --git a/requirements-test.txt b/requirements-test.txt
@@ -9,7 +9,7 @@ git+git://github.com/scikit-learn-contrib/lightning.git@782c18c12961e509099ae84c
 flake8==3.8.3
 pytest==5.4.3
 pytest-mock==3.2.0
-coveralls==2.1.0
+coveralls==2.1.1
 pytest-cov==2.10.0
 
 # Other stuff