piotr_majewski_magic 0.0.3 → 0.0.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 54f030e5007aa276f96f57d2ddf36a80279a6a43526c2dd3e0eaf6104952c120
-  data.tar.gz: 735e920318f6d5b0e80cf3bbfb15db6d58b6f39ed29680b4c25e12051be76272
+  metadata.gz: ed5b3f6e3f03be200af1ad183e2018e7f6d8d1eafe176e9cd8fd36b390996794
+  data.tar.gz: bc1e6106249788a6557ac3834754e21984c4e69f93269d9885b74a23a6b22e12
 SHA512:
-  metadata.gz: a7b0f349504dfcb6ffa8fad4db778e90b6870e5acbe62eb31d240cea6d09773b0cfc743bba877a5d20acb7970613f466a7b9a4001d08523cc7d17c316d1c7db6
-  data.tar.gz: ba7fa731574493015ad361929c0a53e9d105f357ce1bb52300188c0d234b8ea7190995d8fcc84553d8d4bb27a65deca2106f44cd90f7f9d1b3a60130a846d9c2
+  metadata.gz: b0f242a31a2a4abb76daa022d29949c4a9808ec898df2f26c4cb0da5f3b86520063def5d25931e88bc3db5372af2e96dbb6f922190de34b0b8fe99269f54f451
+  data.tar.gz: 3c10ce8899227c17b46e4be6d87843b00e66dcab3aa2fc1d60c4a2cfdce9dfb8b72c205e21a28fe01ae93f2c708b8b49eda7b84592bb84665047f1ad8ddbe2f8

data/lib/{cards_source.rb → cards.rb}

@@ -3,13 +3,13 @@
 require 'json'
 require_relative 'mtg_api_response'
 
-class CardsSource
+class Cards
   def self.get(no_cache = nil)
     new(no_cache).get
   end
 
   def get
-    JSON.parse(cards_source)
+    JSON.parse(cards)
   end
 
   private
@@ -20,14 +20,21 @@ class CardsSource
     @no_cache = no_cache
   end
 
-  def cards_source
-    return File.read(cache_file_path) if File.file?(cache_file_path) && !no_cache
+  def cards
+    fetch_and_save_cards_from_api if no_cache || cache_empty?
 
-    cache_cards
+    read_cache
+  end
+
+  def read_cache
     File.read(cache_file_path)
   end
 
-  def cache_cards
+  def cache_empty?
+    File.size(cache_file_path).zero?
+  end
+
+  def fetch_and_save_cards_from_api
     File.open(cache_file_path, 'w') do |file|
       file.write(MtgApiResponse.results)
     end

data/lib/cards_manager.rb

@@ -22,15 +22,16 @@ class CardsManager
   private
 
   def group_and_select(query, cards)
-    raise UnknownAttribute unless cards.first.key?(query[0][:attribute])
+    attribute = query[0][:attribute]
+    unknown_attribute(attribute) unless cards.first.key?(attribute)
 
     selected_cards = matching_cards(query[0], cards)
     new_query = query[1..]
 
     return selected_cards if new_query.empty?
 
-    selected_cards.map.with_object({}) do |(attribute_value, cards), hash|
-      hash[attribute_value] = group_and_select(new_query, cards)
+    selected_cards.map.with_object({}) do |(attribute_value, card_set), hash|
+      hash[attribute_value] = group_and_select(new_query, card_set)
     end
   end
 
@@ -46,6 +47,14 @@ class CardsManager
 
     cli_value.tally == source_value&.tally
   rescue NoMethodError
+    invalid_value(cli_value)
+  end
+
+  def unknown_attribute(attribute)
+    raise UnknownAttribute, "`#{attribute}`: Is not an MTG card attribute"
+  end
+
+  def invalid_value(cli_value)
     message = "`#{cli_value}`: Type of provided value doesn't match type of card attribute"
     raise InvalidValue, message
   end

data/lib/mtg_api_request.rb

@@ -30,10 +30,10 @@ class MtgApiRequest
   private
 
   def unfetched_pages
-    @unfetched_pages ||= (1..total_count).to_a
+    @unfetched_pages ||= (1..total_page_count).to_a
   end
 
-  def total_count
+  def total_page_count
     response = Faraday.get(URL)
     (response.headers['total-count'].to_f / 100).ceil
   end

data/lib/query.rb

@@ -5,6 +5,9 @@ require 'forwardable'
 class Query
   extend Forwardable
 
+  ATTRIBUTE_AND_VALUE_DIVIDERS = /(?<!~)=/.freeze
+  VALUE_TO_LIST_DIVIDERS = /(?<!~)&/.freeze
+
   RepeatedFilter = Class.new(StandardError)
   Group = Class.new
 
@@ -16,11 +19,10 @@ class Query
 
   def build
     @build ||= argv.map.with_object([]) do |argument, query_options|
-      query_element = parse_argument(argument)
+      query_element = query_element(argument)
 
       if query_options.find { |el| el[:attribute] == query_element[:attribute] }
-        message = "#{query_element[:attribute]}: Every filter can be passed only once"
-        raise RepeatedFilter, message
+        repeated_filter(query_element[:attribute])
       end
 
       query_options << query_element
@@ -31,15 +33,35 @@ class Query
 
   private
 
-  # TODO: pick a divider (something that is unlikely to show up)
-  # potential problem with imageUrl values
-  # NOTE: should allow escaping `#`
-  def parse_argument(argument)
-    return { attribute: argument, value: Group } unless argument.match?('=')
+  def query_element(argument)
+    return { attribute: argument, value: Group } unless argument.match?(ATTRIBUTE_AND_VALUE_DIVIDERS)
+
+    attribute, value = argument.split(ATTRIBUTE_AND_VALUE_DIVIDERS)
+
+    {
+      attribute: sanitize(attribute),
+      value: sanitize(parse_value(value))
+    }
+  end
+
+  def parse_value(value)
+    return value unless value&.match?(VALUE_TO_LIST_DIVIDERS)
 
-    attribute, value = argument.split('=')
-    value = value&.match?('#') ? value.split('#') : value
+    value.split(VALUE_TO_LIST_DIVIDERS)
+  end
+
+  def sanitize(query_part)
+    return unless query_part
+
+    if query_part.is_a?(String)
+      query_part.gsub(/~=/, '=').gsub(/~&/, '&')
+    else
+      query_part.map { |part| sanitize(part) }
+    end
+  end
 
-    { attribute: attribute, value: value }
+  def repeated_filter(attribute)
+    message = "#{attribute}: Every filter can be passed only once"
+    raise RepeatedFilter, message
   end
 end

data/lib/query_mtg.rb

@@ -3,7 +3,7 @@
 require 'json'
 require_relative 'cards_manager'
 require_relative 'query'
-require_relative 'cards_source'
+require_relative 'cards'
 
 class QueryMtg
   def self.call(argv)
@@ -34,6 +34,6 @@ class QueryMtg
   end
 
   def cards_source
-    CardsSource.get(no_cache)
+    Cards.get(no_cache)
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: piotr_majewski_magic
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.0.8
 platform: ruby
 authors:
 - Piotr Majewski
@@ -48,8 +48,8 @@ extra_rdoc_files: []
 files:
 - bin/query_mtg
 - cache/cards.json
+- lib/cards.rb
 - lib/cards_manager.rb
-- lib/cards_source.rb
 - lib/mtg_api_request.rb
 - lib/mtg_api_response.rb
 - lib/query.rb