Revert "[fix/ISSUE-80] Attempt to force download even if LOGIN_REQUIRED"

This reverts commit 879c59e.
azihassan · Aug 16, 2024 · 9883ee8 · 9883ee8
1 parent 5cfeeb2
commit 9883ee8
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -40,6 +40,7 @@ jobs:
     runs-on: ${{ matrix.os.name }}
     needs: unit_test
     continue-on-error: true
+    on: workflow_dispatch
 
     steps:
     - uses: actions/checkout@v1

diff --git a/source/parsers.d b/source/parsers.d
@@ -119,7 +119,7 @@ class SimpleYoutubeVideoURLExtractor : YoutubeVideoURLExtractor
         this.html = html;
         this.logger = logger;
         parser = createDocument(html);
-        //failIfUnplayable();
+        failIfUnplayable();
     }
 
     this(string html, string baseJS, StdoutLogger logger)
@@ -182,7 +182,6 @@ unittest
     assert(extractor.getID() == "Q_-p2q5FHy0");
 }
 
-/*
 unittest
 {
     import std.exception : collectExceptionMsg;
@@ -193,7 +192,6 @@ unittest
     string expectedExceptionMessage = "Video is unplayable because of status LOGIN_REQUIRED";
     assert(exceptionMessage == expectedExceptionMessage, "Expected message " ~ expectedExceptionMessage ~ " but got " ~ exceptionMessage);
 }
-*/
 
 enum AudioVisual : string
 {
@@ -281,7 +279,7 @@ class AdvancedYoutubeVideoURLExtractor : YoutubeVideoURLExtractor
         this.parser = createDocument(html);
         this.baseJS = baseJS;
         this.logger = logger;
-        //failIfUnplayable();
+        failIfUnplayable();
     }
 
     override string getURL(int itag, bool attemptDethrottle = false)