Merge pull request #85 from IanLee1521/better-msg-on-download-error

Added extra context to error message when remote theme download fails
benbalter · Feb 19, 2021 · a179153 · a179153
2 parents bdcb1fc + 348627d
commit a179153
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/lib/jekyll-remote-theme/downloader.rb b/lib/jekyll-remote-theme/downloader.rb
@@ -64,7 +64,7 @@ def request
       def raise_unless_sucess(response)
         return if response.is_a?(Net::HTTPSuccess)
 
-        raise DownloadError, "#{response.code} - #{response.message}"
+        raise DownloadError, "#{response.code} - #{response.message} - Loading URL: #{zip_url}"
       end
 
       def enforce_max_file_size(size)

diff --git a/spec/jekyll-remote-theme/downloader_spec.rb b/spec/jekyll-remote-theme/downloader_spec.rb
@@ -69,7 +69,7 @@
       after { WebMock.allow_net_connect! }
 
       it "raises a DownloadError" do
-        msg = "404 - Not Found"
+        msg = "404 - Not Found - Loading URL: https://codeload.github.com/benbalter/_invalid_/zip/master"
         expect { subject.run }.to raise_error(Jekyll::RemoteTheme::DownloadError, msg)
       end
     end