simple-images-downloader 1.0.3 → 1.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5d6712c3ea229e3cb09526ef0f0144f79fa9dcf1d7af6f20b23a83502ea34be1
-  data.tar.gz: b1147429efdf91682067decf309aec22050f2ed835b0b0261560f93fd6a63aa6
+  metadata.gz: 0e7adf857ef1d9f8490a079e5828b6025d0b44a21ec098bfc0012124c61223a9
+  data.tar.gz: 50175489068f5c9c7587b940d0690d335f8787f4e78c3f7ede9b26982b13b1c5
 SHA512:
-  metadata.gz: b3cf6db2633b3ebf0e094482079193fa2dbcb75e169c70ea3778e9c46b00abbdae4089378c36d8570630aa0698ca5343ee1a97e67f6e8e193c1d234c240bb35f
-  data.tar.gz: cd9645e49d0a1658c74a6acde0a5608bd5a79fea67acda6235057c018066aaee7b5352ca150a5bf77d926189cd41dfb43950e62f0257edab98f31fc25f7a3f1b
+  metadata.gz: e205d6d7feb52617f2d526765dd3b3f82509d5cfc2343ca90dbf966966784fa8099cae222904b245ff338d46f5f38883a648d50914c39ec7d3b285af0c6e61a6
+  data.tar.gz: 2660f410b9d48c4876e48fd387c69ad12707b29fef9e02ccb453331fc557c42249d0311da6f0df9eafe50e894ce23fa50c7260d2a6b4f091664ab3247b8dbcbe

data/CHANGELOG.md

@@ -1,6 +1,16 @@
+## 1.1.1 - 2023-09-30
+- Fixed setting of mime types from configuration. It was using default values instead of the ones set in the configuration
+- Added documentation to classes and methods
+
+## 1.1.0 - 2023-09-30
+- Add validation of downloaded content over path validation
+- Add ability to extend mime types from configuration
+- Removed zeitwerk
+- Refactored main interface
+
 ## 1.0.3 - 2023-09-27
 - Corrected destination validation in SimpleImagesDownloader::Dispenser
-- Corrected source file validation in SimpleImagesDownloader::SourceFiles
+- Corrected source file validation in SimpleImagesDownloader::SourceFile
 - Refactored
 - Fixed error message when destination is invalid
 

data/Gemfile.lock

@@ -1,8 +1,7 @@
 PATH
   remote: .
   specs:
-    simple-images-downloader (1.0.3)
-      zeitwerk (~> 2.4.0)
+    simple-images-downloader (1.1.1)
 
 GEM
   remote: https://rubygems.org/
@@ -76,7 +75,6 @@ GEM
       addressable (>= 2.3.6)
       crack (>= 0.3.2)
       hashdiff (>= 0.4.0, < 2.0.0)
-    zeitwerk (2.4.2)
 
 PLATFORMS
   ruby

data/README.md

@@ -33,7 +33,7 @@ Or install it yourself as:
 
 The gem is provided with executable.
 
-You can either run plain commmand with file containing list of url:
+You can either run plain command with file containing list of url:
 
 ```bash
 simple-images-downloader <path_to_file>
@@ -67,6 +67,7 @@ Downloading https://test-for-simple-images-downloader.s3.eu-central-1.amazonaws.
 Downloading is finished
 ```
 
+## Configuration
 **By default the images stored in a directory the script is run.**
 You can also configure the place where the images would be stored to:
 
@@ -78,6 +79,18 @@ You can also configure the place where the images would be stored to:
 [2] pry(main)* end
 ```
 
+**By default the gem downloads only images with mime types: image/avif, image/gif, image/apng, image/jpg, image/jpeg, image/png, image/svg+xml, image/webp.**
+You can also configure the mime types to download images with:
+
+
+```ruby
+[1] pry(main)> require 'simple_images_downloader'
+
+[2] pry(main)> SimpleImagesDownloader::Configuration.configure do  |config|
+[2] pry(main)*   # You must use RFC 1341 - MIME (Multipurpose Internet Mail Extensions) format
+[2] pry(main)*   config.valid_mime_types = %w(image/png image/jpg image/jpeg)
+[2] pry(main)* end
+```
 ## Development
 
 After checking out the repo, run `bin/setup` to install dependencies. Then, run `rake spec` to run the tests. You can also run `bin/console` for an interactive prompt that will allow you to experiment.

data/lib/simple_images_downloader/client.rb

@@ -1,11 +1,21 @@
 # frozen_string_literal: true
 
 module SimpleImagesDownloader
+  # add documentation to this class using YARD
+
+  # Client class
+  # Responsible for opening the URI and handling errors
+  #
+  # @example
+  #   SimpleImagesDownloader::Client.new.open(uri)
+  #
   class Client
     def initialize(options = Configuration::REQUEST_OPTIONS)
       @options = options
     end
 
+    # @param uri [URI] URI object
+    # @return [StringIO] StringIO object
     def open(uri)
       uri.open(@options)
     rescue OpenURI::HTTPRedirect

data/lib/simple_images_downloader/configuration.rb

@@ -1,11 +1,25 @@
 # frozen_string_literal: true
 
 module SimpleImagesDownloader
+  # Configuration class
+  # @example
+  #   SimpleImagesDownloader::Configuration.configure do |config|
+  #     config.destination = './images'
+  #     config.valid_mime_types = ['image/jpeg', 'image/png']
+  #   end
+  #
+  #   SimpleImagesDownloader::Configuration.destination
+  #   # => './images'
+  #
+  #   SimpleImagesDownloader::Configuration.valid_mime_types
+  #   # => ['image/jpeg', 'image/png']
+  #
   class Configuration
     include Singleton
 
     ACCESSORS = %i[
       destination
+      valid_mime_types
     ].freeze
 
     REQUEST_OPTIONS = {
@@ -15,12 +29,50 @@ module SimpleImagesDownloader
       read_timeout: 30
     }.freeze
 
+    # https://developer.mozilla.org/en-US/docs/Web/Media/Formats/Image_types#common_image_file_types
+    DEFAULT_VALID_MIME_TYPES = [
+      'image/avif',
+      'image/gif',
+      'image/apng',
+      'image/jpg',
+      'image/jpeg',
+      'image/png',
+      'image/svg+xml',
+      'image/webp'
+    ].freeze
+
+    DEFAULT_DESTINATION = './'
+
     attr_accessor(*ACCESSORS)
 
     def initialize
-      @destination = './'
+      @destination = DEFAULT_DESTINATION
+      @valid_mime_types = DEFAULT_VALID_MIME_TYPES
+    end
+
+    # Allows to set valid_mime_types to check against allowed mime types
+    #
+    # @param value [Array] Array of valid mime types followed by
+    # RFC 1341 - MIME (Multipurpose Internet Mail Extensions) format
+    # @raise [BaseError] if value is not an Array
+    #
+    def valid_mime_types=(value)
+      raise Errors::BaseError, 'valid_mime_types must be an array' unless value.is_a?(Array)
+
+      @valid_mime_types = value
     end
 
+    #### Configurable options
+    #
+    # destination: String, default: './', description: 'Destination folder to put downloaded images'
+    # valid_mime_types: Array,
+    # default: [
+    #  'image/avif', 'image/gif', 'image/apng', 'image/jpg',
+    #  'image/jpeg', 'image/png', 'image/svg+xml', 'image/webp'
+    # ], description: 'Valid mime types to download'
+    #
+    # @yield [config] Yields the Configuration object to the block
+    #
     def self.configure
       yield instance
     end

data/lib/simple_images_downloader/dispenser.rb

@@ -1,20 +1,34 @@
 # frozen_string_literal: true
 
 module SimpleImagesDownloader
+  # Dispenser class
+  # Responsible for moving tempfile to destination directory
+  #
+  # @example
+  #   SimpleImagesDownloader::Dispenser.new(tempfile, 'https://example.com/image.jpg').place
+  #
   class Dispenser
     extend Forwardable
     include Validatable
 
     def_delegator 'SimpleImagesDownloader::Configuration', :destination, :destination_dir
 
+    # @param source [Tempfile] Tempfile object
+    # @param remote_path [String] original path of the image from input of SimpleImagesDownloader module
+    # @param validators [Array] array of validators for validating the destination directory.
+    # Default: [DestinationValidator.new]
     def initialize(source, remote_path, validators = [DestinationValidator.new])
       @source      = source
       @remote_path = remote_path
       @validators  = validators
     end
 
+    # Moves tempfile to destination directory
+    #
+    # @raise [Errors::BadDestination] if destination directory is not valid
+    # @see DestinationValidator
     def place
-      validate!(destination_dir)
+      validate!({ path: destination_dir })
 
       FileUtils.mv @source, target
     end

data/lib/simple_images_downloader/downloader.rb

@@ -1,23 +1,48 @@
 # frozen_string_literal: true
 
 module SimpleImagesDownloader
+  # Downloader class
+  # Responsible for downloading images from URI and placing them to the destination folder
+  # allows to use custom client and validators for downloading
+  #
+  # @example
+  #   SimpleImagesDownloader::Downloader.new(uri).download
+  #
   class Downloader
-    def initialize(uri)
-      @uri = uri
+    include Validatable
+
+    # @param uri [URI] URI object from which image will be downloaded
+    # @param client [Client] Client object for opening the URI. Default: Client.new
+    # @param validators [Array] array of validators for validating the response. Default: [MimeTypeValidator.new]
+    def initialize(uri, client = Client.new, validators = [MimeTypeValidator.new])
+      @uri        = uri
+      @client     = client
+      @validators = validators
     end
 
+    # Downloads image from URI and places it to the destination folder
+    #
+    # @raise [Errors::EmptyResponse] if response is empty
+    # @raise [Errors::BadMimeType] if response is not an image
+    # @see Errors module
+    # @see MimeTypeValidator
     def download
       puts "Downloading #{@uri}"
 
-      io = Client.new.open(@uri)
+      io = @client.open(@uri)
+
+      raise Errors::EmptyResponse, @uri if io.nil?
+
+      validate!({ path: @uri.to_s, io: io })
 
-      downloaded_file = StringioToTempfile.convert(io) unless io.nil?
+      tempfile = StringioToTempfile.convert(io)
 
-      Dispenser.new(downloaded_file, @uri.path).place
+      Dispenser.new(tempfile, @uri.path).place
 
       puts 'Downloading is finished'
     ensure
-      downloaded_file&.close
+      io&.close
+      tempfile&.close
     end
   end
 end

data/lib/simple_images_downloader/errors.rb

@@ -1,6 +1,9 @@
 # frozen_string_literal: true
 
 module SimpleImagesDownloader
+  # Errors module
+  # Responsible for storing all errors
+  #
   module Errors
     class BaseError < StandardError; end
 
@@ -25,9 +28,9 @@ module SimpleImagesDownloader
       end
     end
 
-    class MissingImageInPath < BaseError
-      def initialize(path)
-        message = "The path doesn't contain image #{path}"
+    class BadMimeType < BaseError
+      def initialize(path, mime_type)
+        message = "The image with path: #{path} has wrong mime type #{mime_type}"
         super(message)
       end
     end
@@ -66,5 +69,12 @@ module SimpleImagesDownloader
         super(message)
       end
     end
+
+    class EmptyResponse < BaseError
+      def initialize(uri)
+        message = "Nothing returned from request #{uri}"
+        super(message)
+      end
+    end
   end
 end

data/lib/simple_images_downloader/line.rb

@@ -1,20 +1,24 @@
 # frozen_string_literal: true
 
 module SimpleImagesDownloader
+  # Line class
+  # Responsible for parsing the string into URI object
+  #
+  # @example
+  #   SimpleImagesDownloader::Line.new('https://example.com/image.jpg').uri
+  #   # => #<URI::HTTPS https://example.com/image.jpg>
+  #
   class Line
-    include Validatable
-
-    def initialize(string, validators = [ImagePathValidator.new])
-      @string     = string
-      @validators = validators
+    def initialize(string)
+      @string = string
     end
 
+    # @return [URI] URI object
+    # @raise [Errors::BadUrl] if string is not a valid URI
+    # @see Errors::BadUrl
+    #
     def uri
-      parsed_uri = URI.parse(@string)
-
-      validate!(@string)
-
-      parsed_uri
+      URI.parse(@string)
     rescue URI::Error
       raise Errors::BadUrl, @string
     end

data/lib/simple_images_downloader/runner.rb

@@ -1,7 +1,14 @@
 # frozen_string_literal: true
 
 module SimpleImagesDownloader
+  # Runner class
+  # Responsible for invoking interface to download images from file
+  #
+  # @example
+  #   SimpleImagesDownloader::Runner.invoke('./urls.txt')
+  #
   class Runner
+    # Allows to invoke interface to download images from file
     def self.invoke
       raise SimpleImagesDownloader::Errors::MissingFileArgumentError if ARGV.size.zero?
 

data/lib/simple_images_downloader/source_file.rb

@@ -1,16 +1,28 @@
 # frozen_string_literal: true
 
 module SimpleImagesDownloader
+  # SourceFile class
+  # Responsible for opening the file of URLs and validating it
+  #
+  # @example
+  #   SimpleImagesDownloader::SourceFile.new('./urls.txt').each_line do |line|
+  #     puts line
+  #   end
+  #
   class SourceFile
     include Validatable
 
+    # @param path [String] path to file
+    # @param validators [Array] array of validators
     def initialize(path, validators = [FilePersistanceValidator.new, FileAccessibilityValidator.new])
       @path       = path
       @validators = validators
     end
 
+    # @yield [line] passes each line of file to block
+    # @yieldparam line [String] line of file
     def each_line(&block)
-      validate!(@path)
+      validate!({ path: @path })
 
       begin
         file.each(chomp: true, &block)

data/lib/simple_images_downloader/strategies/from_file_strategy.rb

@@ -0,0 +1,24 @@
+# frozen_string_literal: true
+
+module SimpleImagesDownloader
+  module Strategies
+    class FromFileStrategy < Strategy
+      def initialize(path)
+        super
+        @path = path
+      end
+
+      def process
+        source_file = SourceFile.new(@path)
+
+        source_file.each_line do |line|
+          uri = Line.new(line).uri
+          Downloader.new(uri).download
+        rescue Errors::BaseError => e
+          puts e.message
+          next
+        end
+      end
+    end
+  end
+end

data/lib/simple_images_downloader/strategies/from_url_strategy.rb

@@ -0,0 +1,18 @@
+# frozen_string_literal: true
+
+module SimpleImagesDownloader
+  module Strategies
+    class FromUrlStrategy < Strategy
+      def initialize(url)
+        super
+        @url = url
+      end
+
+      def process
+        uri = Line.new(@url).uri
+
+        Downloader.new(uri).download
+      end
+    end
+  end
+end

data/lib/simple_images_downloader/strategies/strategy.rb

@@ -0,0 +1,14 @@
+# frozen_string_literal: true
+
+module SimpleImagesDownloader
+  module Strategies
+    class Strategy
+      def initialize(_)
+      end
+
+      def process
+        raise NotImplementedError, 'must be implemented in subclass'
+      end
+    end
+  end
+end

data/lib/simple_images_downloader/stringio_to_tempfile.rb

@@ -1,16 +1,23 @@
 # frozen_string_literal: true
 
 module SimpleImagesDownloader
+  # StringIOToTempfile module
+  # Responsible for converting StringIO to Tempfile
+  #
+  # @example
+  #   SimpleImagesDownloader::StringioToTempfile.convert(stringio)
+  #  # => #<Tempfile:0x00007f9b9c0b3a38>
+  #
   module StringioToTempfile
     module_function
 
+    # @param stringio [StringIO] StringIO object
+    # @return [Tempfile] Tempfile object
     def convert(stringio)
       tempfile = Tempfile.new(binmode: true)
 
       IO.copy_stream(stringio, tempfile)
 
-      stringio.close
-
       OpenURI::Meta.init tempfile, stringio
 
       tempfile

data/lib/simple_images_downloader/validatable/destination_validator.rb

@@ -2,10 +2,21 @@
 
 module SimpleImagesDownloader
   module Validatable
+    # DestinationValidator class
+    # Responsible for validating destination path
+    #
+    # @example
+    #   SimpleImagesDownloader::DestinationValidator.new.validate({ path: './images' })
+    #
     class DestinationValidator < Validator
-      def validate(destination_dir)
-        raise Errors::DestinationIsNotDirectory, destination_dir unless File.directory?(destination_dir)
-        raise Errors::DestinationIsNotWritable, destination_dir unless File.writable?(destination_dir)
+      # @param options [Hash] hash with path to destination directory
+      # @raise [Errors::DestinationIsNotDirectory] if destination is not directory
+      # @raise [Errors::DestinationIsNotWritable] if destination is not writable
+      def validate(options)
+        path = options[:path]
+
+        raise Errors::DestinationIsNotDirectory, path unless File.directory?(path)
+        raise Errors::DestinationIsNotWritable, path unless File.writable?(path)
       end
     end
   end

data/lib/simple_images_downloader/validatable/file_accessibility_validator.rb

@@ -2,11 +2,19 @@
 
 module SimpleImagesDownloader
   module Validatable
+    # FileAccessibilityValidator class
+    # Responsible for validating file accessibility
+    #
+    # @example
+    #   SimpleImagesDownloader::FileAccessibilityValidator.new.validate({ path: './urls.txt' })
+    #
     class FileAccessibilityValidator < Validator
-      def validate(path)
-        return if File.readable?(path)
+      # @param options [Hash] hash with path to file
+      # @raise [Errors::PermissionsError] if file is not readable
+      def validate(options)
+        return if File.readable?(options[:path])
 
-        raise Errors::PermissionsError, path
+        raise Errors::PermissionsError, options[:path]
       end
     end
   end

data/lib/simple_images_downloader/validatable/file_persistance_validator.rb

@@ -2,11 +2,19 @@
 
 module SimpleImagesDownloader
   module Validatable
+    # FilePersistanceValidator class
+    # Responsible for validating file persistance
+    #
+    # @example
+    #   SimpleImagesDownloader::FilePersistanceValidator.new.validate({ path: './urls.txt' })
+    #
     class FilePersistanceValidator < Validator
-      def validate(path)
-        return if File.exist?(path)
+      # @param options [Hash] hash with path to file
+      # @raise [Errors::MissingFileError] if file does not exist
+      def validate(options)
+        return if File.exist?(options[:path])
 
-        raise Errors::MissingFileError, path
+        raise Errors::MissingFileError, options[:path]
       end
     end
   end

data/lib/simple_images_downloader/validatable/mime_type_validator.rb

@@ -0,0 +1,74 @@
+# frozen_string_literal: true
+
+module SimpleImagesDownloader
+  module Validatable
+    # MimeTypeValidator class
+    # Responsible for validating mime type of file
+    #
+    # @example
+    #   SimpleImagesDownloader::MimeTypeValidator.new.validate({ path: './image.jpg', io: StringIO.new })
+    #
+    class MimeTypeValidator < Validator
+      extend Forwardable
+
+      def_delegator 'SimpleImagesDownloader::Configuration', :valid_mime_types, :valid_mime_types
+
+      # Validates mime type of file. The mime types are taken from file Configuration
+      #
+      # @param options [Hash] hash with path to file and io object. Example: { path: './image.jpg', io: StringIO }
+      # @raise [Errors::BadMimeType] if mime type is not valid
+      # @see Configuration::DEFAULT_VALID_MIME_TYPES to see default valid mime types or add your own
+      def validate(options)
+        path = options[:path]
+        io   = options[:io]
+
+        mime_type = mime_type_of(io)
+
+        return if SimpleImagesDownloader::Configuration.valid_mime_types.include?(mime_type)
+
+        raise Errors::BadMimeType.new(path, mime_type)
+      end
+
+      private
+
+      # Returns mime type of file. Uses UNIX file command
+      #
+      # @see https://github.com/shrinerb/shrine/blob/master/lib/shrine/plugins/determine_mime_type.rb#L94
+      # @param io [IO] io object
+      # @return [String] mime type of file
+      def mime_type_of(io)
+        Open3.popen3('file --mime-type --brief -') do |stdin, stdout, stderr, thread|
+          copy_stream(from: io, to: stdin.binmode)
+
+          io.rewind
+          stdin.close
+
+          status = thread.value
+
+          validate_thread_status(status)
+          $stderr.print(stderr.read)
+
+          output = stdout.read.strip
+
+          validate_command_output(output)
+
+          output
+        end
+      end
+
+      def copy_stream(from:, to:)
+        IO.copy_stream(from, to)
+      rescue Errno::EPIPE # rubocop:disable Lint/SuppressedException
+      end
+
+      def validate_thread_status(status)
+        raise Errors::BaseError, "file command failed to spawn: #{stderr.read}" if status.nil?
+        raise Errors::BaseError, "file command failed: #{stderr.read}" unless status.success?
+      end
+
+      def validate_command_output(output)
+        raise Errors::BaseError, "file command failed: #{output}" if output.include?('cannot open')
+      end
+    end
+  end
+end

data/lib/simple_images_downloader/validatable/validator.rb

@@ -2,6 +2,8 @@
 
 module SimpleImagesDownloader
   module Validatable
+    # Validator class
+    # Responsible for defining interface for validators
     class Validator
       def validate(_value)
         raise NotImplementedError, 'must be implemented in subclass'

data/lib/simple_images_downloader/validatable.rb

@@ -1,7 +1,26 @@
 # frozen_string_literal: true
 
 module SimpleImagesDownloader
+  # Validatable module
+  # Responsible for validating the object using array of validators
+  #
+  # @example
+  #   class SourceFile
+  #     include Validatable
+  #
+  #     def initialize(path, validators = [FilePersistanceValidator.new, FileAccessibilityValidator.new])
+  #       @path       = path
+  #       @validators = validators
+  #     end
+  #
+  #     def each_line(&block)
+  #       validate!({ path: @path })
+  #       # ...
+  #     end
+  #   end
   module Validatable
+    # @param value [Object] value to validate
+    # @raise [Errors::<Particular>Error] if value is not valid
     def validate!(value)
       (@validators ||= []).each { |validator| validator.validate(value) }
     end

data/lib/simple_images_downloader/version.rb

@@ -1,5 +1,6 @@
 # frozen_string_literal: true
 
 module SimpleImagesDownloader
-  VERSION = '1.0.3'
+  # gem version
+  VERSION = '1.1.1'
 end

data/lib/simple_images_downloader.rb

@@ -4,29 +4,52 @@ require 'open-uri'
 require 'tempfile'
 require 'forwardable'
 require 'singleton'
-require 'zeitwerk'
-
-loader = Zeitwerk::Loader.for_gem
-loader.setup
+require 'open3'
+require_relative 'simple_images_downloader/version'
+require_relative 'simple_images_downloader/errors'
+require_relative 'simple_images_downloader/configuration'
+require_relative 'simple_images_downloader/validatable'
+require_relative 'simple_images_downloader/validatable/validator'
+require_relative 'simple_images_downloader/validatable/destination_validator'
+require_relative 'simple_images_downloader/validatable/file_accessibility_validator'
+require_relative 'simple_images_downloader/validatable/file_persistance_validator'
+require_relative 'simple_images_downloader/validatable/mime_type_validator'
+require_relative 'simple_images_downloader/stringio_to_tempfile'
+require_relative 'simple_images_downloader/client'
+require_relative 'simple_images_downloader/source_file'
+require_relative 'simple_images_downloader/runner'
+require_relative 'simple_images_downloader/line'
+require_relative 'simple_images_downloader/dispenser'
+require_relative 'simple_images_downloader/strategies/strategy'
+require_relative 'simple_images_downloader/downloader'
+require_relative 'simple_images_downloader/strategies/from_file_strategy'
+require_relative 'simple_images_downloader/strategies/from_url_strategy'
 
+# SimpleImagesDownloader module
+# It is a main module of the gem
+# It is responsible for providing interface to download images from file or url
+#
+# @example
+#   SimpleImagesDownloader.from_file('./urls.txt')
+#   SimpleImagesDownloader.from_url('https://example.com/image.jpg')
+#
 module SimpleImagesDownloader
+  # Downloads images from file by taking urls from it. Places images to the destination folder set in configuration
+  #
+  # @param path [String] path to file with urls
   def self.from_file(path)
-    source_file = SourceFile.new(path)
-
-    source_file.each_line do |line|
-      uri = Line.new(line).uri
-      Downloader.new(uri).download
-    rescue Errors::BaseError => e
-      puts e.message
-      next
-    end
+    SimpleImagesDownloader::Strategies::FromFileStrategy.new(path).process
   end
 
+  # Downloads image from url. Places image to the destination folder set in configuration
+  #
+  # @param url [String] url of image
   def self.from_url(url)
-    uri = Line.new(url).uri
-    Downloader.new(uri).download
+    SimpleImagesDownloader::Strategies::FromUrlStrategy.new(url).process
   end
 
+  # Returns root path of the gem. It is used for testing purposes
+  # @return [String] root path of the gem
   def self.root
     File.dirname __dir__
   end

data/rubocop.yml

@@ -69,6 +69,9 @@ Layout/LineLength:
 Metrics/BlockLength:
   Enabled: false
 
+Metrics/MethodLength:
+ Max: 11
+
 Layout/EndAlignment:
   EnforcedStyleAlignWith: variable
   SupportedStylesAlignWith:

data/simple_images_downloader.gemspec

@@ -30,8 +30,6 @@ Gem::Specification.new do |spec|
   spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
   spec.require_paths = ['lib']
 
-  spec.add_runtime_dependency 'zeitwerk', '~> 2.4.0'
-
   spec.add_development_dependency 'faker', '~> 2.14'
   spec.add_development_dependency 'pry', '~> 0.13.1'
   spec.add_development_dependency 'rake', '~> 12.0'

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: simple-images-downloader
 version: !ruby/object:Gem::Version
-  version: 1.0.3
+  version: 1.1.1
 platform: ruby
 authors:
 - IlkhamGaysin
@@ -10,20 +10,6 @@ bindir: exe
 cert_chain: []
 date: 2023-09-30 00:00:00.000000000 Z
 dependencies:
-- !ruby/object:Gem::Dependency
-  name: zeitwerk
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 2.4.0
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 2.4.0
 - !ruby/object:Gem::Dependency
   name: faker
   requirement: !ruby/object:Gem::Requirement
@@ -214,12 +200,15 @@ files:
 - lib/simple_images_downloader/line.rb
 - lib/simple_images_downloader/runner.rb
 - lib/simple_images_downloader/source_file.rb
+- lib/simple_images_downloader/strategies/from_file_strategy.rb
+- lib/simple_images_downloader/strategies/from_url_strategy.rb
+- lib/simple_images_downloader/strategies/strategy.rb
 - lib/simple_images_downloader/stringio_to_tempfile.rb
 - lib/simple_images_downloader/validatable.rb
 - lib/simple_images_downloader/validatable/destination_validator.rb
 - lib/simple_images_downloader/validatable/file_accessibility_validator.rb
 - lib/simple_images_downloader/validatable/file_persistance_validator.rb
-- lib/simple_images_downloader/validatable/image_path_validator.rb
+- lib/simple_images_downloader/validatable/mime_type_validator.rb
 - lib/simple_images_downloader/validatable/validator.rb
 - lib/simple_images_downloader/version.rb
 - rubocop.yml

data/lib/simple_images_downloader/validatable/image_path_validator.rb

@@ -1,17 +0,0 @@
-# frozen_string_literal: true
-
-module SimpleImagesDownloader
-  module Validatable
-    class ImagePathValidator < Validator
-      VALID_EXTENSIONS = %w[.png .jpg .gif .jpeg].freeze
-
-      def validate(path)
-        extension = File.extname(path)
-
-        return if VALID_EXTENSIONS.include?(extension)
-
-        raise Errors::MissingImageInPath, path
-      end
-    end
-  end
-end