Revert "ci: Only upload server-side coverage report if .py files are changed"

partially reverts this PR https://github.com/frappe/frappe/pull/16048

This is causing random fluctations.

.github/helper/roulette.py

@@ -62,7 +62,6 @@ if __name__ == "__main__":
 	# this is a push build, run all builds
 	if not pr_number:
 		os.system('echo "::set-output name=build::strawberry"')
-		os.system('echo "::set-output name=build-server::strawberry"')
 		sys.exit(0)
 
 	files_list = files_list or get_files_list(pr_number=pr_number, repo=repo)

.github/workflows/ui-tests.yml

@@ -149,20 +149,15 @@ jobs:
         env:
           CYPRESS_RECORD_KEY: 4a48f41c-11b3-425b-aa88-c58048fa69eb
 
+      # Required to flush python coverage files
       - name: Stop server
-        if: ${{ steps.check-build.outputs.build-server == 'strawberry' }}
+        if: ${{ steps.check-build.outputs.build == 'strawberry' }}
         run: |
           ps -ef | grep "frappe serve" | awk '{print $2}' | xargs kill -s SIGINT 2> /dev/null || true
           sleep 5
 
-      - name: Check If Coverage Report Exists
-        id: check_coverage
-        uses: andstor/file-existence-action@v1
-        with:
-          files: "/home/runner/frappe-bench/apps/frappe/.cypress-coverage/clover.xml"
-
       - name: Upload JS coverage data
-        if: ${{ steps.check-build.outputs.build == 'strawberry' && steps.check_coverage.outputs.files_exists == 'true' }}
+        if: ${{ steps.check-build.outputs.build == 'strawberry' }}
         uses: actions/upload-artifact@v3
         with:
           name: coverage-js-${{ matrix.container }}