From 28bc5a6c89dd2be26b293e1b5a5f7bbf32e2ceea Mon Sep 17 00:00:00 2001 From: Gosha Arinich Date: Sat, 2 Mar 2013 10:40:32 +0300 Subject: [PATCH] make Enum#valid? specs test the right thing --- spec/components/enum_spec.rb | 28 +++++++++++++--------------- 1 file changed, 13 insertions(+), 15 deletions(-) diff --git a/spec/components/enum_spec.rb b/spec/components/enum_spec.rb index aadf954e384..9a5f1f793ef 100644 --- a/spec/components/enum_spec.rb +++ b/spec/components/enum_spec.rb @@ -2,39 +2,37 @@ require 'spec_helper' require 'email' describe Enum do - let(:enum) { Enum.new(:jake, :finn, :princess_bubblegum, :peppermint_butler) } - context ".[]" do - it "allows us to look up a number by symbol" do + describe ".[]" do + it "looks up a number by symbol" do enum[:princess_bubblegum].should == 3 end - it "allows us to look up a symbol by number" do + it "looks up a symbol by number" do enum[2].should == :finn end end - context ".valid?" do - it "returns true for a value that exists" do - enum.valid?(4).should be_false + describe ".valid?" do + it "returns true if a key exists" do + enum.valid?(:finn).should be_true end - it "returns true for a key that doesn't exist" do - enum.valid?(:ice_king).should be_false + it "returns false if a key does not exist" do + enum.valid?(:obama).should be_false end end - context ".only" do + describe ".only" do it "returns only the values we ask for" do - enum.only(:jake, :princess_bubblegum).should == {jake: 1, princess_bubblegum: 3} + enum.only(:jake, :princess_bubblegum).should == { jake: 1, princess_bubblegum: 3 } end end - context ".except" do - it "doesn't return the values we don't want" do - enum.except(:jake, :princess_bubblegum).should == {finn: 2, peppermint_butler: 4} + describe ".except" do + it "returns everything but the values we ask to delete" do + enum.except(:jake, :princess_bubblegum).should == { finn: 2, peppermint_butler: 4 } end end - end