vines 0.3.2 → 0.4.0

data/test/stanza/pubsub/subscribe_test.rb

@@ -0,0 +1,186 @@
+# encoding: UTF-8
+
+require 'tmpdir'
+require 'vines'
+require 'ext/nokogiri'
+require 'minitest/autorun'
+
+class SubscribePubSubTest < MiniTest::Unit::TestCase
+  def setup
+    @user = Vines::User.new(jid: 'alice@wonderland.lit/tea')
+    @config = Vines::Config.new do
+      host 'wonderland.lit' do
+        storage(:fs) { dir Dir.tmpdir }
+        pubsub 'games'
+      end
+    end
+    @stream = MiniTest::Mock.new
+    @stream.expect(:config, @config)
+    @stream.expect(:user, @user)
+  end
+
+  def test_missing_to_address_raises
+    node = node(%q{
+      <iq type='set' id='42'>
+        <pubsub xmlns='http://jabber.org/protocol/pubsub'>
+          <subscribe node='game_13' jid="alice@wonderland.lit/tea"/>
+        </pubsub>
+      </iq>
+    }.strip.gsub(/\n|\s{2,}/, ''))
+
+    @stream.expect(:domain, 'wonderland.lit')
+
+    stanza = Vines::Stanza::PubSub::Subscribe.new(node, @stream)
+    assert_raises(Vines::StanzaErrors::FeatureNotImplemented) { stanza.process }
+    assert @stream.verify
+  end
+
+  def test_server_domain_to_address_raises
+    node = node(%q{
+      <iq type='set' to='wonderland.lit' id='42'>
+        <pubsub xmlns='http://jabber.org/protocol/pubsub'>
+          <subscribe node='game_13' jid="alice@wonderland.lit/tea"/>
+        </pubsub>
+      </iq>
+    }.strip.gsub(/\n|\s{2,}/, ''))
+
+    stanza = Vines::Stanza::PubSub::Subscribe.new(node, @stream)
+    assert_raises(Vines::StanzaErrors::FeatureNotImplemented) { stanza.process }
+    assert @stream.verify
+  end
+
+  def test_non_pubsub_to_address_routes
+    node = node(%q{
+      <iq type='set' to='bogus.wonderland.lit' id='42'>
+        <pubsub xmlns='http://jabber.org/protocol/pubsub'>
+          <subscribe node='game_13' jid="alice@wonderland.lit/tea"/>
+        </pubsub>
+      </iq>
+    }.strip.gsub(/\n|\s{2,}/, ''))
+
+    router = MiniTest::Mock.new
+    router.expect(:route, nil, [node])
+
+    @stream.expect(:router, router)
+
+    stanza = Vines::Stanza::PubSub::Subscribe.new(node, @stream)
+    stanza.process
+    assert @stream.verify
+    assert router.verify
+  end
+
+  def test_multiple_subscribe_elements_raises
+    node = node(%q{
+      <iq type='set' to='games.wonderland.lit' id='42'>
+        <pubsub xmlns='http://jabber.org/protocol/pubsub'>
+          <subscribe node='game_13' jid="alice@wonderland.lit/tea"/>
+          <subscribe node='game_14' jid="alice@wonderland.lit/tea"/>
+        </pubsub>
+      </iq>
+    }.strip.gsub(/\n|\s{2,}/, ''))
+
+    stanza = Vines::Stanza::PubSub::Subscribe.new(node, @stream)
+    assert_raises(Vines::StanzaErrors::BadRequest) { stanza.process }
+    assert @stream.verify
+  end
+
+  def test_subscribe_missing_node_raises
+    node = node(%q{
+      <iq type='set' to='games.wonderland.lit' id='42'>
+        <pubsub xmlns='http://jabber.org/protocol/pubsub'>
+          <subscribe node='game_13' jid="alice@wonderland.lit/tea"/>
+        </pubsub>
+      </iq>
+    }.strip.gsub(/\n|\s{2,}/, ''))
+
+    stanza = Vines::Stanza::PubSub::Subscribe.new(node, @stream)
+    assert_raises(Vines::StanzaErrors::ItemNotFound) { stanza.process }
+    assert @stream.verify
+  end
+
+  def test_subscribe_twice_raises
+    node = node(%q{
+      <iq type='set' to='games.wonderland.lit' id='42'>
+        <pubsub xmlns='http://jabber.org/protocol/pubsub'>
+          <subscribe node='game_13' jid="alice@wonderland.lit/tea"/>
+        </pubsub>
+      </iq>
+    }.strip.gsub(/\n|\s{2,}/, ''))
+
+    stanza = Vines::Stanza::PubSub::Subscribe.new(node, @stream)
+    def stanza.mock_pubsub; @mock_pubsub; end
+    def stanza.pubsub
+      unless @mock_pubsub
+        @mock_pubsub = MiniTest::Mock.new
+        @mock_pubsub.expect(:node?, true, ['game_13'])
+        @mock_pubsub.expect(:subscribed?, true, ['game_13', 'alice@wonderland.lit/tea'])
+      end
+      @mock_pubsub
+    end
+    assert_raises(Vines::StanzaErrors::PolicyViolation) { stanza.process }
+    assert @stream.verify
+    assert stanza.mock_pubsub.verify
+  end
+
+  def test_subscribe_illegal_jid_raises
+    node = node(%q{
+      <iq type='set' to='games.wonderland.lit' id='42'>
+        <pubsub xmlns='http://jabber.org/protocol/pubsub'>
+          <subscribe node='game_13' jid="not_alice@wonderland.lit/tea"/>
+        </pubsub>
+      </iq>
+    }.strip.gsub(/\n|\s{2,}/, ''))
+
+    stanza = Vines::Stanza::PubSub::Subscribe.new(node, @stream)
+    assert_raises(Vines::StanzaErrors::BadRequest) { stanza.process }
+    assert @stream.verify
+  end
+
+  def test_good_stanza_processes
+    node = node(%q{
+      <iq type='set' to='games.wonderland.lit' id='42'>
+        <pubsub xmlns='http://jabber.org/protocol/pubsub'>
+          <subscribe node='game_13' jid="alice@wonderland.lit/tea"/>
+        </pubsub>
+      </iq>
+    }.strip.gsub(/\n|\s{2,}/, ''))
+
+    def @stream.nodes; @nodes; end
+    def @stream.write(node)
+      @nodes ||= []
+      @nodes << node
+    end
+
+    stanza = Vines::Stanza::PubSub::Subscribe.new(node, @stream)
+    def stanza.mock_pubsub; @mock_pubsub; end
+    def stanza.pubsub
+      unless @mock_pubsub
+        @mock_pubsub = MiniTest::Mock.new
+        @mock_pubsub.expect(:node?, true, ['game_13'])
+        @mock_pubsub.expect(:subscribed?, false, ['game_13', 'alice@wonderland.lit/tea'])
+        @mock_pubsub.expect(:subscribe, nil, ['game_13', 'alice@wonderland.lit/tea'])
+      end
+      @mock_pubsub
+    end
+    stanza.process
+
+    assert @stream.verify
+    assert stanza.mock_pubsub.verify
+    assert_equal 1, @stream.nodes.size
+
+    expected = node(%q{
+      <iq from="games.wonderland.lit" id="42" to="alice@wonderland.lit/tea" type="result">
+          <pubsub xmlns="http://jabber.org/protocol/pubsub">
+            <subscription node="game_13" jid="alice@wonderland.lit/tea" subscription="subscribed"/>
+          </pubsub>
+        </iq>
+      }.strip.gsub(/\n|\s{2,}/, ''))
+    assert_equal expected, @stream.nodes[0]
+  end
+
+  private
+
+  def node(xml)
+    Nokogiri::XML(xml).root
+  end
+end

data/test/stanza/pubsub/unsubscribe_test.rb

@@ -0,0 +1,179 @@
+# encoding: UTF-8
+
+require 'tmpdir'
+require 'vines'
+require 'ext/nokogiri'
+require 'minitest/autorun'
+
+class UnsubscribePubSubTest < MiniTest::Unit::TestCase
+  def setup
+    @user = Vines::User.new(jid: 'alice@wonderland.lit/tea')
+    @config = Vines::Config.new do
+      host 'wonderland.lit' do
+        storage(:fs) { dir Dir.tmpdir }
+        pubsub 'games'
+      end
+    end
+    @stream = MiniTest::Mock.new
+    @stream.expect(:config, @config)
+    @stream.expect(:user, @user)
+  end
+
+  def test_missing_to_address_raises
+    node = node(%q{
+      <iq type='set' id='42'>
+        <pubsub xmlns='http://jabber.org/protocol/pubsub'>
+          <unsubscribe node='game_13' jid="alice@wonderland.lit/tea"/>
+        </pubsub>
+      </iq>
+    }.strip.gsub(/\n|\s{2,}/, ''))
+
+    @stream.expect(:domain, 'wonderland.lit')
+
+    stanza = Vines::Stanza::PubSub::Unsubscribe.new(node, @stream)
+    assert_raises(Vines::StanzaErrors::FeatureNotImplemented) { stanza.process }
+    assert @stream.verify
+  end
+
+  def test_server_domain_to_address_raises
+    node = node(%q{
+      <iq type='set' to='wonderland.lit' id='42'>
+        <pubsub xmlns='http://jabber.org/protocol/pubsub'>
+          <unsubscribe node='game_13' jid="alice@wonderland.lit/tea"/>
+        </pubsub>
+      </iq>
+    }.strip.gsub(/\n|\s{2,}/, ''))
+
+    stanza = Vines::Stanza::PubSub::Unsubscribe.new(node, @stream)
+    assert_raises(Vines::StanzaErrors::FeatureNotImplemented) { stanza.process }
+    assert @stream.verify
+  end
+
+  def test_non_pubsub_to_address_routes
+    node = node(%q{
+      <iq type='set' to='bogus.wonderland.lit' id='42'>
+        <pubsub xmlns='http://jabber.org/protocol/pubsub'>
+          <unsubscribe node='game_13' jid="alice@wonderland.lit/tea"/>
+        </pubsub>
+      </iq>
+    }.strip.gsub(/\n|\s{2,}/, ''))
+
+    router = MiniTest::Mock.new
+    router.expect(:route, nil, [node])
+    @stream.expect(:router, router)
+
+    stanza = Vines::Stanza::PubSub::Unsubscribe.new(node, @stream)
+    stanza.process
+    assert @stream.verify
+    assert router.verify
+  end
+
+  def test_multiple_unsubscribe_elements_raises
+    node = node(%q{
+      <iq type='set' to='games.wonderland.lit' id='42'>
+        <pubsub xmlns='http://jabber.org/protocol/pubsub'>
+          <unsubscribe node='game_13' jid="alice@wonderland.lit/tea"/>
+          <unsubscribe node='game_14' jid="alice@wonderland.lit/tea"/>
+        </pubsub>
+      </iq>
+    }.strip.gsub(/\n|\s{2,}/, ''))
+
+    stanza = Vines::Stanza::PubSub::Unsubscribe.new(node, @stream)
+    assert_raises(Vines::StanzaErrors::BadRequest) { stanza.process }
+    assert @stream.verify
+  end
+
+  def test_unsubscribe_missing_node_raises
+    node = node(%q{
+      <iq type='set' to='games.wonderland.lit' id='42'>
+        <pubsub xmlns='http://jabber.org/protocol/pubsub'>
+          <unsubscribe node='game_13' jid="alice@wonderland.lit/tea"/>
+        </pubsub>
+      </iq>
+    }.strip.gsub(/\n|\s{2,}/, ''))
+
+    stanza = Vines::Stanza::PubSub::Unsubscribe.new(node, @stream)
+    assert_raises(Vines::StanzaErrors::ItemNotFound) { stanza.process }
+    assert @stream.verify
+  end
+
+  def test_unsubscribe_without_subscription_raises
+    node = node(%q{
+      <iq type='set' to='games.wonderland.lit' id='42'>
+        <pubsub xmlns='http://jabber.org/protocol/pubsub'>
+          <unsubscribe node='game_13' jid="alice@wonderland.lit/tea"/>
+        </pubsub>
+      </iq>
+    }.strip.gsub(/\n|\s{2,}/, ''))
+
+    stanza = Vines::Stanza::PubSub::Unsubscribe.new(node, @stream)
+    def stanza.mock_pubsub; @mock_pubsub; end
+    def stanza.pubsub
+      unless @mock_pubsub
+        @mock_pubsub = MiniTest::Mock.new
+        @mock_pubsub.expect(:node?, true, ['game_13'])
+        @mock_pubsub.expect(:subscribed?, false, ['game_13', 'alice@wonderland.lit/tea'])
+      end
+      @mock_pubsub
+    end
+    assert_raises(Vines::StanzaErrors::UnexpectedRequest) { stanza.process }
+    assert @stream.verify
+    assert stanza.mock_pubsub.verify
+  end
+
+  def test_unsubscribe_illegal_jid_raises
+    node = node(%q{
+      <iq type='set' to='games.wonderland.lit' id='42'>
+        <pubsub xmlns='http://jabber.org/protocol/pubsub'>
+          <unsubscribe node='game_13' jid="not_alice@wonderland.lit/tea"/>
+        </pubsub>
+      </iq>
+    }.strip.gsub(/\n|\s{2,}/, ''))
+
+    stanza = Vines::Stanza::PubSub::Unsubscribe.new(node, @stream)
+    assert_raises(Vines::StanzaErrors::Forbidden) { stanza.process }
+    assert @stream.verify
+  end
+
+  def test_good_stanza_processes
+    node = node(%q{
+      <iq type='set' to='games.wonderland.lit' id='42'>
+        <pubsub xmlns='http://jabber.org/protocol/pubsub'>
+          <unsubscribe node='game_13' jid="alice@wonderland.lit/tea"/>
+        </pubsub>
+      </iq>
+    }.strip.gsub(/\n|\s{2,}/, ''))
+
+    def @stream.nodes; @nodes; end
+    def @stream.write(node)
+      @nodes ||= []
+      @nodes << node
+    end
+
+    stanza = Vines::Stanza::PubSub::Unsubscribe.new(node, @stream)
+    def stanza.mock_pubsub; @mock_pubsub; end
+    def stanza.pubsub
+      unless @mock_pubsub
+        @mock_pubsub = MiniTest::Mock.new
+        @mock_pubsub.expect(:node?, true, ['game_13'])
+        @mock_pubsub.expect(:subscribed?, true, ['game_13', 'alice@wonderland.lit/tea'])
+        @mock_pubsub.expect(:unsubscribe, nil, ['game_13', 'alice@wonderland.lit/tea'])
+      end
+      @mock_pubsub
+    end
+    stanza.process
+
+    assert @stream.verify
+    assert stanza.mock_pubsub.verify
+    assert_equal 1, @stream.nodes.size
+
+    expected = node(%q{<iq from="games.wonderland.lit" id="42" to="alice@wonderland.lit/tea" type="result"/>})
+    assert_equal expected, @stream.nodes[0]
+  end
+
+  private
+
+  def node(xml)
+    Nokogiri::XML(xml).root
+  end
+end

data/test/stanza_test.rb

@@ -1,5 +1,6 @@
 # encoding: UTF-8
 
+require 'tmpdir'
 require 'vines'
 require 'ext/nokogiri'
 require 'minitest/autorun'
@@ -11,7 +12,7 @@ class StanzaTest < MiniTest::Unit::TestCase
     @stream = MiniTest::Mock.new
     @config = Vines::Config.new do
       host 'wonderland.lit' do
-        storage(:fs) { dir '.' }
+        storage(:fs) { dir Dir.tmpdir }
       end
     end
   end

data/test/storage/local_test.rb

@@ -1,27 +1,35 @@
 # encoding: UTF-8
 
 require 'storage_tests'
+require 'tmpdir'
 require 'vines'
 require 'minitest/autorun'
 
 class LocalTest < MiniTest::Unit::TestCase
   include StorageTests
 
+  DIR = Dir.mktmpdir
+
   def setup
-    @files = {
-      :empty      => './empty@wonderland.lit.user',
-      :no_pass    => './no_password@wonderland.lit.user',
-      :clear_pass => './clear_password@wonderland.lit.user',
-      :bcrypt     => './bcrypt_password@wonderland.lit.user',
-      :full       => './full@wonderland.lit.user',
-      :vcard      => './full@wonderland.lit.vcard',
-      :fragment   => "./full@wonderland.lit-#{FRAGMENT_ID}.fragment"
+    Dir.mkdir(DIR) unless File.exists?(DIR)
+    %w[user vcard fragment].each do |d|
+      Dir.mkdir(File.join(DIR, d))
+    end
+
+    files = {
+      :empty      => "#{DIR}/user/empty@wonderland.lit",
+      :no_pass    => "#{DIR}/user/no_password@wonderland.lit",
+      :clear_pass => "#{DIR}/user/clear_password@wonderland.lit",
+      :bcrypt     => "#{DIR}/user/bcrypt_password@wonderland.lit",
+      :full       => "#{DIR}/user/full@wonderland.lit",
+      :vcard      => "#{DIR}/vcard/full@wonderland.lit",
+      :fragment   => "#{DIR}/fragment/full@wonderland.lit-#{FRAGMENT_ID}"
     }
-    File.open(@files[:empty], 'w') {|f| f.write('') }
-    File.open(@files[:no_pass], 'w') {|f| f.write('foo: bar') }
-    File.open(@files[:clear_pass], 'w') {|f| f.write('password: secret') }
-    File.open(@files[:bcrypt], 'w') {|f| f.write("password: #{BCrypt::Password.create('secret')}") }
-    File.open(@files[:full], 'w') do |f|
+    File.open(files[:empty], 'w') {|f| f.write('') }
+    File.open(files[:no_pass], 'w') {|f| f.write('foo: bar') }
+    File.open(files[:clear_pass], 'w') {|f| f.write('password: secret') }
+    File.open(files[:bcrypt], 'w') {|f| f.write("password: #{BCrypt::Password.create('secret')}") }
+    File.open(files[:full], 'w') do |f|
       f.puts("password: #{BCrypt::Password.create('secret')}")
       f.puts("name: Tester")
       f.puts("roster:")
@@ -32,29 +40,22 @@ class LocalTest < MiniTest::Unit::TestCase
       f.puts("    name: Contact2")
       f.puts("    groups: [Group3, Group4]")
     end
-    File.open(@files[:vcard], 'w') {|f| f.write(StorageTests::VCARD.to_xml) }
-    File.open(@files[:fragment], 'w') {|f| f.write(StorageTests::FRAGMENT.to_xml) }
+    File.open(files[:vcard], 'w') {|f| f.write(StorageTests::VCARD.to_xml) }
+    File.open(files[:fragment], 'w') {|f| f.write(StorageTests::FRAGMENT.to_xml) }
   end
 
   def teardown
-    misc = %w[user vcard].map {|ext| "./save_user@domain.tld.#{ext}" }
-    [
-      "./save_user@domain.tld-#{FRAGMENT_ID}.fragment",
-      *misc,
-      *@files.values
-    ].each do |f|
-      File.delete(f) if File.exist?(f)
-    end
+    FileUtils.remove_entry_secure(DIR)
   end
 
   def storage
-    Vines::Storage::Local.new { dir '.' }
+    Vines::Storage::Local.new { dir DIR }
   end
 
   def test_init
     assert_raises(RuntimeError) { Vines::Storage::Local.new {} }
     assert_raises(RuntimeError) { Vines::Storage::Local.new { dir 'bogus' } }
     assert_raises(RuntimeError) { Vines::Storage::Local.new { dir '/sbin' } }
-    Vines::Storage::Local.new { dir '.' } # shouldn't raise an error
+    Vines::Storage::Local.new { dir DIR } # shouldn't raise an error
   end
 end

data/test/storage/mock_mongo.rb

@@ -0,0 +1,40 @@
+# encoding: UTF-8
+
+# A mock implementation of the Mongo::Connection class.
+class MockMongo
+  def initialize
+    @collections = collections
+  end
+
+  def collection(name)
+    @collections[name]
+  end
+
+  def clear
+    @collections = collections
+  end
+
+  def collections
+    {
+      users: MockCollection.new,
+      vcards: MockCollection.new,
+      fragments: MockCollection.new
+    }
+  end
+
+  class MockCollection
+    def initialize
+      @docs = {}
+    end
+
+    def find_one(doc)
+      id = doc['_id'] || doc[:_id]
+      @docs[id]
+    end
+
+    def save(doc, opts={})
+      id = doc['_id'] || doc[:_id]
+      @docs[id] = doc
+    end
+  end
+end

data/test/storage/mock_redis.rb

@@ -0,0 +1,98 @@
+# encoding: UTF-8
+
+# A mock redis storage implementation that saves data to an in-memory Hash.
+class MockRedis
+  attr_reader :db
+
+  # Mimic em-hiredis behavior.
+  def self.defer(method)
+    old = "_deferred_#{method}"
+    alias_method old, method
+    define_method method do |*args, &block|
+      result = method(old).call(*args)
+      deferred = EM::DefaultDeferrable.new
+      deferred.callback(&block) if block
+      EM.next_tick { deferred.succeed(result) }
+      deferred
+    end
+  end
+
+  def initialize
+    @db = {}
+  end
+
+  def del(key)
+    @db.delete(key)
+  end
+  defer :del
+
+  def get(key)
+    @db[key]
+  end
+  defer :get
+
+  def set(key, value)
+    @db[key] = value
+  end
+  defer :set
+
+  def hget(key, field)
+    @db[key][field] rescue nil
+  end
+  defer :hget
+
+  def hdel(key, field)
+    @db[key].delete(field) rescue nil
+  end
+  defer :hdel
+
+  def hgetall(key)
+    (@db[key] || {}).map do |k, v|
+      [k, v]
+    end.flatten
+  end
+  defer :hgetall
+
+  def hset(key, field, value)
+    @db[key] ||= {}
+    @db[key][field] = value
+  end
+  defer :hset
+
+  def hmset(key, *args)
+    @db[key] = Hash[*args]
+  end
+  defer :hmset
+
+  def sadd(key, obj)
+    @db[key] ||= Set.new
+    @db[key] << obj
+  end
+  defer :sadd
+
+  def srem(key, obj)
+    @db[key].delete(obj) rescue nil
+  end
+  defer :srem
+
+  def smembers
+    @db[key].to_a rescue []
+  end
+  defer :smembers
+
+  def flushdb
+    @db.clear
+  end
+  defer :flushdb
+
+  def multi
+    @transaction = true
+  end
+  defer :multi
+
+  def exec
+    raise 'transaction must start with multi' unless @transaction
+    @transaction = false
+  end
+  defer :exec
+end

data/test/storage/mongodb_test.rb

@@ -0,0 +1,81 @@
+# encoding: UTF-8
+
+require 'mock_mongo'
+require 'storage_tests'
+require 'vines'
+require 'minitest/autorun'
+
+class MongoDBTest < MiniTest::Unit::TestCase
+  include StorageTests
+
+  MOCK_MONGO = MockMongo.new
+
+  def setup
+    EMLoop.new do
+      db = MOCK_MONGO
+      db.collection(:users).save({'_id' => 'empty@wonderland.lit'})
+      db.collection(:users).save({'_id' => 'no_password@wonderland.lit', 'foo' => 'bar'})
+      db.collection(:users).save({'_id' => 'clear_password@wonderland.lit', 'password' => 'secret'})
+      db.collection(:users).save({'_id' => 'bcrypt_password@wonderland.lit', 'password' => BCrypt::Password.create('secret')})
+      db.collection(:users).save({
+        '_id'      => 'full@wonderland.lit',
+        'password' => BCrypt::Password.create('secret'),
+        'name'     => 'Tester',
+        'roster'   => {
+          'contact1@wonderland.lit' => {
+            'name'   => 'Contact1',
+            'groups' => %w[Group1 Group2]
+          },
+          'contact2@wonderland.lit' => {
+            'name'   => 'Contact2',
+            'groups' => %w[Group3 Group4]
+          }
+        }
+      })
+      db.collection(:vcards).save({'_id' => 'full@wonderland.lit', 'card' => VCARD.to_xml})
+      db.collection(:fragments).save({'_id' => "full@wonderland.lit:#{FRAGMENT_ID}", 'xml' => FRAGMENT.to_xml})
+    end
+  end
+
+  def teardown
+    MOCK_MONGO.clear
+  end
+
+  def storage
+    storage = Vines::Storage::MongoDB.new do
+      host 'localhost', 27017
+      database 'xmpp_testcase'
+    end
+    def storage.db
+      MongoDBTest::MOCK_MONGO
+    end
+    storage
+  end
+
+  def test_init
+    EMLoop.new do
+      # missing database
+      assert_raises(RuntimeError) { Vines::Storage::MongoDB.new {} }
+      # missing a host
+      assert_raises(RuntimeError) { Vines::Storage::MongoDB.new { database 'test' } }
+      # duplicate hosts
+      assert_raises(RuntimeError) do
+        Vines::Storage::MongoDB.new do
+          host 'localhost', 27017
+          host 'localhost', 27017
+          database 'test'
+        end
+      end
+      # shouldn't raise an error
+      Vines::Storage::MongoDB.new do
+        host 'localhost', 27017
+        database 'test'
+      end
+      Vines::Storage::MongoDB.new do
+        host 'localhost', 27017
+        host 'localhost', 27018
+        database 'test'
+      end
+    end
+  end
+end