centralize download and unzip logic into ruby libs

Author: ianfixes

lib/arduino_ci/arduino_downloader.rb

@@ -1,3 +1,7 @@
+require "fileutils"
+require 'open-uri'
+require 'zip'
+
 DOWNLOAD_ATTEMPTS = 3
 
 module ArduinoCI
@@ -74,14 +78,14 @@ def self.force_installed_executable
     # (for logging purposes)
     # @return [string]
     def downloader
-      self.class.must_implement(__method__)
+      "open-uri"
     end
 
     # The technology that will be used to extract the download
     # (for logging purposes)
     # @return [string]
     def extracter
-      self.class.must_implement(__method__)
+      "Zip"
     end
 
     # The URL of the desired IDE package (zip/tar/etc) for this platform
@@ -107,22 +111,44 @@ def self.force_install_location
       File.join(ENV['HOME'], 'arduino_ci_ide')
     end
 
-    # Download the package_url to package_file, and maybe print a line of dots......
+    # Download the package_url to package_file
     # @return [bool] whether successful
     def download
-      self.class.must_implement(__method__)
+      # Turned off ssl verification
+      # This should be acceptable because it won't happen on a user's machine, just CI
+
+      # define a progress-bar printer
+      chunk_size = 1024 * 1024 * 1024
+      total_size = 0
+      dots = 0
+      dot_printer = lambda do |size|
+        total_size += size
+        needed_dots = (total_size / chunk_size).to_i
+        unprinted_dots = needed_dots - dots
+        print("." * unprinted_dots) if unprinted_dots > 0
+        dots = needed_dots
+      end
+
+      open(package_url, ssl_verify_mode: 0, progress_proc: dot_printer) do |url|
+        File.open(package_file, 'wb') { |file| file.write(url.read) }
+      end
     end
 
     # Extract the package_file to extracted_file
     # @return [bool] whether successful
     def extract
-      self.class.must_implement(__method__)
+      Zip::File.open(package_file) do |zip|
+        zip.each do |file|
+          file.extract(file.name)
+        end
+      end
     end
 
     # Move the extracted package file from extracted_file to the force_install_location
     # @return [bool] whether successful
     def install
-      self.class.must_implement(__method__)
+      # Move only the content of the directory
+      FileUtils.mv extracted_file, self.class.force_install_location
     end
 
     # Forcibly install Arduino on linux from the web

lib/arduino_ci/arduino_downloader_linux.rb

@@ -1,18 +1,28 @@
-require "arduino_ci/arduino_downloader_posix"
+require "arduino_ci/arduino_downloader"
 
 USE_BUILDER = false
 
 module ArduinoCI
 
   # Manage the linux download & install of Arduino
-  class ArduinoDownloaderLinux < ArduinoDownloaderPosix
+  class ArduinoDownloaderLinux < ArduinoDownloader
 
     # The local filename of the desired IDE package (zip/tar/etc)
     # @return [string]
     def package_file
       "#{extracted_file}-linux64.tar.xz"
     end
 
+    # Make any preparations or run any checks prior to making changes
+    # @return [string] Error message, or nil if success
+    def prepare
+      reqs = [extracter]
+      reqs.each do |req|
+        return "#{req} does not appear to be installed!" unless Host.which(req)
+      end
+      nil
+    end
+
     # The technology that will be used to extract the download
     # (for logging purposes)
     # @return [string]

lib/arduino_ci/arduino_downloader_osx.rb

@@ -1,29 +1,16 @@
-require "arduino_ci/arduino_downloader_posix"
+require "arduino_ci/arduino_downloader"
 
 module ArduinoCI
 
   # Manage the OSX download & install of Arduino
-  class ArduinoDownloaderOSX < ArduinoDownloaderPosix
+  class ArduinoDownloaderOSX < ArduinoDownloader
 
     # The local filename of the desired IDE package (zip/tar/etc)
     # @return [string]
     def package_file
       "arduino-#{@desired_ide_version}-macosx.zip"
     end
 
-    # The technology that will be used to extract the download
-    # (for logging purposes)
-    # @return [string]
-    def extracter
-      "unzip"
-    end
-
-    # Extract the package_file to extracted_file
-    # @return [bool] whether successful
-    def extract
-      system(extracter, package_file)
-    end
-
     # The local file (dir) name of the extracted IDE package (zip/tar/etc)
     # @return [string]
     def extracted_file

lib/arduino_ci/arduino_downloader_posix.rb

@@ -39,8 +39,6 @@ def download
     def install
       # Move only the content of the directory
       FileUtils.mv extracted_file, self.class.force_install_location
-      # clean up the no longer required root extracted folder
-      FileUtils.rm_rf extracted_file
     end
 
     # The local filename of the desired IDE package (zip/tar/etc)
@@ -64,8 +62,6 @@ def extract
           file.extract(file.name)
         end
       end
-      # clean up the no longer required zip
-      FileUtils.rm_rf package_file
     end
 
     # The local file (dir) name of the extracted IDE package (zip/tar/etc)