Do not set GITHUB_ACTIONS env var when running tests

Author: javihernandez

.github/workflows/pytest.yml

@@ -1,6 +1,6 @@
 name: pytest
 on:
-  pull_request_target:
+  pull_request:
     branches:
       - "**"
   push:
@@ -52,7 +52,7 @@ jobs:
             db
             web_server
       - name: Run unit tests (pytest)
-        run: docker-compose run -e GITHUB_ACTIONS=true -v /tmp:/tmp --no-deps web_server bash -o pipefail -c 'source
+        run: docker-compose run -v /tmp:/tmp --no-deps web_server bash -o pipefail -c 'source
           env/bin/activate && pip3 install -r requirements.txt &&
           pytest -vv --ignore alws/ --cov-report term-missing:skip-covered
           --cov-report xml:/tmp/coverage.xml --junitxml=/tmp/pytest.xml --cov=alws

docker-compose.yml

@@ -528,8 +528,6 @@ services:
 
   sign_file:
     image: sign-file:latest
-    env_file:
-      - ../albs-sign-file/.env
     build:
       dockerfile: Dockerfile
       context: ../albs-sign-file

tests/test_scripts/test_packages_exporter.py

@@ -10,42 +10,7 @@
 
 
 @pytest.mark.skip(
-    reason="""
-        This test needs to be rewritten.
-
-        First, it needs to be converted into an async test.
-
-        We need to bring up a configured sign_file instance when setting up the
-        test environment. This is, prepare everything to run:
-
-        `docker compose run --no-deps web_server db sign_file ...`
-
-        The sign_file service should:
-        * Have a user inside db
-        * A key in gpg database
-
-        Also, there is no point in getting the sign_keys from web_server as we
-        haven't loaded any keys into testing db. We should load sign keys into
-        db to request keys from web_server if we want this test to work that
-        way. Which can be easily done using a fixture.
-        In any case, I propose to, simply, pass the key_id of the key(s) that
-        we add into sign_file service when setting up the test environment and
-        skip the call to `exporter.get_sign_keys`, which can be tested
-        elsewhere, but not in this file.
-
-        Bear in mind that the exporter should know the sign_file user
-        credentials as they are used to request a token from sign_file
-        service.
-
-        pytest's tmp_path fixture can be of help to deal with tmp_files,
-        i.e.:
-        ```
-            def test_repomd_signer(tmp_path: Path):
-                tmp_file = tmp_path / "file.txt"
-                tmp_file.write_bytes(b"Hello world!")
-                ...
-        ```
-    """
+    reason="See https://github.com/AlmaLinux/build-system/issues/204"
 )
 def test_repomd_signer():
     exporter = Exporter(