eight_ball 1.0.5 → 2.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0b909046d84dce15d996867c4f73e472242ba11518f0c54e202bdc5a9333dd60
-  data.tar.gz: 01c7bece8234dd01c11d3d9cf204aabef1fb443287d53bf31d4de9e3e396e9e6
+  metadata.gz: d034f89a36b50a0d0c9b8b9a4831a064e5ba94040b420f87a5afd440ff219674
+  data.tar.gz: 9b545a863f9eb98522ae8d039fae1c4f53d21c16d9a86427f919753327ff5d7d
 SHA512:
-  metadata.gz: 2e3cfe13696c5d5c9e60859ed02fae02f7e056771d285a60ec7bb7d85eee917617992a0753b2d54a49470555b56fc8c8a4f2cb3e8a438d743e92a1ec5a06a1dc
-  data.tar.gz: fc0dddd891374607287e27e6a7b50f0286043c9f288d894b50266b4ab59cd690e15c949bbfaa3aa5ff314e43ca4f1cffce2e0f71d7ccb1dabe094ebc305be8d4
+  metadata.gz: eb363e7384b897cf5a17f6be921881de912828a496ddf1d403be301c145bcfc85cebc76acca1b3938e7c823f1176f76ab4297ed8b45407ec83a90b18ed2adfbf
+  data.tar.gz: ffa4369cfba089198cac8495cc1b4ccdc2c158a82feddbf6df88e33c5da77e0810fd372eb2fcd9c5d4f5cd343d1e99647a36c0ef9aa095e2d60f37c89f29456d

data/.rspec

@@ -0,0 +1 @@
+--require spec_helper

data/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## [2.0.0]
+ - [BREAKING] `Parsers` have been replaced with `Marshallers`, allowing bi-directional conversions
+ - Added `EightBall.marshall` as a way to output the Feature list to an external format (e.g. to create a JSON file)
+ - Added `EightBall.features` as a shortcut to `EightBall.provider.features`
+ - Testing framework has been moved from Minitest to rspec
+ - Updated dev dependencies
+
 ## [1.0.5]
 Security: Update rake to >= 12.3.3
 
@@ -14,4 +21,3 @@ Security: Update yard 0.9.16 -> 0.9.20
 
 ## [1.0.0]
 Initial release!
-

data/Gemfile

@@ -1,6 +1,6 @@
 source 'https://rubygems.org'
 
-git_source(:github) {|repo_name| "https://github.com/#{repo_name}" }
+git_source(:github) { |repo_name| "https://github.com/#{repo_name}" }
 
 # Specify your gem's dependencies in eight_ball.gemspec
 gemspec

data/Gemfile.lock

@@ -1,43 +1,50 @@
 PATH
   remote: .
   specs:
-    eight_ball (1.0.5)
+    eight_ball (2.0.0)
+      awrence (~> 1.1)
       plissken (~> 1.2)
 
 GEM
   remote: https://rubygems.org/
   specs:
     ansi (1.5.0)
-    builder (3.2.4)
-    byebug (11.1.1)
-    coderay (1.1.2)
+    awrence (1.1.1)
+    byebug (11.1.3)
+    coderay (1.1.3)
+    diff-lcs (1.3)
     docile (1.3.2)
     inch (0.8.0)
       pry
       sparkr (>= 0.2.0)
       term-ansicolor
       yard (~> 0.9.12)
-    method_source (0.9.2)
-    minitest (5.14.0)
-    minitest-reporters (1.4.2)
-      ansi
-      builder
-      minitest (>= 5.0)
-      ruby-progressbar
-    mocha (1.11.2)
+    method_source (1.0.0)
     plissken (1.3.1)
-    pry (0.12.2)
-      coderay (~> 1.1.0)
-      method_source (~> 0.9.0)
-    pry-byebug (3.8.0)
+    pry (0.13.1)
+      coderay (~> 1.1)
+      method_source (~> 1.0)
+    pry-byebug (3.9.0)
       byebug (~> 11.0)
-      pry (~> 0.10)
-    rake (12.3.3)
-    ruby-progressbar (1.10.1)
+      pry (~> 0.13.0)
+    rake (13.0.1)
+    rspec (3.9.0)
+      rspec-core (~> 3.9.0)
+      rspec-expectations (~> 3.9.0)
+      rspec-mocks (~> 3.9.0)
+    rspec-core (3.9.2)
+      rspec-support (~> 3.9.3)
+    rspec-expectations (3.9.2)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.9.0)
+    rspec-mocks (3.9.1)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.9.0)
+    rspec-support (3.9.3)
     simplecov (0.18.5)
       docile (~> 1.1)
       simplecov-html (~> 0.11)
-    simplecov-console (0.6.0)
+    simplecov-console (0.7.2)
       ansi
       simplecov
       terminal-table
@@ -48,10 +55,10 @@ GEM
       tins (~> 1.0)
     terminal-table (1.8.0)
       unicode-display_width (~> 1.1, >= 1.1.1)
-    tins (1.24.1)
+    tins (1.25.0)
       sync
-    unicode-display_width (1.6.1)
-    yard (0.9.24)
+    unicode-display_width (1.7.0)
+    yard (0.9.25)
 
 PLATFORMS
   ruby
@@ -60,11 +67,9 @@ DEPENDENCIES
   bundler (~> 1.17)
   eight_ball!
   inch (~> 0.8)
-  minitest (~> 5.0)
-  minitest-reporters (~> 1.3)
-  mocha (~> 1.7)
   pry-byebug (~> 3.6)
-  rake (~> 12.3.3)
+  rake (~> 13.0)
+  rspec (~> 3.9.0)
   simplecov (~> 0.16)
   simplecov-console (~> 0.4)
 

data/README.md

@@ -42,8 +42,8 @@ json_input = %(
 )
 
 # Transform the JSON into a list of Features
-parser = EightBall::Parsers::Json.new
-features = parser.parse json_input
+marshaller = EightBall::Marshallers::Json.new
+features = marshaller.unmarshall json_input
 
 # Tell EightBall about these Features
 EightBall.provider = EightBall::Providers::Static.new features
@@ -76,7 +76,7 @@ A Condition must either be `true` or `false`. It describes when a Feature is ena
 A Provider is able to give EightBall the list of Features it needs to answer queries.
 
 **Supported Providers**
-- [HTTP](lib/eight_ball/providers/http.rb): Connect to a URL and use the given Parser to convert the response into a list of Features.
+- [HTTP](lib/eight_ball/providers/http.rb): Connect to a URL and use the given Marshaller to convert the response into a list of Features.
 - [Static](lib/eight_ball/providers/static.rb): Once initialized with a list of Features, always provides that same list of Features.
 
 #### RefreshPolicies
@@ -85,15 +85,15 @@ Some Providers are able to automatically "refresh" their list of Features using
 **Supported RefreshPolicies**
 - [Interval](lib/eight_ball/providers/refresh_policies/interval.rb): The data is considered fresh for a given number of seconds, after which it is considered stale and should be refreshed.
 
-### Parser
-A Parser converts the given input to an array of Features.
+### Marshallers
+A Marshaller converts Features to and from another format.
 
-**Supported Parsers**
-- [JSON](lib/eight_ball/parsers/json.rb)
+**Supported Marshaller**
+- [JSON](lib/eight_ball/marshallers/json.rb)
 
 ## Development
 
-After checking out the repo, run `bin/setup` to install dependencies. Then, run `rake test` to run the tests. You can also run `bin/console` for an interactive prompt that will allow you to experiment.
+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.
 
 To install this gem onto your local machine, run `bundle exec rake install`.
 

data/Rakefile

@@ -1,10 +1,9 @@
-require "bundler/gem_tasks"
-require "rake/testtask"
+require 'bundler/gem_tasks'
 
-Rake::TestTask.new(:test) do |t|
-  t.libs << "test"
-  t.libs << "lib"
-  t.test_files = FileList["test/**/*_test.rb"]
+begin
+  require 'rspec/core/rake_task'
+  RSpec::Core::RakeTask.new(:spec)
+rescue LoadError
 end
 
-task :default => :test
+task default: :spec

data/eight_ball.gemspec

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-lib = File.expand_path('../lib', __FILE__)
+lib = File.expand_path('lib', __dir__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require 'eight_ball/version'
 
@@ -17,22 +17,22 @@ Gem::Specification.new do |spec|
 
   spec.required_ruby_version = '>= 2.5.0'
 
-  spec.files         = `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(test|.github|examples)/}) }
+  spec.files         = `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(spec|.github|examples)/}) }
   spec.bindir        = 'exe'
   spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
   spec.require_paths = ['lib']
 
   ### DEPENDENCIES
+
+  spec.add_dependency 'awrence', '~> 1.1'
   spec.add_dependency 'plissken', '~> 1.2'
 
   # Development
   spec.add_development_dependency 'bundler', '~> 1.17'
   spec.add_development_dependency 'inch', '~> 0.8'
-  spec.add_development_dependency 'minitest', '~> 5.0'
-  spec.add_development_dependency 'minitest-reporters', '~> 1.3'
-  spec.add_development_dependency 'mocha', '~> 1.7'
   spec.add_development_dependency 'pry-byebug', '~> 3.6'
-  spec.add_development_dependency 'rake', '~> 12.3.3'
+  spec.add_development_dependency 'rake', '~> 13.0'
+  spec.add_development_dependency 'rspec', '~> 3.9.0'
   spec.add_development_dependency 'simplecov', '~> 0.16'
   spec.add_development_dependency 'simplecov-console', '~> 0.4'
 end

data/lib/eight_ball.rb

@@ -3,6 +3,8 @@
 require 'eight_ball/version'
 require 'eight_ball/feature'
 
+require 'eight_ball/configuration_error'
+
 require 'eight_ball/conditions/conditions'
 require 'eight_ball/conditions/base'
 
@@ -11,7 +13,7 @@ require 'eight_ball/conditions/list'
 require 'eight_ball/conditions/never'
 require 'eight_ball/conditions/range'
 
-require 'eight_ball/parsers/json'
+require 'eight_ball/marshallers/json'
 
 require 'eight_ball/providers/http'
 require 'eight_ball/providers/static'
@@ -33,6 +35,24 @@ module EightBall
     @provider = provider
   end
 
+  # Gets the {EightBall::Providers Provider} instance
+  # EightBall is configured to use
+  #
+  # @return {EightBall::Providers Provider}
+  def self.provider
+    @provider
+  end
+
+  # Serves as a shortcut to access the {EightBall::Feature Features} available
+  # on the configured {EightBall::Providers Provider}
+  #
+  # @return [Array<EightBall::Feature>]
+  def self.features
+    raise EightBall::ConfigurationError, 'No Provider has been configured; there can be no features. Please see "EightBall.provider="' unless provider
+
+    provider.features
+  end
+
   # "EightBall, is the feature named 'NewFeature' enabled?"
   #
   # @return whether or not the {EightBall::Feature} is enabled.
@@ -45,7 +65,7 @@ module EightBall
   # @example
   #   EightBall.enabled? 'feature1', account_id: 1
   def self.enabled?(name, parameters = {})
-    feature = @provider.features.find { |f| f.name == name }
+    feature = provider.features.find { |f| f.name == name }
     return false unless feature
 
     feature.enabled? parameters
@@ -69,7 +89,7 @@ module EightBall
   # Yields to the given block of code if the {EightBall::Feature} is enabled.
   #
   # @return [nil] if block is yielded to
-  # @return [false] if {EightBall::Feature} is disabled
+  # @return [false] if no block is given
   #
   # @param name [String] The name of the {EightBall::Feature}.
   # @param parameters [Hash] The parameters the {EightBall::Conditions} of this
@@ -85,9 +105,28 @@ module EightBall
     yield if enabled? name, parameters
   end
 
+  # Marshalls the {EightBall::Feature Features}. This can be useful for
+  # converting the data to, e.g., a JSON file.
+  #
+  # If a {EightBall::Marshallers Marshaller} is provided, use it.
+  #
+  # If no {EightBall::Marshallers Marshaller} is provided, uses the same
+  # Marshaller that the Provider is configured with.
+  #
+  # If the {EightBall::Providers Provider} does not expose a
+  # {EightBall::Marshallers Marshaller}, this will default to the
+  # {EightBall::Marshallers::Json JSON Marshaller}.
+  def self.marshall(marshaller = nil)
+    marshaller ||=
+      (provider.respond_to?(:marshaller) && provider.marshaller) ||
+      EightBall::Marshallers::Json.new
+
+    marshaller.marshall features
+  end
+
   def self.logger
     @logger ||= Logger.new(STDOUT).tap do |log|
-      log.progname = self.name
+      log.progname = name
     end
   end
 

data/lib/eight_ball/conditions/base.rb

@@ -4,7 +4,7 @@ module EightBall::Conditions
   class Base
     attr_reader :parameter
 
-    def initialize(options = [])
+    def initialize(_options = [])
       @parameter = nil
     end
 
@@ -16,7 +16,8 @@ module EightBall::Conditions
 
     def parameter=(parameter)
       return if parameter.nil?
-      @parameter = parameter.gsub(/(.)([A-Z])/,'\1_\2').downcase
+
+      @parameter = parameter.gsub(/(.)([A-Z])/, '\1_\2').downcase
     end
   end
 end

data/lib/eight_ball/configuration_error.rb

@@ -0,0 +1,9 @@
+# frozen_string_literal: true
+
+module EightBall
+  class ConfigurationError < StandardError
+    def initialize(msg = 'An invalid configuration was detected')
+      super
+    end
+  end
+end

data/lib/eight_ball/marshallers/json.rb

@@ -0,0 +1,115 @@
+# frozen_string_literal: true
+
+require 'awrence'
+require 'json'
+require 'plissken'
+
+# A JSON marshaller can convert back and forth between JSON and a list of {EightBall::Feature Features}
+# The JSON produced will be pretty-printed, as it is assumed the output will be written to a file.
+#
+# When converting from JSON, the top-level JSON element must be an array and
+# its keys must use camel-case; this will be converted to snake-case by EightBall
+# in order to adhere to both JSON and Ruby standards.
+#
+# Below are some examples of valid JSON:
+#
+# @example A single {EightBall::Feature} is enabled for accounts 1-5 as well as region Europe
+#   [{
+#     "name": "Feature1",
+#     "enabledFor": [{
+#       "type": "range",
+#       "parameter": "accountId",
+#       "min": 1,
+#       "max": 5
+#     }, {
+#       "type": "list",
+#       "parameter": "regionName",
+#       "values": ["Europe"]
+#     }]
+#   }]
+#
+# @example A single {EightBall::Feature} is disabled completely using the {EightBall::Conditions::Always Always} condition
+#   [{
+#     "name": "Feature1",
+#     "disabledFor": [{
+#       "type": "always"
+#     }]
+#   }]
+module EightBall::Marshallers
+  class Json
+    # Convert the given {EightBall::Feature Features} into a JSON array.
+    #
+    # @param [Array<EightBall::Feature>] features The {EightBall::Feature Features} to convert.
+    # @return [String] The resulting JSON string.
+    #
+    # @example
+    #   json_string = <Read from somewhere>
+    #
+    #   marshaller = EightBall::Marshallers::Json.new
+    #   marshaller.marshall [Array<EightBall::Feature>] => json
+    def marshall(features)
+      JSON.generate(features.map { |feature| feature_to_hash(feature).to_camelback_keys })
+    end
+
+    # Convert the given JSON into a list of {EightBall::Feature Features}.
+    #
+    # @param [String] json The JSON string to convert.
+    # @return [Array<EightBall::Feature>] The parsed {EightBall::Feature Features}
+    #
+    # @example
+    #   json_string = <Read from somewhere>
+    #
+    #   marshaller = EightBall::Marshallers::Json.new
+    #   marshaller.unmarshall json_string => [Features]
+    def unmarshall(json)
+      parsed = JSON.parse(json, symbolize_names: true).to_snake_keys
+
+      raise ArgumentError, 'JSON input was not an array' unless parsed.is_a? Array
+
+      parsed.map do |feature|
+        enabled_for = create_conditions_from_json feature[:enabled_for]
+        disabled_for = create_conditions_from_json feature[:disabled_for]
+
+        EightBall::Feature.new feature[:name], enabled_for, disabled_for
+      end
+    rescue JSON::ParserError => e
+      EightBall.logger.error { "Failed to parse JSON: #{e.message}" }
+      []
+    end
+
+    private
+
+    def feature_to_hash(feature)
+      hash = {
+        name: feature.name
+      }
+
+      hash[:enabled_for] = feature.enabled_for.map { |condition| condition_to_hash(condition) } unless feature.enabled_for.empty?
+      hash[:disabled_for] = feature.disabled_for.map { |condition| condition_to_hash(condition) } unless feature.disabled_for.empty?
+
+      hash
+    end
+
+    def condition_to_hash(condition)
+      hash = {
+        type: condition.class.name.split('::').last.downcase
+      }
+      condition.instance_variables.each do |var|
+        next unless condition.instance_variable_get(var)
+
+        hash[var.to_s.delete('@')] = condition.instance_variable_get(var)
+      end
+
+      hash
+    end
+
+    def create_conditions_from_json(json_conditions)
+      return [] unless json_conditions&.is_a?(Array)
+
+      json_conditions.map do |condition|
+        condition_class = EightBall::Conditions.by_name condition[:type]
+        condition_class.new condition
+      end
+    end
+  end
+end

data/lib/eight_ball/providers/http.rb

@@ -1,22 +1,26 @@
 # frozen_string_literal: true
 
+require 'net/http'
+
 module EightBall::Providers
   # An HTTP Provider will make a GET request to a given URI, and convert
   # the response into an array of {EightBall::Feature Features} using the
-  # given {EightBall::Parsers Parser}.
+  # given {EightBall::Marshallers Marshaller}.
   #
   # The {EightBall::Feature Features} will be automatically kept up to date
   # according to the given {EightBall::Providers::RefreshPolicies RefreshPolicy}.
   class Http
     SUPPORTED_SCHEMES = %w[http https].freeze
 
+    attr_reader :marshaller
+
     # @param uri [String] The URI to GET the {EightBall::Feature Features} from.
     # @param options [Hash] The options to create the Provider with.
     #
-    # @option options [EightBall::Parsers] :parser
-    #   The {EightBall::Parsers Parser} used to convert the response to an array
+    # @option options [EightBall::Marshallers] :marshaller
+    #   The {EightBall::Marshallers Marshaller} used to convert the response to an array
     #   of {EightBall::Feature Features}. Defaults to an instance of
-    #   {EightBall::Parsers::Json}
+    #   {EightBall::Marshallers::Json}
     #
     # @option options [EightBall::Providers::RefreshPolicies] :refresh_policy
     #   The {EightBall::Providers::RefreshPolicies Policy} used to determine
@@ -34,7 +38,7 @@ module EightBall::Providers
 
       @uri = URI.parse uri
 
-      @parser = options[:parser] || EightBall::Parsers::Json.new
+      @marshaller = options[:marshaller] || EightBall::Marshallers::Json.new
       @policy = options[:refresh_policy] || EightBall::Providers::RefreshPolicies::Interval.new
     end
 
@@ -48,8 +52,8 @@ module EightBall::Providers
     private
 
     def fetch
-      @features = @parser.parse Net::HTTP.get(@uri)
-    rescue => e
+      @features = @marshaller.unmarshall Net::HTTP.get(@uri)
+    rescue StandardError => e
       EightBall.logger.error { "Failed to fetch data from #{@uri}: #{e.message}" }
       @features = []
     end

data/lib/eight_ball/providers/refresh_policies/interval.rb

@@ -7,7 +7,7 @@ module EightBall::Providers::RefreshPolicies
   # amount of time, after which it is considered stale and should be refreshed.
   class Interval
     SECONDS_IN_A_DAY = 86_400
-    
+
     # Creates a new instance of an Interval RefreshPolicy.
     #
     # @param seconds [Integer] The number of seconds the data is considered fresh.

data/lib/eight_ball/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module EightBall
-  VERSION = '1.0.5'
+  VERSION = '2.0.0'
 end

metadata

@@ -1,127 +1,113 @@
 --- !ruby/object:Gem::Specification
 name: eight_ball
 version: !ruby/object:Gem::Version
-  version: 1.0.5
+  version: 2.0.0
 platform: ruby
 authors:
 - Rewind.io
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2020-03-02 00:00:00.000000000 Z
+date: 2020-06-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: plissken
+  name: awrence
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.2'
+        version: '1.1'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.2'
-- !ruby/object:Gem::Dependency
-  name: bundler
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '1.17'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '1.17'
+        version: '1.1'
 - !ruby/object:Gem::Dependency
-  name: inch
+  name: plissken
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.8'
-  type: :development
+        version: '1.2'
+  type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.8'
+        version: '1.2'
 - !ruby/object:Gem::Dependency
-  name: minitest
+  name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '5.0'
+        version: '1.17'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '5.0'
+        version: '1.17'
 - !ruby/object:Gem::Dependency
-  name: minitest-reporters
+  name: inch
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.3'
+        version: '0.8'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.3'
+        version: '0.8'
 - !ruby/object:Gem::Dependency
-  name: mocha
+  name: pry-byebug
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.7'
+        version: '3.6'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.7'
+        version: '3.6'
 - !ruby/object:Gem::Dependency
-  name: pry-byebug
+  name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.6'
+        version: '13.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.6'
+        version: '13.0'
 - !ruby/object:Gem::Dependency
-  name: rake
+  name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 12.3.3
+        version: 3.9.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 12.3.3
+        version: 3.9.0
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
@@ -158,6 +144,7 @@ extensions: []
 extra_rdoc_files: []
 files:
 - ".gitignore"
+- ".rspec"
 - ".travis.yml"
 - CHANGELOG.md
 - Gemfile
@@ -175,8 +162,9 @@ files:
 - lib/eight_ball/conditions/list.rb
 - lib/eight_ball/conditions/never.rb
 - lib/eight_ball/conditions/range.rb
+- lib/eight_ball/configuration_error.rb
 - lib/eight_ball/feature.rb
-- lib/eight_ball/parsers/json.rb
+- lib/eight_ball/marshallers/json.rb
 - lib/eight_ball/providers/http.rb
 - lib/eight_ball/providers/refresh_policies/interval.rb
 - lib/eight_ball/providers/static.rb

data/lib/eight_ball/parsers/json.rb

@@ -1,72 +0,0 @@
-# frozen_string_literal: true
-
-require 'plissken'
-
-module EightBall::Parsers
-  # A JSON parser will parse JSON into a list of {EightBall::Feature Features}.
-  # The top-level JSON element must be an array and must use camel-case;
-  # this will be converted to snake-case by EightBall.
-  #
-  # Below are some examples of valid JSON:
-  #
-  # @example A single {EightBall::Feature} is enabled for accounts 1-5 as well as region Europe
-  #   [{
-  #     "name": "Feature1",
-  #     "enabledFor": [{
-  #       "type": "range",
-  #       "parameter": "accountId",
-  #       "min": 1,
-  #       "max": 5
-  #     }, {
-  #       "type": "list",
-  #       "parameter": "regionName",
-  #       "values": ["Europe"]
-  #     }]
-  #   }]
-  #
-  # @example A single {EightBall::Feature} is disabled completely using the {EightBall::Conditions::Always Always} condition
-  #   [{
-  #     "name": "Feature1",
-  #     "disabledFor": [{
-  #       "type": "always"
-  #     }]
-  #   }]
-  class Json
-    # Convert the JSON into a list of {EightBall::Feature Features}.
-    #
-    # @param [String] json The JSON string to parse.
-    # @return [Array<EightBall::Feature>] The parsed {EightBall::Feature Features}
-    #
-    # @example
-    #   json_string = <Read from somewhere>
-    #
-    #   parser = EightBall::Parsers::Json.new
-    #   parser.parse json_string => [Features]
-    def parse(json)
-      parsed = JSON.parse(json, :symbolize_names => true).to_snake_keys
-      
-      raise ArgumentError, 'JSON input was not an array' unless parsed.is_a? Array
-
-      parsed.map do |feature|
-        enabled_for = create_conditions feature[:enabled_for]
-        disabled_for = create_conditions feature[:disabled_for]
-
-        EightBall::Feature.new feature[:name], enabled_for, disabled_for
-      end
-    rescue JSON::ParserError => e
-      EightBall.logger.error { "Failed to parse JSON: #{e.message}" }
-      []
-    end
-
-    private
-
-    def create_conditions(json_conditions)
-      return [] unless json_conditions && json_conditions.is_a?(Array)
-
-      json_conditions.map do |condition|
-        condition_class = EightBall::Conditions.by_name condition[:type]
-        condition_class.new condition
-      end
-    end
-  end
-end