Skip to content

Commit

Permalink
fix spec tests
Browse files Browse the repository at this point in the history
  • Loading branch information
igrigorik committed Mar 17, 2012
1 parent 91ec1ab commit 67500f8
Show file tree
Hide file tree
Showing 4 changed files with 15 additions and 11 deletions.
7 changes: 5 additions & 2 deletions Gemfile.lock
@@ -1,15 +1,17 @@
PATH
remote: .
specs:
bloomfilter-rb (2.0.0)
bloomfilter-rb (2.1.1)
redis

GEM
remote: http://rubygems.org/
specs:
diff-lcs (1.1.2)
rake (0.8.7)
redis (2.2.0)
rake-compiler (0.8.0)
rake
redis (2.2.2)
rspec (2.5.0)
rspec-core (~> 2.5.0)
rspec-expectations (~> 2.5.0)
Expand All @@ -25,4 +27,5 @@ PLATFORMS
DEPENDENCIES
bloomfilter-rb!
rake
rake-compiler
rspec
5 changes: 3 additions & 2 deletions spec/counting_redis_spec.rb
@@ -1,7 +1,8 @@
require 'helper'

describe BloomFilter::CountingRedis do
include BloomFilter
include BloomFilter

describe CountingRedis do

context "use Redis for storage" do
it "should store data in Redis" do
Expand Down
5 changes: 3 additions & 2 deletions spec/native_spec.rb
@@ -1,7 +1,8 @@
require 'helper'

describe BloomFilter::Native do
include BloomFilter
include BloomFilter

describe Native do

it "should clear" do
bf = Native.new(:size => 100, :hashes => 2, :seed => 1, :bucket => 3, :raise => false)
Expand Down
9 changes: 4 additions & 5 deletions spec/redis_spec.rb
@@ -1,10 +1,9 @@
require 'helper'

describe BloomFilter::Redis do
include BloomFilter

context "use Redis bitstring for storage" do
let(:bf) { Redis.new }
let(:bf) { BloomFilter::Redis.new }

it "should store data in Redis" do
bf.insert(:abcd)
Expand Down Expand Up @@ -39,12 +38,12 @@
end

it "should connect to remote redis server" do
lambda { Redis.new }.should_not raise_error
lambda { BloomFilter::Redis.new }.should_not raise_error
end

it "should allow namespaced BloomFilters" do
bf1 = Redis.new(:namespace => :a)
bf2 = Redis.new(:namespace => :b)
bf1 = BloomFilter::Redis.new(:namespace => :a)
bf2 = BloomFilter::Redis.new(:namespace => :b)

bf1.insert('test')
bf1.include?('test').should be_true
Expand Down

0 comments on commit 67500f8

Please sign in to comment.