mashed 0.5.1 → 0.9.0

data/.travis.yml

@@ -0,0 +1,10 @@
+language: ruby
+rvm:
+  - "1.9.3"
+  - "2.0.0"
+# - jruby-18mode # JRuby in 1.8 mode
+  - jruby-19mode # JRuby in 1.9 mode
+# - rbx-18mode
+# - rbx-19mode
+# uncomment this line if your project needs to run something other than `rake`:
+# script: bundle exec rspec spec

data/README.md

@@ -33,6 +33,20 @@ tasks.each do |task|
 end
 ```
 
+### A Mash Is Not A Hash
+
+```ruby
+Mash = Mashed::Mash
+
+m = Mash.new(a: "A", b: "B")
+m.map { |key, value| puts [key, value].inspect } # => raise
+
+# assuming activesupport is present...
+
+m.with_indifferent_access # => nil
+# there is no key of "with_indifferent_access" for the internal hash
+```
+
 ## Contributing
 
 1. Fork it
@@ -40,3 +54,28 @@ end
 3. Commit your changes (`git commit -am 'Add some feature'`)
 4. Push to the branch (`git push origin my-new-feature`)
 5. Create new Pull Request
+
+## StringyHash
+
+To help make the Mash work there also is a StringyHash class. It's a
+simple delegate to the normal Hash except that it tries to enforce that
+all incoming and outgoing keys are strings.
+
+### Usage
+
+```ruby
+StringyHash = Mash::StringyHash
+
+sh = StringyHash.new(title: "Hello", starred: false, completed_at: nil)
+sh.keys # => ["title", "starred", "completed_at"]
+sh[:title] # => "Hello"
+sh["title"] # => "Hello"
+
+class Title
+  def to_s
+    "title"
+  end
+end
+
+sh[Title.new] # => "Hello"
+```

data/Rakefile

@@ -1 +1,6 @@
 require "bundler/gem_tasks"
+require 'rspec/core/rake_task'
+
+RSpec::Core::RakeTask.new(:spec)
+
+task :default => :spec

data/lib/mashed/mash.rb

@@ -54,34 +54,26 @@ module Mashed
     end
     alias kind_of? is_a?
 
-    def [](key)
-      key = key.to_s
-      wrap_up @hash[key]
-    end
-
-    def []=(key, value)
-      key = key.to_s
-      @hash[key] = value
-    end
-
-    def keys
-      @hash.keys
-    end
-
     def delete(key)
       wrap_up @hash.delete(key)
     end
 
     def method_missing(symbol, *args, &blk)
       string = symbol.to_s
-      if @hash.key?(string)
-        self[string]
-      elsif string =~ /\?$/
-        !!self[string[0..-2]]
-      elsif string =~ /=$/
+      if blk
+        super
+      elsif args.length == 0
+        if @hash.key?(string)
+          self[string]
+        elsif string =~ /\?$/
+          !!self[string[0..-2]]
+        else
+          nil
+        end
+      elsif args.length == 1 && string =~ /=$/
         self[string[0..-2]] = args.first
       else
-        nil
+        super
       end
     end
 
@@ -104,6 +96,20 @@ module Mashed
 
     private
 
+    def [](key)
+      key = key.to_s
+      wrap_up @hash[key]
+    end
+
+    def []=(key, value)
+      key = key.to_s
+      @hash[key] = value
+    end
+
+    def keys
+      @hash.keys
+    end
+
     def wrap_up(thing)
       if thing.respond_to?(:to_ary)
         thing.map { |t| wrap(t) }

data/lib/mashed/version.rb

@@ -1,3 +1,3 @@
 module Mashed
-  VERSION = "0.5.1"
+  VERSION = "0.9.0"
 end

data/spec/mashed/mash_spec.rb

@@ -11,18 +11,9 @@ describe Mashed::Mash do
     it { expect(mash.class).to be(Mashed::Mash) }
   end
 
-  describe "#keys" do
-    it { expect(mash.keys).to eq(["a", "b", "c"]) }
-  end
-
   describe "#delete" do
     before { mash.delete(:a) }
-    it { expect(mash.keys).to eq(["b", "c"]) }
-  end
-
-  describe "#[]" do
-    it { expect(mash[:a]).to eq(1) }
-    it { expect(mash["b"]).to eq(2) }
+    it { expect(mash.methods).to include("b", "c") }
   end
 
   describe "#method_missing" do
@@ -31,6 +22,18 @@ describe Mashed::Mash do
       expect(mash.b).to eq(2)
       expect(mash.c).to eq(3)
     }
+
+    context "for actual missing keys" do
+      it { expect(mash.d).to be_nil }
+    end
+
+    context "for methods with an arity > 0 that are not setters" do
+      it { expect { mash.something(1) }.to raise_error(StandardError) }
+    end
+
+    context "for methods with block" do
+      it { expect { mash.map { |k,v| [k,v] } }.to raise_error(StandardError) }
+    end
   end
 
   describe "#method_missing=" do
@@ -44,7 +47,7 @@ describe Mashed::Mash do
 
   describe "respond_to?" do
     it { expect(mash.respond_to?(:a)).to be_true }
-    it { expect(mash.respond_to?(:[])).to be_true }
+    it { expect(mash.respond_to?(:delete)).to be_true }
     it { expect(mash.respond_to?(:to_hash)).to be_true }
   end
 
@@ -89,4 +92,15 @@ describe Mashed::Mash do
       mash.to_json
     }
   end
+
+  context "private methods" do
+    describe "#keys" do
+      it { expect(mash.__send__(:keys)).to eq(["a", "b", "c"]) }
+    end
+
+    describe "#[]" do
+      it { expect(mash.__send__(:[],:a)).to eq(1) }
+      it { expect(mash.__send__(:[],"b")).to eq(2) }
+    end
+  end
 end

data/spec/mashed/stringy_hash_spec.rb

@@ -13,4 +13,14 @@ describe Mashed::StringyHash do
     it { expect(s[:a]).to eq("A") }
     it { expect(s["a"]).to eq("A") }
   end
+
+  describe "calls to_s on objects" do
+    let(:s) { hash.stringify }
+    let(:klass) {
+      Class.new do
+        def to_s; "a"; end
+      end
+    }
+    it { expect(s[klass.new]).to eq(1) }
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: mashed
 version: !ruby/object:Gem::Version
-  version: 0.5.1
+  version: 0.9.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-01-05 00:00:00.000000000 Z
+date: 2014-01-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -69,6 +69,7 @@ files:
 - .gitignore
 - .rspec
 - .ruby-version
+- .travis.yml
 - Gemfile
 - LICENSE.txt
 - README.md