try out integration test

beetbox · Oct 1, 2024 · 2602482 · 2602482
1 parent 9399f66
commit 2602482
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 12 deletions.
diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml
@@ -45,6 +45,8 @@ jobs:
 
       - if: ${{ env.IS_MAIN_PYTHON == 'true' }}
         name: Test with coverage
+        env:
+          INTEGRATION_TEST: 1
         uses: liskin/gh-problem-matcher-wrap@v3
         with:
           linters: pytest

diff --git a/test/plugins/test_lyrics.py b/test/plugins/test_lyrics.py
@@ -14,7 +14,6 @@
 
 """Tests for the 'lyrics' plugin."""
 
-import os
 import re
 from functools import partial
 from http import HTTPStatus
@@ -34,12 +33,6 @@
 _p = pytest.param
 
 
-skip_ci = pytest.mark.skipif(
-    os.environ.get("GITHUB_ACTIONS") == "true",
-    reason="GitHub actions is on some form of Cloudflare blacklist",
-)
-
-
 class TestLyricsUtils:
     def test_search_artist(self):
         item = Item(artist="Alice ft. Bob", title="song")
@@ -276,13 +269,9 @@ def file_name(self):
                     "https://sweetslyrics.com/the-beatles/lady-madonna-lyrics",
                     "https://www.musica.com/letras.asp?letra=59862",
                     "https://www.lacoccinelle.net/259956-the-beatles-lady-madonna.html",
+                    "https://www.azlyrics.com/lyrics/beatles/ladymadonna.html",
                 )
             ),
-            pytest.param(
-                "Lady Madonna",
-                "https://www.azlyrics.com/lyrics/beatles/ladymadonna.html",
-                marks=skip_ci,
-            ),
             (
                 "Jazz'n'blues",
                 "https://www.lyricsontop.com/amy-winehouse-songs/jazz-n-blues-lyrics.html",  # noqa: E501