Add pytest-sentry (test)

Tribler · Nov 21, 2022 · d534eca · d534eca
1 parent 4ea5f08
commit d534eca
Show file tree

Hide file tree

Showing 3 changed files with 137 additions and 116 deletions.
diff --git a/.github/workflows/!PR.yml b/.github/workflows/!PR.yml
@@ -1,7 +1,7 @@
 name: PR
 
 on:
-  pull_request:
+  pull_request_target:
     types:
       - opened
       - synchronize
@@ -45,77 +45,88 @@ jobs:
               - 'build/**'
               - 'src/**'
               - 'requirements*'
-  
-
+
+  gatekeeper: # check user's permissions
+    runs-on: ubuntu-latest
+    steps:
+      - name: Check if user has write access
+        uses: lannonbr/repo-permission-check-action@2bb8c89ba8bf115c4bfab344d6a6f442b24c9a1f
+        with:
+          permission: write
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+
   # PR is Draft and PR is Ready
   pytest:
-    needs: changes
-    if: ${{ needs.changes.outputs.src == 'true' }}
+    needs: [changes, gatekeeper]
+#    if: ${{ needs.changes.outputs.src == 'true' }}
     uses: ./.github/workflows/pytest.yml
+#    secrets:
+#      PYTEST_SENTRY_DSN: ${{secrets.PYTEST_SENTRY_DSN}}
     with:
       python-version: 3.8
       matrix: '{"os": ["windows-latest"]}'
-
-  guitest:
-    needs: changes
-    if: ${{ needs.changes.outputs.src == 'true' }}
-    uses: ./.github/workflows/guitest.yml
-    with:
-      python-version: 3.8
-      matrix: '{"os": ["windows-latest"]}'
-
-  scripttest:
-    needs: changes
-    if: ${{ needs.changes.outputs.scripts == 'true' }}
-    uses: ./.github/workflows/scripttest.yml
-    with:
-      python-version: 3.8
-
-  coverage:
-    needs: changes
-    if: ${{ needs.changes.outputs.src == 'true' }}
-    uses: ./.github/workflows/coverage.yml
-    with:
-      python-version: 3.8
-
-  # PR is Ready only
-  pytest_nix:
-    needs: changes
-    if: ${{needs.changes.outputs.src == 'true' && !github.event.pull_request.draft}}
-    uses: ./.github/workflows/pytest.yml
-    with:
-      python-version: 3.8
-      matrix: '{"os": ["macos-latest", "ubuntu-latest"]}'
-
-  guitest_nix:
-    needs: changes
-    if: ${{needs.changes.outputs.src == 'true' && !github.event.pull_request.draft}}
-    uses: ./.github/workflows/guitest.yml
-    with:
-      python-version: 3.8
-      matrix: '{"os": ["macos-latest", "ubuntu-latest"]}'
-
-  ubuntu:
-    needs: changes
-    if: ${{needs.changes.outputs.build == 'true' && !github.event.pull_request.draft}}
-    uses: ./.github/workflows/build_ubuntu.yml
-    with:
-      upload: false
-      os: ubuntu-20.04
-      python-version: 3.8
-
-  windows:
-    needs: changes
-    if: ${{needs.changes.outputs.build == 'true' && !github.event.pull_request.draft}}
-    uses: ./.github/workflows/build_windows.yml
-    with:
-      upload: false
-      os: windows-latest
-      python-version: 3.8
-
-  documentation:
-    needs: changes
-    if: ${{needs.changes.outputs.doc == 'true' && !github.event.pull_request.draft}}
-    uses: ./.github/workflows/documentation.yml
-    with:
-      python-version: 3.8
+#
+#  guitest:
+#    needs: changes
+#    if: ${{ needs.changes.outputs.src == 'true' }}
+#    uses: ./.github/workflows/guitest.yml
+#    with:
+#      python-version: 3.8
+#      matrix: '{"os": ["windows-latest"]}'
+#
+#  scripttest:
+#    needs: changes
+#    if: ${{ needs.changes.outputs.scripts == 'true' }}
+#    uses: ./.github/workflows/scripttest.yml
+#    with:
+#      python-version: 3.8
+#
+#  coverage:
+#    needs: changes
+#    if: ${{ needs.changes.outputs.src == 'true' }}
+#    uses: ./.github/workflows/coverage.yml
+#    with:
+#      python-version: 3.8
+#
+#  # PR is Ready only
+#  pytest_nix:
+#    needs: changes
+#    if: ${{needs.changes.outputs.src == 'true' && !github.event.pull_request.draft}}
+#    uses: ./.github/workflows/pytest.yml
+#    with:
+#      python-version: 3.8
+#      matrix: '{"os": ["macos-latest", "ubuntu-latest"]}'
+#
+#  guitest_nix:
+#    needs: changes
+#    if: ${{needs.changes.outputs.src == 'true' && !github.event.pull_request.draft}}
+#    uses: ./.github/workflows/guitest.yml
+#    with:
+#      python-version: 3.8
+#      matrix: '{"os": ["macos-latest", "ubuntu-latest"]}'
+#
+#  ubuntu:
+#    needs: changes
+#    if: ${{needs.changes.outputs.build == 'true' && !github.event.pull_request.draft}}
+#    uses: ./.github/workflows/build_ubuntu.yml
+#    with:
+#      upload: false
+#      os: ubuntu-20.04
+#      python-version: 3.8
+#
+#  windows:
+#    needs: changes
+#    if: ${{needs.changes.outputs.build == 'true' && !github.event.pull_request.draft}}
+#    uses: ./.github/workflows/build_windows.yml
+#    with:
+#      upload: false
+#      os: windows-latest
+#      python-version: 3.8
+#
+#  documentation:
+#    needs: changes
+#    if: ${{needs.changes.outputs.doc == 'true' && !github.event.pull_request.draft}}
+#    uses: ./.github/workflows/documentation.yml
+#    with:
+#      python-version: 3.8
diff --git a/.github/workflows/pytest.yml b/.github/workflows/pytest.yml
@@ -2,6 +2,11 @@ name: Pytest
 
 on:
   workflow_call:
+    secrets:
+      PYTEST_SENTRY_DSN:
+        description: 'Sentry URL'
+        required: false
+
     inputs:
       python-version:
         default: 3.8
@@ -30,52 +35,56 @@ jobs:
         shell: bash
 
     timeout-minutes: 10
+    env:
+      PYTEST_SENTRY_DSN: ${{secrets.PYTEST_SENTRY_DSN}}
 
     steps:
       - uses: actions/checkout@v3
+      - run:
+          echo $PYTEST_SENTRY_DSN
 
-      - name: Create python environment
-        uses: ./.github/actions/pyenv
-        with:
-          python-version: ${{inputs.python-version}}
-          requirements: requirements-test.txt
-
-      - name: Install dependencies (Windows)
-        if: runner.os == 'Windows'
-        uses: ./.github/actions/windows_dependencies
-
-      - name: Export env
-        uses: cardinalby/export-env-action@v2
-        with:
-          envFile: ./.github/workflows/vars/pytest.env
-          expand: true
-
-      - name: Add --looptime
-        if: runner.os != 'Windows'
-        run: |
-          echo "PYTEST_CORE_ARGUMENTS=${PYTEST_CORE_ARGUMENTS} --looptime" >> $GITHUB_ENV
-          echo "PYTEST_TUNNELS_ARGUMENTS=${PYTEST_TUNNELS_ARGUMENTS} --looptime" >> $GITHUB_ENV
-
-      - name: Run Pytest
-        if: ${{!inputs.enable_profiling}}
-        run: |
-          pytest ${PYTEST_CORE_ARGUMENTS}
-
-      - name: Run Pytest (Profiler)
-        if: ${{inputs.enable_profiling}}
-        uses: ./.github/actions/profile
-        with:
-          artifact_name: pytest_prof.svg
-          arguments: ${PYTEST_CORE_ARGUMENTS}
-
-      - name: Run Tunnels Tests
-        if: ${{!inputs.enable_profiling}}
-        run: |
-          pytest ${PYTEST_TUNNELS_ARGUMENTS}
-
-      - name: Run Tunnel Tests (Profiler)
-        if: ${{inputs.enable_profiling}}
-        uses: ./.github/actions/profile
-        with:
-          artifact_name: tunneltest_prof.svg
-          arguments: ${PYTEST_TUNNELS_ARGUMENTS}
+#      - name: Create python environment
+#        uses: ./.github/actions/pyenv
+#        with:
+#          python-version: ${{inputs.python-version}}
+#          requirements: requirements-test.txt
+#
+#      - name: Install dependencies (Windows)
+#        if: runner.os == 'Windows'
+#        uses: ./.github/actions/windows_dependencies
+#
+#      - name: Export env
+#        uses: cardinalby/export-env-action@v2
+#        with:
+#          envFile: ./.github/workflows/vars/pytest.env
+#          expand: true
+#
+#      - name: Add --looptime
+#        if: runner.os != 'Windows'
+#        run: |
+#          echo "PYTEST_CORE_ARGUMENTS=${PYTEST_CORE_ARGUMENTS} --looptime" >> $GITHUB_ENV
+#          echo "PYTEST_TUNNELS_ARGUMENTS=${PYTEST_TUNNELS_ARGUMENTS} --looptime" >> $GITHUB_ENV
+#
+#      - name: Run Pytest
+#        if: ${{!inputs.enable_profiling}}
+#        run: |
+#          pytest ${PYTEST_CORE_ARGUMENTS}
+#
+#      - name: Run Pytest (Profiler)
+#        if: ${{inputs.enable_profiling}}
+#        uses: ./.github/actions/profile
+#        with:
+#          artifact_name: pytest_prof.svg
+#          arguments: ${PYTEST_CORE_ARGUMENTS}
+#
+#      - name: Run Tunnels Tests
+#        if: ${{!inputs.enable_profiling}}
+#        run: |
+#          pytest ${PYTEST_TUNNELS_ARGUMENTS}
+#
+#      - name: Run Tunnel Tests (Profiler)
+#        if: ${{inputs.enable_profiling}}
+#        uses: ./.github/actions/profile
+#        with:
+#          artifact_name: tunneltest_prof.svg
+#          arguments: ${PYTEST_TUNNELS_ARGUMENTS}
diff --git a/requirements-test.txt b/requirements-test.txt
@@ -8,12 +8,13 @@ pytest-randomly==3.12.0
 pytest-timeout==2.1.0
 pytest-freezegun==0.4.2
 pytest-rerunfailures==10.2
+pytest-sentry==0.1.10
+pytest-profiling==1.7.0 # for pytest profiling
+
 freezegun==1.2.1
 coverage==6.3.2
 looptime==0.2
 
 asynctest==0.13.0 # this library has to be installed to properly work with ipv8 TestBase.
 
 scipy==1.8.0
-
-pytest-profiling==1.7.0 # for pytest profiling