Merge pull request #676 from allure-framework/workflow-rework

Rework workflows

Author: skhomuti

.flake8

@@ -0,0 +1,4 @@
+[flake8]
+max-line-length = 120
+import-order-style = google
+inline-quotes = "

.github/workflows/build.yaml

@@ -1,44 +1,110 @@
 name: build allure python
 
-on: [push, pull_request]
+on:
+  pull_request:
+    branches:
+      - master
 
 jobs:
+  changes:
+    name: Collect file changes
+    runs-on: ubuntu-latest
+    outputs:
+      packages: ${{ steps.filter.outputs.changes }}
+    steps:
+      - uses: dorny/paths-filter@v2
+        id: filter
+        with:
+          filters: |
+            allure-behave: allure-behave/**
+            allure-nose2: allure-nose2/**
+            allure-pytest: allure-pytest/**
+            allure-pytest-bdd: allure-pytest-bdd/**
+            allure-python-commons: allure-python-commons/**
+            allure-python-commons-test: allure-python-commons-test/**
+            allure-robotframework: allure-robotframework/**
+
+  commons:
+    name: Build commons
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v3
+
+      - name: Cache commons
+        id: commons
+        uses: actions/cache@v3
+        with:
+          path: dist/
+          key: commons-${{ github.sha }}
+
+      - name: Build python commons
+        if: steps.commons.outputs.cache-hit != 'true'
+        run: pip install build &&
+             python -m build allure-python-commons --outdir dist/ &&
+             python -m build allure-python-commons-test --outdir dist/
+
+  linters:
+    name: Static check
+    runs-on: ubuntu-latest
+    needs: [commons, changes]
+    strategy:
+      matrix:
+        package: ${{ fromJSON(needs.changes.outputs.packages) }}
+    steps:
+      - uses: actions/checkout@v3
+
+      - name: Set up Python
+        uses: actions/setup-python@v4
+        with:
+          python-version: "3.10"
+
+      - name: Get commons from cache
+        id: commons
+        uses: actions/cache@v3
+        with:
+          path: dist/
+          key: commons-${{ github.sha }}
+
+      - name: Install packages
+        run: pip install dist/allure-python-commons*.tar.gz &&
+             pip install ./${{ matrix.package }} &&
+             pip install -r ${{ matrix.package }}/requirements.txt
+
+      - name: Static check ${{ matrix.package }}
+        working-directory: ${{ matrix.package }}
+        run: poe linter
+
   build:
+    name: Build package
     runs-on: ubuntu-latest
+    needs: [linters, commons, changes]
     strategy:
       matrix:
-        package: [
-          allure-python-commons-test,
-          allure-python-commons,
-          allure-nose2,
-          allure-behave,
-          allure-pytest,
-          allure-pytest-bdd,
-          allure-robotframework
-        ]
+        package: ${{ fromJSON(needs.changes.outputs.packages) }}
         python-version: [3.7, 3.8, 3.9, "3.10"]
-        include:
-          - python-version: 3.10
-            static-check: yes
-
+    env:
+      TEST_TMP: /tmp
+      ALLURE_INDENT_OUTPUT: yep
     steps:
-    - uses: actions/checkout@v2
-
-    - name: Set up Python ${{ matrix.python-version }}
-      uses: actions/setup-python@v1
-      with:
-        python-version: ${{ matrix.python-version }}
-
-    - name: Install tox
-      run: pip install tox
-      
-    - name: Static check ${{ matrix.package }}
-      if: matrix.static-check
-      run: tox -e static-check -c ${{ matrix.package }}/tox.ini
-
-    - name: Prepare commons
-      if: "!startsWith(matrix.package, 'allure-python-commons')"
-      run: tox --sdistonly -c ./allure-python-commons && tox --sdistonly -c ./allure-python-commons-test
-
-    - name: Test ${{ matrix.package }}
-      run: tox -e py`echo ${{ matrix.python-version }} | sed 's/\.//g'` -c ${{ matrix.package }}/tox.ini
+      - uses: actions/checkout@v3
+
+      - name: Set up Python ${{ matrix.python-version }}
+        uses: actions/setup-python@v4
+        with:
+          python-version: ${{ matrix.python-version }}
+
+      - name: Get commons from cache
+        id: commons
+        uses: actions/cache@v3
+        with:
+          path: dist/
+          key: commons-${{ github.sha }}
+
+      - name: Install packages
+        run: pip install dist/allure-python-commons*.tar.gz &&
+             pip install ./${{ matrix.package }} &&
+             pip install -r ${{ matrix.package }}/requirements.txt
+
+      - name: Test ${{ matrix.package }}
+        working-directory: ${{ matrix.package }}
+        run: poe tests

allure-behave/pyproject.toml

@@ -0,0 +1,4 @@
+[tool.poe.tasks]
+linter = "flake8 ./src"
+tests = """behave -f allure_behave.formatter:AllureFormatter -o allure-results
+              -f pretty ./features"""

allure-behave/requirements.txt

@@ -0,0 +1,3 @@
+poethepoet
+# linters
+flake8==4.*

allure-behave/tox.ini

@@ -0,0 +1,4 @@
+[tool.poe.tasks]
+linter = "flake8 ./src ./test"
+tests = """nose2 --plugin=test.example_loader --plugin=allure_nose2.plugin --allure
+                --current-example -t . -s ./test"""

allure-nose2/pyproject.toml

@@ -0,0 +1,4 @@
+poethepoet
+# linters
+flake8==4.*
+flake8-builtins

allure-nose2/requirements.txt

@@ -56,6 +56,6 @@ def main():
         install_requires=install_requires
     )
 
+
 if __name__ == '__main__':
     main()
-

allure-nose2/setup.py

@@ -57,7 +57,6 @@ def run_docstring_example(**kwargs):
         with redirect_stdout(stdout):
             test_nose2_instance = test_nose2(**kwargs)
 
-    #test_nose2_instance = test_nose2(**kwargs)
+    # test_nose2_instance = test_nose2(**kwargs)
 
     return get_plugin(test_nose2_instance, _Allure).logger
-

allure-nose2/test/example_runner.py

@@ -56,4 +56,4 @@ def test_parametrized_method(self, first, second):
                                   has_parameter(first_param_name, represent(first_param_value)),
                                   has_parameter(second_param_name, represent(second_param_value))
                                   )
-                    )
+                    )