simple-images-downloader 1.0.2 → 1.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: eaab3c2b2fd909ddb983e0f23b20bb4361c41288ea834ab698b972c42a3b211b
-  data.tar.gz: ea9be3356750b3ec278b01f439eac533c12dc757ab9c0fb26496e7b90a954302
+  metadata.gz: 902c3b0e5fe1566296ac139123707162e33ded87accc9ff5705231fb3fa96f4f
+  data.tar.gz: c2812c8d700ca81d1f29b6a29d0063aef74b9dfd3162c652453733870b8073fe
 SHA512:
-  metadata.gz: a127db82c6529f8c592f1a85f88f8cc7742aa5ea175a18cdca895e7f4e17a08b14213713aa1b5030cdf44fda1ddd785c7b600f7f7ad69854d09dc5179e1ba708
-  data.tar.gz: 32538f8dd10a8d7d3eca94819cac12a2170364b11a612b7a555cb3898eba95e19628ea8726ab95e2cbc86e5cac65dfbbab128461c5e54cae01b37a3354cfcc61
+  metadata.gz: 5501d04aa338c0f992d391d94a5b29e8b63a410938c7a8f31941ed3178b2efa63a6ef84b9bb419f87278880197498367eeb235dea017509c57c708f758f7b4ab
+  data.tar.gz: 26ba7c6dbd4cfdf1e34750ce6b3f7e99650511ba56597a7c8ab1ee7950c210f7919f0a1104053fcc6195bec822b215678e9c8bc5015fee9d6fc8ae6eb27612e7

data/CHANGELOG.md

@@ -1,3 +1,15 @@
+## 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::SourceFile
+- Refactored
+- Fixed error message when destination is invalid
+
 ## 1.0.2 - 2020-09-25
 - Refactored SimpleImagesDownloader::Dispenser keeping the same behaviour
 - Updated README

data/Gemfile.lock

@@ -1,8 +1,7 @@
 PATH
   remote: .
   specs:
-    simple-images-downloader (1.0.2)
-      zeitwerk (~> 2.4.0)
+    simple-images-downloader (1.1.0)
 
 GEM
   remote: https://rubygems.org/
@@ -10,7 +9,6 @@ GEM
     addressable (2.7.0)
       public_suffix (>= 2.0.2, < 5.0)
     ast (2.4.1)
-    byebug (11.1.3)
     coderay (1.1.3)
     concurrent-ruby (1.1.7)
     crack (0.4.4)
@@ -29,9 +27,6 @@ GEM
     pry (0.13.1)
       coderay (~> 1.1)
       method_source (~> 1.0)
-    pry-byebug (3.9.0)
-      byebug (~> 11.0)
-      pry (~> 0.13.0)
     public_suffix (4.0.6)
     rainbow (3.0.0)
     rake (12.3.3)
@@ -80,7 +75,6 @@ GEM
       addressable (>= 2.3.6)
       crack (>= 0.3.2)
       hashdiff (>= 0.4.0, < 2.0.0)
-    zeitwerk (2.4.0)
 
 PLATFORMS
   ruby
@@ -88,7 +82,6 @@ PLATFORMS
 DEPENDENCIES
   faker (~> 2.14)
   pry (~> 0.13.1)
-  pry-byebug (~> 3.9.0)
   rake (~> 12.0)
   rspec (~> 3.0)
   rspec_junit_formatter (~> 0.4.1)
@@ -104,4 +97,4 @@ RUBY VERSION
    ruby 2.7.1p83
 
 BUNDLED WITH
-   2.1.4
+   2.2.19

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

@@ -0,0 +1,17 @@
+# frozen_string_literal: true
+
+module SimpleImagesDownloader
+  class Client
+    def initialize(options = Configuration::REQUEST_OPTIONS)
+      @options = options
+    end
+
+    def open(uri)
+      uri.open(@options)
+    rescue OpenURI::HTTPRedirect
+      raise Errors::RedirectError, uri
+    rescue OpenURI::HTTPError
+      raise Errors::ConnectionError, uri
+    end
+  end
+end

data/lib/simple_images_downloader/configuration.rb

@@ -6,12 +6,41 @@ module SimpleImagesDownloader
 
     ACCESSORS = %i[
       destination
+      valid_mime_types
     ].freeze
 
+    REQUEST_OPTIONS = {
+      'User-Agent' => "SimpleImagesDownloader/#{SimpleImagesDownloader::VERSION}",
+      redirect: false,
+      open_timeout: 30,
+      read_timeout: 30
+    }.freeze
+
+    # https://developer.mozilla.org/en-US/docs/Web/Media/Formats/Image_types#common_image_file_types
+    DEFAULT_VALID_MIME_TYPES_MAP = {
+      'image/avif' => true,
+      'image/gif' => true,
+      'image/apng' => true,
+      'image/jpg' => true,
+      'image/jpeg' => true,
+      'image/png' => true,
+      'image/svg+xml' => true,
+      'image/webp' => true
+    }.freeze
+
+    DEFAULT_DESTINATION = './'
+
     attr_accessor(*ACCESSORS)
 
     def initialize
-      @destination = './'
+      @destination = DEFAULT_DESTINATION
+      @valid_mime_types = DEFAULT_VALID_MIME_TYPES_MAP.keys
+    end
+
+    def valid_mime_types=(value)
+      raise BaseError, 'valid_mime_types must be an array' unless value.is_a?(Array)
+
+      @valid_mime_types = value
     end
 
     def self.configure

data/lib/simple_images_downloader/dispenser.rb

@@ -3,28 +3,30 @@
 module SimpleImagesDownloader
   class Dispenser
     extend Forwardable
+    include Validatable
 
     def_delegator 'SimpleImagesDownloader::Configuration', :destination, :destination_dir
 
-    def initialize(source, remote_path)
+    def initialize(source, remote_path, validators = [DestinationValidator.new])
       @source      = source
       @remote_path = remote_path
+      @validators  = validators
     end
 
     def place
-      raise Errors::DestinationIsNotWritable, destination unless File.writable?(destination_dir)
+      validate!({ path: destination_dir })
 
-      FileUtils.mv @source, destination
+      FileUtils.mv @source, target
     end
 
-    def destination
-      @destination ||= destination_dir + file_name
+    def target
+      @target ||= destination_dir + file_name
     end
 
     def file_name
       @file_name ||= File.basename(@source) + File.extname(@remote_path)
     end
 
-    private :destination_dir, :destination, :file_name
+    private :destination_dir, :target, :file_name
   end
 end

data/lib/simple_images_downloader/downloader.rb

@@ -2,37 +2,31 @@
 
 module SimpleImagesDownloader
   class Downloader
-    REQUEST_OPTIONS = {
-      'User-Agent' => "SimpleImagesDownloader/#{SimpleImagesDownloader::VERSION}",
-      redirect: false,
-      open_timeout: 30,
-      read_timeout: 30
-    }.freeze
-
-    def initialize(uri)
-      @uri = uri
+    include Validatable
+
+    def initialize(uri, client = Client.new, validators = [MimeTypeValidator.new])
+      @uri        = uri
+      @client     = client
+      @validators = validators
     end
 
     def download
       puts "Downloading #{@uri}"
 
-      @downloaded_file = StringioToTempfile.convert(downloaded_file) if downloaded_file.is_a?(StringIO)
+      io = @client.open(@uri)
 
-      Dispenser.new(downloaded_file, @uri.path).place
+      raise Errors::EmptyResponse, @uri if io.nil?
 
-      puts 'Downloading is finished'
-    ensure
-      downloaded_file.close
-    end
+      validate!({ path: @uri.to_s, io: io })
 
-    private
+      tempfile = StringioToTempfile.convert(io)
 
-    def downloaded_file
-      @downloaded_file ||= @uri.open(REQUEST_OPTIONS)
-    rescue OpenURI::HTTPRedirect
-      raise Errors::RedirectError, @uri
-    rescue OpenURI::HTTPError
-      raise Errors::ConnectionError, @uri
+      Dispenser.new(tempfile, @uri.path).place
+
+      puts 'Downloading is finished'
+    ensure
+      io&.close
+      tempfile&.close
     end
   end
 end

data/lib/simple_images_downloader/errors.rb

@@ -25,9 +25,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
@@ -52,5 +52,26 @@ module SimpleImagesDownloader
         super(message)
       end
     end
+
+    class DestinationIsNotDirectory < BaseError
+      def initialize(path)
+        message = "The destination - #{path} is not directory"
+        super(message)
+      end
+    end
+
+    class PermissionsError < BaseError
+      def initialize(path)
+        message = "Couldn't read file #{path} due to permissions error"
+        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

@@ -7,11 +7,7 @@ module SimpleImagesDownloader
     end
 
     def uri
-      parsed_uri = URI.parse(@string)
-
-      ImagePathValidator.new(@string).validate
-
-      parsed_uri
+      URI.parse(@string)
     rescue URI::Error
       raise Errors::BadUrl, @string
     end

data/lib/simple_images_downloader/source_file.rb

@@ -2,13 +2,15 @@
 
 module SimpleImagesDownloader
   class SourceFile
-    def initialize(path, validator = nil)
-      @path      = path
-      @validator = validator || SimpleImagesDownloader::FilePersistanceValidator.new(path)
+    include Validatable
+
+    def initialize(path, validators = [FilePersistanceValidator.new, FileAccessibilityValidator.new])
+      @path       = path
+      @validators = validators
     end
 
     def each_line(&block)
-      @validator.validate
+      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

@@ -9,8 +9,6 @@ module SimpleImagesDownloader
 
       IO.copy_stream(stringio, tempfile)
 
-      stringio.close
-
       OpenURI::Meta.init tempfile, stringio
 
       tempfile

data/lib/simple_images_downloader/validatable/destination_validator.rb

@@ -0,0 +1,14 @@
+# frozen_string_literal: true
+
+module SimpleImagesDownloader
+  module Validatable
+    class DestinationValidator < Validator
+      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
+end

data/lib/simple_images_downloader/validatable/file_accessibility_validator.rb

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+
+module SimpleImagesDownloader
+  module Validatable
+    class FileAccessibilityValidator < Validator
+      def validate(options)
+        return if File.readable?(options[:path])
+
+        raise Errors::PermissionsError, options[:path]
+      end
+    end
+  end
+end

data/lib/simple_images_downloader/validatable/file_persistance_validator.rb

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+
+module SimpleImagesDownloader
+  module Validatable
+    class FilePersistanceValidator < Validator
+      def validate(options)
+        return if File.exist?(options[:path])
+
+        raise Errors::MissingFileError, options[:path]
+      end
+    end
+  end
+end

data/lib/simple_images_downloader/validatable/mime_type_validator.rb

@@ -0,0 +1,59 @@
+# frozen_string_literal: true
+
+module SimpleImagesDownloader
+  module Validatable
+    class MimeTypeValidator < Validator
+      extend Forwardable
+
+      def_delegator 'SimpleImagesDownloader::Configuration', :valid_mime_types, :valid_mime_types
+
+      def validate(options)
+        path = options[:path]
+        io   = options[:io]
+
+        mime_type = mime_type_of(io)
+
+        return if Configuration::DEFAULT_VALID_MIME_TYPES_MAP[mime_type]
+
+        raise Errors::BadMimeType.new(path, mime_type)
+      end
+
+      private
+
+      # Taken from Shrine https://github.com/shrinerb/shrine/blob/master/lib/shrine/plugins/determine_mime_type.rb#L94
+      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

@@ -0,0 +1,11 @@
+# frozen_string_literal: true
+
+module SimpleImagesDownloader
+  module Validatable
+    class Validator
+      def validate(_value)
+        raise NotImplementedError, 'must be implemented in subclass'
+      end
+    end
+  end
+end

data/lib/simple_images_downloader/validatable.rb

@@ -0,0 +1,9 @@
+# frozen_string_literal: true
+
+module SimpleImagesDownloader
+  module Validatable
+    def validate!(value)
+      (@validators ||= []).each { |validator| validator.validate(value) }
+    end
+  end
+end

data/lib/simple_images_downloader/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module SimpleImagesDownloader
-  VERSION = '1.0.2'
+  VERSION = '1.1.0'
 end

data/lib/simple_images_downloader.rb

@@ -4,27 +4,34 @@ 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'
 
 module SimpleImagesDownloader
   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
 
   def self.from_url(url)
-    uri = Line.new(url).uri
-    Downloader.new(uri).download
+    SimpleImagesDownloader::Strategies::FromUrlStrategy.new(url).process
   end
 
   def self.root

data/rubocop.yml

@@ -33,7 +33,7 @@ Style/Documentation:
   Enabled: false
 
 Style/MethodCalledOnDoEndBlock:
-  Enabled: true
+  Enabled: false
 
 Style/CollectionMethods:
   Enabled: true
@@ -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,11 +30,8 @@ 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 'pry-byebug', '~> 3.9.0'
   spec.add_development_dependency 'rake', '~> 12.0'
   spec.add_development_dependency 'rspec', '~> 3.0'
   spec.add_development_dependency 'rspec_junit_formatter', '~> 0.4.1'

metadata

@@ -1,29 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: simple-images-downloader
 version: !ruby/object:Gem::Version
-  version: 1.0.2
+  version: 1.1.0
 platform: ruby
 authors:
 - IlkhamGaysin
-autorequire: 
+autorequire:
 bindir: exe
 cert_chain: []
-date: 2020-09-25 00:00:00.000000000 Z
+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
@@ -52,20 +38,6 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: 0.13.1
-- !ruby/object:Gem::Dependency
-  name: pry-byebug
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 3.9.0
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 3.9.0
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -192,7 +164,7 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: 3.9.1
-description: 
+description:
 email:
 - ilgamgaysin@gmail.com
 executables:
@@ -220,16 +192,24 @@ files:
 - bin/setup
 - exe/simple-images-downloader
 - lib/simple_images_downloader.rb
+- lib/simple_images_downloader/client.rb
 - lib/simple_images_downloader/configuration.rb
 - lib/simple_images_downloader/dispenser.rb
 - lib/simple_images_downloader/downloader.rb
 - lib/simple_images_downloader/errors.rb
-- lib/simple_images_downloader/file_persistance_validator.rb
-- lib/simple_images_downloader/image_path_validator.rb
 - 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/mime_type_validator.rb
+- lib/simple_images_downloader/validatable/validator.rb
 - lib/simple_images_downloader/version.rb
 - rubocop.yml
 - simple_images_downloader.gemspec
@@ -240,7 +220,7 @@ metadata:
   homepage_uri: https://github.com/IlkhamGaysin/simple-images-downloader
   source_code_uri: https://github.com/IlkhamGaysin/simple-images-downloader
   changelog_uri: https://github.com/IlkhamGaysin/simple-images-downloader/blob/master/CHANGELOG.md
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -256,7 +236,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubygems_version: 3.1.2
-signing_key: 
+signing_key:
 specification_version: 4
 summary: simple-images-downloader allows to download images from a file containing
   list of urls to those images.

data/lib/simple_images_downloader/file_persistance_validator.rb

@@ -1,13 +0,0 @@
-# frozen_string_literal: true
-
-module SimpleImagesDownloader
-  class FilePersistanceValidator
-    def initialize(path)
-      @path = path
-    end
-
-    def validate
-      raise Errors::MissingFileError, @path unless File.exist?(@path)
-    end
-  end
-end

data/lib/simple_images_downloader/image_path_validator.rb

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