toggles 0.1.1 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 4c0bf3043c4b85c32062fa22750cc20c88af4cd4
-  data.tar.gz: f8aeb5d080385e546815ccc57d8401e8abc1c9e4
+SHA256:
+  metadata.gz: ece49487f49d27e3a10944d318e3b2c1b9a0e9a8a3c231581ebf5341e7abc4f2
+  data.tar.gz: d278a07f9e9e5e379936e02fa27e4a4cd7270ef2e7933181e5948c30d88bb4fd
 SHA512:
-  metadata.gz: 2b7f5e471ba5456ee1ecc659f902b2826fecd2de532c1a4aef9b49e4791985df6cbdf73260e9643cd340711c3ab8f63c6daefb2691bcbc761fade903d3eacf93
-  data.tar.gz: 08d1d0b300525bde3f0166b3fced8bc426b11f383eeb0ef4a6a60a1530e4e52f1e67522ea96472f2546a4f98d35ffece5642b3997569fb00777639626e9b14e0
+  metadata.gz: 2e80d24bb41c699217fa865e8ac2c346c4fbdc411d93eec98dc0d4ff11953f1c757a778606d62de664602a4320788b63389beca5f831431d7bc90bbee24eea0d
+  data.tar.gz: 4c4f3ea7f4d06f1a465f53e3114e5a456eed9c7de889e940910a01ed300823852e2b2cb823c89152685ed489800cf406d51efc876179feda4ad204077305ba72

data/.github/workflows/ci.yml

@@ -0,0 +1,24 @@
+name: 'CI'
+
+on:
+  push:
+    branches: [ master ]
+  pull_request: ~
+  schedule:
+    - cron: "0 16 * * 1"
+
+jobs:
+  run-tests:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        rubyversion: ['2.7', '3.0', '3.1']
+    steps:
+      - uses: actions/checkout@v2
+      - name: set up ruby
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: ${{ matrix.rubyversion }}
+          bundler-cache: true
+      - name: run tests
+        run: bundle exec rspec

data/CHANGELOG.md

@@ -0,0 +1,83 @@
+# Changelog
+
+## [v0.4.0](https://github.com/EasyPost/toggles/tree/v0.4.0)
+
+[Full Changelog](https://github.com/EasyPost/toggles/compare/v0.3.0...v0.4.0)
+
+**Merged pull requests:**
+
+- refactor: remove Feature::Base, just use Feature::Permissions [\#13](https://github.com/EasyPost/toggles/pull/13) ([lanej](https://github.com/lanej))
+- refactor: collapse some hierarchies, model operations as functions [\#12](https://github.com/EasyPost/toggles/pull/12) ([lanej](https://github.com/lanej))
+- fix: handle abbreviations and numbers in feature name [\#11](https://github.com/EasyPost/toggles/pull/11) ([lanej](https://github.com/lanej))
+- Deprecate constant lookups via .const\_missing [\#10](https://github.com/EasyPost/toggles/pull/10) ([lanej](https://github.com/lanej))
+
+## [v0.3.0](https://github.com/EasyPost/toggles/tree/v0.3.0) (2021-08-16)
+
+[Full Changelog](https://github.com/EasyPost/toggles/compare/v0.1.2...v0.3.0)
+
+**Merged pull requests:**
+
+- add github actions integration [\#9](https://github.com/EasyPost/toggles/pull/9) ([Roguelazer](https://github.com/Roguelazer))
+- point build status link at travis-ci.com instead of travis-ci.org [\#8](https://github.com/EasyPost/toggles/pull/8) ([Roguelazer](https://github.com/Roguelazer))
+- fix tests [\#7](https://github.com/EasyPost/toggles/pull/7) ([Roguelazer](https://github.com/Roguelazer))
+- run tests on travis [\#6](https://github.com/EasyPost/toggles/pull/6) ([Roguelazer](https://github.com/Roguelazer))
+
+## [v0.1.2](https://github.com/EasyPost/toggles/tree/v0.1.2) (2016-11-08)
+
+[Full Changelog](https://github.com/EasyPost/toggles/compare/v0.1.1...v0.1.2)
+
+**Merged pull requests:**
+
+- store the actual feature configuration in a variable instead of in real constants under the Feature module [\#4](https://github.com/EasyPost/toggles/pull/4) ([Roguelazer](https://github.com/Roguelazer))
+
+## [v0.1.1](https://github.com/EasyPost/toggles/tree/v0.1.1) (2015-08-24)
+
+[Full Changelog](https://github.com/EasyPost/toggles/compare/v0.1.0...v0.1.1)
+
+## [v0.1.0](https://github.com/EasyPost/toggles/tree/v0.1.0) (2015-08-24)
+
+[Full Changelog](https://github.com/EasyPost/toggles/compare/v0.0.8...v0.1.0)
+
+**Closed issues:**
+
+- Handle when properties are nil [\#2](https://github.com/EasyPost/toggles/issues/2)
+
+## [v0.0.8](https://github.com/EasyPost/toggles/tree/v0.0.8) (2015-02-25)
+
+[Full Changelog](https://github.com/EasyPost/toggles/compare/v0.0.7...v0.0.8)
+
+**Merged pull requests:**
+
+- Features automatically disabled\_for nil input [\#3](https://github.com/EasyPost/toggles/pull/3) ([victoryftw](https://github.com/victoryftw))
+
+## [v0.0.7](https://github.com/EasyPost/toggles/tree/v0.0.7) (2014-12-24)
+
+[Full Changelog](https://github.com/EasyPost/toggles/compare/v0.0.6...v0.0.7)
+
+**Closed issues:**
+
+- Allow Fixnums etc. as top-level subjects [\#1](https://github.com/EasyPost/toggles/issues/1)
+
+## [v0.0.6](https://github.com/EasyPost/toggles/tree/v0.0.6) (2014-12-22)
+
+[Full Changelog](https://github.com/EasyPost/toggles/compare/v0.0.5...v0.0.6)
+
+## [v0.0.5](https://github.com/EasyPost/toggles/tree/v0.0.5) (2014-12-17)
+
+[Full Changelog](https://github.com/EasyPost/toggles/compare/v0.0.4...v0.0.5)
+
+## [v0.0.4](https://github.com/EasyPost/toggles/tree/v0.0.4) (2014-12-10)
+
+[Full Changelog](https://github.com/EasyPost/toggles/compare/v0.0.3...v0.0.4)
+
+## [v0.0.3](https://github.com/EasyPost/toggles/tree/v0.0.3) (2014-12-10)
+
+[Full Changelog](https://github.com/EasyPost/toggles/compare/v0.0.2...v0.0.3)
+
+## [v0.0.2](https://github.com/EasyPost/toggles/tree/v0.0.2) (2014-12-10)
+
+[Full Changelog](https://github.com/EasyPost/toggles/compare/b19564d6ef5eb5d9c3b44d4eafe2fbac9d8a8938...v0.0.2)
+
+
+
+\* *This Changelog was automatically generated by [github_changelog_generator](https://github.com/github-changelog-generator/github-changelog-generator)*

data/Gemfile

@@ -1,3 +1,8 @@
 source 'https://rubygems.org'
 
 gemspec
+
+group :test do
+  gem "pry"
+  gem "pry-nav"
+end

data/LICENSE.txt

@@ -0,0 +1,13 @@
+Copyright (c) 2016-2016, Simpler Postage, Inc. <oss@easypost.com>
+
+Permission to use, copy, modify, and/or distribute this software for any
+purpose with or without fee is hereby granted, provided that the above
+copyright notice and this permission notice appear in all copies.
+
+THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES
+WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF
+MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR ANY
+SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES
+WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN ACTION
+OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF OR IN
+CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.

data/README.md

@@ -0,0 +1,76 @@
+# Toggles
+
+[![Gem Version](https://badge.fury.io/rb/toggles.svg)](https://badge.fury.io/rb/toggles)
+[![CI](https://github.com/EasyPost/toggles/workflows/CI/badge.svg)](https://github.com/EasyPost/toggles/actions?query=workflow%3ACI)
+
+YAML backed feature toggles
+
+## Installation
+
+Add the following to your Gemfile:
+
+```ruby
+gem "toggles"
+```
+
+and run `bundle install` from your shell.
+
+To install the gem manually from your shell, run:
+
+```ruby
+gem install toggles
+```
+
+## Configuration
+
+Configure `toggles`:
+
+```ruby
+Toggles.configure do |config|
+  config.features_dir = "features"
+end
+```
+
+You can now express conditional logic within `features_dir`. The structure of the `features_dir` determines the structure of the classes within the `Feature` module. For example if the `features_dir` has the structure:
+
+```
+features
+├── thing
+|   ├── one.yml
+|   └── two.yml
+└── test.yml
+```
+
+You can call the `Toggles.init` method to force re-parsing the configuration and re-initializing all Features
+structures at any time. The `Toggles.reinit_if_necessary` method is a convenience helper which will only
+re-initialize of the top-level features directory has changed. Note that, in general, this will only detect
+changes if you use a system where you swap out the entire features directory on changes and do not edit
+individual files within the directory.
+
+## Usage
+
+Create a file in `features_dir`:
+
+```yaml
+user:
+  id:
+    in:
+      - 12345
+      - 54321
+```
+
+Check if the feature is enabled or disabled:
+
+```ruby
+Feature.enabled?(:new_feature, :available_for_presentation, user: OpenStruct.new(id: 12345)) # true
+Feature.enabled?(:new_feature, :available_for_presentation, user: OpenStruct.new(id: 54321)) # true
+Feature.enabled?(:new_feature, :available_for_presentation, user: OpenStruct.new(id: 7)) # false
+
+Feature.disabled?(:new_feature, :available_for_presentation, user: OpenStruct.new(id: 12345)) # false
+Feature.disabled?(:new_feature, :available_for_presentation, user: OpenStruct.new(id: 54321)) # false
+Feature.disabled?(:new_feature, :available_for_presentation, user: OpenStruct.new(id: 7)) # true
+```
+
+## License
+
+This project is licensed under the ISC License, the contents of which can be found at [LICENSE.txt](LICENSE.txt).

data/features/abbreviations_cn22.yml

@@ -0,0 +1,3 @@
+---
+user:
+  id: 1

data/features/file_s3.yml

@@ -0,0 +1,3 @@
+---
+user:
+  id: 1

data/features/nested_foo/bar_baz.yml

@@ -0,0 +1,3 @@
+id:
+  lt:
+    5

data/features/s3_file.yml

@@ -0,0 +1,3 @@
+---
+user:
+  id: 1

data/lib/toggles/constant_lookup.rb

@@ -0,0 +1,50 @@
+class Feature::ConstantLookup
+  Error = Class.new(NameError) do
+    attr_reader :sym
+
+    def initialize(sym)
+      @sym = sym
+      super(sym.join('::'))
+    end
+  end
+
+  # Return a tree walker that translates Module#const_missing(sym) into the next child node
+  #
+  # So Features::Cat::BearDog walks as:
+  # * next_features = Feature.features # root
+  # * const_missing(:Cat) => next_features = next_features['cat']
+  # * const_missing(:BearDog) => next_features['bear_dog']
+  # * const_missing(:CN22) => next_features['c_n_22']
+  #
+  # Defined at Toggles.features_dir + "/cat/bear_dog.yaml"
+  #
+  # @raise [Error] if constant cannot be resolved
+  def self.from(features, path)
+    Class.new {
+      class << self
+        attr_accessor :features, :path
+
+        def const_missing(sym)
+          # translate class name into path part i.e :BearDog #=> 'bear_dog'
+          key = sym.to_s
+          key.gsub!(/([A-Z\d]+)([A-Z][a-z])/, '\1_\2'.freeze)
+          key.gsub!(/([a-z\d])([A-Z])/, '\1_\2'.freeze)
+          key.downcase!
+
+          subtree_or_feature = features.fetch(key.to_sym)
+
+          if subtree_or_feature.is_a?(Hash)
+            Feature::ConstantLookup.from(subtree_or_feature, path + [sym])
+          else
+            subtree_or_feature
+          end
+        rescue KeyError
+          raise Error, path + [sym]
+        end
+      end
+    }.tap do |resolver|
+      resolver.features = features
+      resolver.path = path
+    end
+  end
+end

data/lib/toggles/feature/attribute.rb

@@ -0,0 +1,15 @@
+class Feature::Attribute
+  def self.call(entity, attr_name, expected)
+    if expected.kind_of? Hash
+      expected.all? do |operation, rules|
+        if Feature.operations.include? operation.to_sym
+          Feature.operations[operation.to_sym].call(entity, attr_name, rules)
+        else
+          call(entity.send(attr_name), operation, rules)
+        end
+      end
+    else
+      entity.send(attr_name) == expected
+    end
+  end
+end

data/lib/toggles/feature/operation/and.rb

@@ -1,15 +0,0 @@
-module Feature
-  module Operation
-    class And
-      def self.call(entity, attr_name, expected)
-        expected.all? do |operation, value|
-          if OPERATIONS.include? operation.to_sym
-            OPERATIONS[operation.to_sym].call(entity, attr_name, value)
-          else
-            Operation::Attribute.call(entity, operation, value)
-          end
-        end
-      end
-    end
-  end
-end

data/lib/toggles/feature/operation/in.rb

@@ -4,7 +4,7 @@ module Feature
       def self.call(entity, attr_name, expected)
         if expected.kind_of? Hash
           expected = expected.reduce([]) do |list, (operation, args)|
-            OPERATIONS[operation.to_sym].call(args)
+            Feature.operations[operation.to_sym].call(args)
           end
         end
 

data/lib/toggles/feature/operation/range.rb

@@ -2,8 +2,6 @@ module Feature
   module Operation
     class Range
       def self.call(args)
-        raise StandardError, "Invalid range operation" if args.size != 2
-        (args.first..args.last)
       end
     end
   end

data/lib/toggles/feature/permissions.rb

@@ -1,43 +1,48 @@
-require "ostruct"
-require "forwardable"
+require 'ostruct'
+require 'forwardable'
+require 'yaml'
 
-module Feature
-  class Permissions
-    extend Forwardable
+Feature::Permissions = Struct.new(:rules) do
+  extend Forwardable
 
-    attr_reader :rules
+  def_delegators :rules, :all?, :keys
 
-    def_delegators :rules, :all?, :keys
+  def self.from_yaml(path)
+    new(
+      YAML.safe_load(File.read(path), permitted_classes: [Symbol])
+    )
+  end
 
-    def initialize(path)
-      @rules = YAML.load(File.read(path))
-    end
+  def subjects
+    @subjects ||= keys.map(&:to_sym)
+  end
 
-    def subjects
-      @subjects ||= keys.map(&:to_sym)
+  def valid_for?(entities)
+    unless subjects == entities.keys
+      raise Feature::Subject::Invalid, Feature::Subject.difference(subjects, entities.keys)
     end
 
-    def valid_for?(entities)
-      unless subjects == entities.keys
-        raise Subject::Invalid, Subject.difference(subjects, entities.keys)
-      end
-
-      rules.all? do |name, rule|
-        entity = entities[name.to_sym]
+    rules.all? do |name, rule|
+      entity = entities[name.to_sym]
 
-        if entity.nil?
-          return false
-        end
+      return false if entity.nil?
 
-        if entity.class.ancestors.find { |ancestor| ancestor == Comparable }
-          entity = OpenStruct.new(name => entity)
-          rule   = {name => rule}
-        end
+      if entity.class.ancestors.find { |ancestor| ancestor == Comparable }
+        entity = OpenStruct.new(name => entity)
+        rule   = { name => rule }
+      end
 
-        rule.all? do |key, value|
-          OPERATIONS.fetch(key.to_sym, Operation::Attribute).call(entity, key, value)
-        end
+      rule.all? do |key, value|
+        Feature.operations.fetch(key.to_sym, Feature::Attribute).call(entity, key, value)
       end
     end
   end
+
+  def enabled_for?(subjects = {})
+    valid_for?(subjects)
+  end
+
+  def disabled_for?(subjects = {})
+    !valid_for?(subjects)
+  end
 end

data/lib/toggles/feature.rb

@@ -1,16 +1,84 @@
-require "find"
+# frozen_string_literal: true
+ 
+module Feature
+  Error = Class.new(StandardError)
+  Unknown = Class.new(Error)
+
+  def self.features
+    @features ||= {}
+  end
+
+  def self.operations
+    @operations ||= {}
+  end
+
+  # @deprecated This is an abuse of lazy dispatch that creates cryptic errors
+  def self.const_missing(sym)
+    ConstantLookup.from(features, [:Feature]).const_missing(sym)
+  end
 
-require "toggles/feature/base"
-require "toggles/feature/operation"
+  def self.enabled?(*sym, **criteria)
+    sym
+      .inject(features) { |a, e| a.fetch(e) }
+      .enabled_for?(criteria)
+  rescue KeyError
+    raise Unknown, sym.inspect
+  end
+
+  def self.disabled?(*sym, **criteria)
+    !enabled?(*sym, **criteria)
+  end
+end
+
+require 'toggles/constant_lookup'
+require "toggles/feature/attribute"
 require "toggles/feature/permissions"
 require "toggles/feature/subject"
 
-module Feature
-  OPERATIONS = {and:   Operation::And,
-                gt:    Operation::GreaterThan,
-                in:    Operation::In,
-                lt:    Operation::LessThan,
-                not:   Operation::Not,
-                or:    Operation::Or,
-                range: Operation::Range}
-end
+Feature.operations[:and] = lambda { |entity, attr_name, expected|
+  expected.all? do |operation, value|
+    if Feature.operations.include? operation.to_sym
+      Feature.operations[operation.to_sym].call(entity, attr_name, value)
+    else
+      Feature::Attribute.call(entity, operation, value)
+    end
+  end
+}
+
+Feature.operations[:in] = lambda { |entity, attr_name, expected|
+  if expected.is_a? Hash
+    expected = expected.reduce([]) do |_list, (operation, args)|
+      Feature.operations[operation.to_sym].call(args)
+    end
+  end
+
+  expected.include? entity.send(attr_name.to_sym)
+}
+
+Feature.operations[:not] = lambda { |entity, attr_name, expected|
+  if expected.is_a? Hash
+    expected.none? do |operation, value|
+      Feature.operations[operation.to_sym].call(entity, attr_name, value)
+    end
+  else
+    entity.send(attr_name) != expected
+  end
+}
+
+Feature.operations[:or] = lambda { |entity, attr_name, expected|
+  expected.any? do |operation, value|
+    if Feature.operations.include? operation.to_sym
+      Feature.operations[operation.to_sym].call(entity, attr_name, value)
+    else
+      Feature::Attribute.call(entity, operation, value)
+    end
+  end
+}
+
+Feature.operations[:gt] = ->(entity, attr_name, expected) { entity.send(attr_name) > expected }
+Feature.operations[:lt] = ->(entity, attr_name, expected) { entity.send(attr_name) < expected }
+Feature.operations[:range] = lambda { |args|
+  raise StandardError, 'Invalid range operation' if args.size != 2
+
+  (args.first..args.last)
+}

data/lib/toggles/version.rb

@@ -0,0 +1,5 @@
+# frozen_string_literal: true
+
+module Toggles
+  VERSION = '0.4.0'
+end

data/lib/toggles.rb

@@ -1,10 +1,19 @@
-require "toggles/configuration"
-require "toggles/feature"
+require 'find'
+require 'pathname'
+require 'set'
+
+require 'toggles/version'
+
+require 'toggles/configuration'
+require 'toggles/feature'
 
 module Toggles
   extend self
 
+  StatResult = Struct.new(:inode, :mtime)
+
   def configure
+    @stat_tuple ||= StatResult.new(0, 0)
     yield configuration
     init
   end
@@ -13,43 +22,35 @@ module Toggles
     @configuration ||= Configuration.new
   end
 
-  # Dynamically create modules and classes within the `Feature` module based on
-  # the directory structure of `features`.
-  #
-  # For example if the `features` directory has the structure:
-  #
-  # features
-  # ├── thing
-  # |   ├── one.yml
-  # |   └── two.yml
-  # └── test.yml
-  #
-  # `Feature::Test`, `Feature::Thing::One`, `Feature::Thing::Two` would be
-  # available by default.
-  #
   def init
-    return unless Dir.exists? configuration.features_dir
-
-    Find.find(configuration.features_dir) do |path|
-      if path.match(/\.ya?ml\Z/)
-        _, *directories, filename = path.chomp(File.extname(path)).split("/")
-
-        previous = Feature
-        directories.each do |directory|
-          module_name = directory.split("_").map(&:capitalize).join.to_sym
-          previous    = if previous.constants.include? module_name
-                          previous.const_get(module_name)
-                        else
-                          previous.const_set(module_name, Module.new)
-                        end
-        end
-
-        cls = Class.new(Feature::Base) do |c|
-          c.const_set(:PERMISSIONS, Feature::Permissions.new(path))
-        end
-
-        previous.const_set(filename.split("_").map(&:capitalize).join.to_sym, cls)
-      end
+    return unless Dir.exist? configuration.features_dir
+
+    top_level = File.realpath(configuration.features_dir)
+    top_level_p = Pathname.new(top_level)
+
+    Feature.features.clear
+
+    Dir[File.join(top_level, '**/*.{yaml,yml}')].each do |abspath|
+      path = Pathname.new(abspath).relative_path_from(top_level_p).to_s
+      features = path.split('/')[0..-2].inject(Feature.features) { |a, e| a[e.to_sym] ||= {} }
+      feature_key = File.basename(path, File.extname(path)).to_sym
+      features[feature_key] = Feature::Permissions.from_yaml(abspath)
     end
+
+    stbuf = File.stat(top_level)
+    @stat_tuple = StatResult.new(stbuf.ino, stbuf.mtime)
+  end
+
+  def reinit_if_changed
+    # Reload the configuration if the top-level directory has changed.
+    # Does not detect changes to files inside that directory unless your
+    # filesystem propagates mtimes.
+    return unless Dir.exist? configuration.features_dir
+
+    top_level = File.realpath(configuration.features_dir)
+    stbuf = File.stat(top_level)
+    stat_tuple = StatResult.new(stbuf.ino, stbuf.mtime)
+
+    init if @stat_tuple != stat_tuple
   end
 end

data/spec/spec_helper.rb

@@ -1,11 +1,16 @@
 require "rspec/its"
+require 'rspec/temp_dir'
 
 require "toggles"
 
-Toggles.configure do |config|
-  config.features_dir = "features"
-end
+Bundler.require(:test)
 
 RSpec.configure do |config|
   config.order = "random"
+
+  config.before(:each) do
+    Toggles.configure do |c|
+      c.features_dir = "features"
+    end
+  end
 end

data/spec/toggles/feature/acceptance/collection_spec.rb

@@ -1,4 +1,4 @@
-describe Feature::Collection do
+describe "Feature::Collection" do
   specify do
     expect(Feature::Collection.enabled_for?(user: double(id: 1))).to eq true
     expect(Feature::Collection.enabled_for?(user: double(id: 5))).to eq true

data/spec/toggles/feature/acceptance/complex_and_spec.rb

@@ -1,4 +1,4 @@
-describe Feature::ComplexAnd do
+describe "Feature::ComplexAnd" do
   specify do
     expect(Feature::ComplexAnd.enabled_for?(data: double(id: 1))).to eq true
     expect(Feature::ComplexAnd.enabled_for?(data: double(id: 2, timestamp: 0))).to eq true

data/spec/toggles/feature/acceptance/multiple_subjects_spec.rb

@@ -1,4 +1,4 @@
-describe Feature::MultipleSubjects do
+describe "Feature::MultipleSubjects" do
   specify do
     expect(Feature::MultipleSubjects.enabled_for?(
       user: double(id: 1, logged_in?: true), widget: double(id: 2))).to eq true

data/spec/toggles/feature/acceptance/nested_attributes_spec.rb

@@ -1,4 +1,4 @@
-describe Feature::NestedAttributes do
+describe "Feature::NestedAttributes" do
   specify do
     expect(Feature::NestedAttributes.enabled_for?(
         foo: double(bar: :two, baz: double(id: 51)))).to eq true