mongo 1.2.rc2 → 1.2.0

data/Rakefile

@@ -26,7 +26,7 @@ namespace :build do
     jar_dir   = File.join(java_dir, 'jar')
 
     jruby_jar = File.join(jar_dir, 'jruby.jar')
-    mongo_jar = File.join(jar_dir, 'mongo-2.2.jar')
+    mongo_jar = File.join(jar_dir, 'mongo-2.4.jar')
     bson_jar = File.join(jar_dir, 'bson-2.2.jar')
 
     src_base   = File.join(java_dir, 'src')

data/docs/HISTORY.md

@@ -1,5 +1,10 @@
 # MongoDB Ruby Driver History
 
+1.2.0
+2011-1-18
+* Some minor improvements. See commit history.
+* Since nothing major was reported for the RC, we're releasing.
+
 1.2.rc0
 2011-1-5
 

data/lib/mongo.rb

@@ -1,9 +1,25 @@
 # encoding: UTF-8
+#
+# --
+# Copyright (C) 2008-2011 10gen Inc.
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+# ++
 
 $:.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
 
 module Mongo
-  VERSION = "1.2.rc2"
+  VERSION = "1.2.0"
 end
 
 module Mongo

data/lib/mongo/collection.rb

@@ -1,7 +1,7 @@
 # encoding: UTF-8
 
 # --
-# Copyright (C) 2008-2010 10gen Inc.
+# Copyright (C) 2008-2011 10gen Inc.
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.

data/lib/mongo/connection.rb

@@ -1,7 +1,7 @@
 # encoding: UTF-8
 
 # --
-# Copyright (C) 2008-2010 10gen Inc.
+# Copyright (C) 2008-2011 10gen Inc.
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
@@ -470,6 +470,7 @@ module Mongo
     def read_primary?
       @read_primary
     end
+    alias :primary? :read_primary?
 
     # Close the connection to the database.
     def close
@@ -750,7 +751,8 @@ module Mongo
     # Requires length and an available socket.
     def receive_message_on_socket(length, socket)
       begin
-        message = socket.read(length)
+        message = new_binary_string
+        socket.read(length, message)
         raise ConnectionFailure, "connection closed" unless message.length > 0
         if message.length < length
           chunk = new_binary_string

data/lib/mongo/cursor.rb

@@ -1,6 +1,6 @@
 # encoding: UTF-8
 
-# Copyright (C) 2008-2010 10gen Inc.
+# Copyright (C) 2008-2011 10gen Inc.
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.

data/lib/mongo/db.rb

@@ -1,7 +1,7 @@
 # encoding: UTF-8
 
 # --
-# Copyright (C) 2008-2010 10gen Inc.
+# Copyright (C) 2008-2011 10gen Inc.
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.

data/lib/mongo/exceptions.rb

@@ -2,7 +2,7 @@
 
 #
 # --
-# Copyright (C) 2008-2010 10gen Inc.
+# Copyright (C) 2008-2011 10gen Inc.
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.

data/lib/mongo/gridfs/grid.rb

@@ -1,7 +1,7 @@
 # encoding: UTF-8
 
 # --
-# Copyright (C) 2008-2010 10gen Inc.
+# Copyright (C) 2008-2011 10gen Inc.
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.

data/lib/mongo/gridfs/grid_ext.rb

@@ -1,7 +1,7 @@
 # encoding: UTF-8
 
 # --
-# Copyright (C) 2008-2010 10gen Inc.
+# Copyright (C) 2008-2011 10gen Inc.
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.

data/lib/mongo/gridfs/grid_file_system.rb

@@ -1,7 +1,7 @@
 # encoding: UTF-8
 
 # --
-# Copyright (C) 2008-2010 10gen Inc.
+# Copyright (C) 2008-2011 10gen Inc.
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.

data/lib/mongo/gridfs/grid_io.rb

@@ -1,7 +1,7 @@
 # encoding: UTF-8
 
 # --
-# Copyright (C) 2008-2010 10gen Inc.
+# Copyright (C) 2008-2011 10gen Inc.
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.

data/lib/mongo/gridfs/grid_io_fix.rb

@@ -1,7 +1,7 @@
 # encoding: UTF-8
 
 # --
-# Copyright (C) 2008-2010 10gen Inc.
+# Copyright (C) 2008-2011 10gen Inc.
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.

data/lib/mongo/repl_set_connection.rb

@@ -1,7 +1,7 @@
 # encoding: UTF-8
 
 # --
-# Copyright (C) 2008-2010 10gen Inc.
+# Copyright (C) 2008-2011 10gen Inc.
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
@@ -137,6 +137,7 @@ module Mongo
     def read_primary?
       !@read_pool || @read_pool.length.zero?
     end
+    alias :primary? :read_primary?
 
     # Close the connection to the database.
     def close

data/lib/mongo/test.rb

@@ -0,0 +1,20 @@
+
+class Foo
+
+  def zed
+    puts "Foo"
+  end
+
+end
+
+class Bar < Foo
+
+  def zed(n=nil)
+    if n.nil?
+      puts "Bar"
+    else
+      super()
+    end
+  end
+
+end

data/lib/mongo/util/conversions.rb

@@ -1,7 +1,7 @@
 # encoding: UTF-8
 
 # --
-# Copyright (C) 2008-2010 10gen Inc.
+# Copyright (C) 2008-2011 10gen Inc.
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.

data/lib/mongo/util/core_ext.rb

@@ -1,7 +1,7 @@
 # encoding: UTF-8
 
 # --
-# Copyright (C) 2008-2010 10gen Inc.
+# Copyright (C) 2008-2011 10gen Inc.
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.

data/lib/mongo/util/pool.rb

@@ -1,7 +1,7 @@
 # encoding: UTF-8
 
 # --
-# Copyright (C) 2008-2010 10gen Inc.
+# Copyright (C) 2008-2011 10gen Inc.
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.

data/lib/mongo/util/server_version.rb

@@ -1,7 +1,7 @@
 # encoding: UTF-8
 
 # --
-# Copyright (C) 2008-2010 10gen Inc.
+# Copyright (C) 2008-2011 10gen Inc.
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.

data/lib/mongo/util/support.rb

@@ -1,7 +1,7 @@
 # encoding: UTF-8
 
 # --
-# Copyright (C) 2008-2010 10gen Inc.
+# Copyright (C) 2008-2011 10gen Inc.
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.

data/lib/mongo/util/uri_parser.rb

@@ -1,7 +1,7 @@
 # encoding: UTF-8
 
 # --
-# Copyright (C) 2008-2010 10gen Inc.
+# Copyright (C) 2008-2011 10gen Inc.
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
@@ -20,7 +20,7 @@ module Mongo
   class URIParser
 
     DEFAULT_PORT = 27017
-    MONGODB_URI_MATCHER = /(([-_.\w\d]+):([-_\w\d]+)@)?([-.\w\d]+)(:([\w\d]+))?(\/([-\d\w]+))?/
+    MONGODB_URI_MATCHER = /(([-_.\w\d]+):([^@]+)@)?([-.\w\d]+)(:([\w\d]+))?(\/([-\d\w]+))?/
     MONGODB_URI_SPEC = "mongodb://[username:password@]host1[:port1][,host2[:port2],...[,hostN[:portN]]][/database]"
     SPEC_ATTRS = [:nodes, :auths]
     OPT_ATTRS  = [:connect, :replicaset, :slaveok, :safe, :w, :wtimeout, :fsync]
@@ -57,6 +57,8 @@ module Mongo
     # Parse a MongoDB URI. This method is used by Connection.from_uri.
     # Returns an array of nodes and an array of db authorizations, if applicable.
     #
+    # Note: passwords can contain any character except for a ','.
+    #
     # @core connections
     def initialize(string)
       if string =~ /^mongodb:\/\//

data/test/async/collection_test.rb

@@ -0,0 +1,224 @@
+require 'test/test_helper'
+
+class TestCollection < Test::Unit::TestCase
+  @@connection ||= Connection.new(ENV['MONGO_RUBY_DRIVER_HOST'] || 'localhost', ENV['MONGO_RUBY_DRIVER_PORT'] || Connection::DEFAULT_PORT)
+  @@db   = @@connection.db(MONGO_TEST_DB)
+  @@test = @@db.collection("test")
+  @@version = @@connection.server_version
+
+  def setup
+    @@test.remove
+  end
+
+  def wait_for_async
+    sleep 0.2
+  end
+
+  def test_async_update
+    id1 = @@test.save("x" => 5)
+    failsafe = mock('failsafe will get called in block', :call => true)
+
+    @@test.update({}, {"$inc" => {"x" => 1}}, :async => true) do |error, result|
+      assert_nil error
+      assert result
+      failsafe.call
+    end
+    wait_for_async
+
+    assert_equal 1, @@test.count()
+    assert_equal 6, @@test.find_one(:_id => id1)["x"]
+  end
+
+  if @@version >= "1.1.3"
+    def test_async_multi_update
+      failsafe = mock('failsafe will get called in block', :call => true)
+
+      @@test.save("num" => 10)
+      @@test.save("num" => 10)
+      @@test.save("num" => 10)
+      assert_equal 3, @@test.count
+
+      @@test.update({"num" => 10}, {"$set" => {"num" => 100}}, :multi => true, :async => true) do |error, result|
+        assert_nil error
+        assert result
+        failsafe.call
+      end
+      wait_for_async
+    end
+  end
+
+  def test_async_upsert
+    failsafe = mock('failsafe will get called in block')
+    failsafe.expects(:call).times(2)
+
+    @@test.update({"page" => "/"}, {"$inc" => {"count" => 1}}, :upsert => true, :async => true) do |error, result|
+      assert_nil error
+      assert result
+      failsafe.call
+    end
+
+    @@test.update({"page" => "/"}, {"$inc" => {"count" => 1}}, :upsert => true, :async => true) do |error, result|
+      assert_nil error
+      assert result
+      failsafe.call
+    end
+    wait_for_async
+
+    assert_equal 1, @@test.count()
+    assert_equal 2, @@test.find_one()["count"]
+  end
+
+  def test_async_save
+    failsafe = mock('failsafe will get called in block', :call => true)
+
+    # note that the first parameter has explicit curly brackets around
+    # the hash; without those brackets as a delimiter, the :async key is
+    # viewed as part of the required +document+ parameter
+    @@test.save({"hello" => "world"}, :async => true) do |error, result|
+      assert_nil error
+      assert result
+      failsafe.call
+    end
+    wait_for_async
+    
+    assert_equal "world", @@test.find_one()["hello"]
+  end
+
+  def test_async_save_with_exception
+    failsafe = mock('failsafe will get called in block', :call => true)
+
+    @@test.create_index("hello", :unique => true)
+    @@test.save("hello" => "world")
+    
+    # all async calls on collections occur in :safe mode
+    @@test.save({"hello" => "world"}, :async => true) do |error, result|
+      assert error
+      assert error.instance_of?(OperationFailure)
+      assert_nil result
+      failsafe.call
+    end
+    wait_for_async
+    
+    assert_equal 1, @@test.count()
+    @@test.drop
+  end
+
+  def test_async_remove
+    failsafe = mock('failsafe will get called in block', :call => true)
+
+    @conn = Connection.new
+    @db   = @conn[MONGO_TEST_DB]
+    @test = @db['test-async-remove']
+    @test.save({:a => 50})
+    @test.remove({}, :async => true) do |error, result|
+      assert_nil error
+      assert result
+      failsafe.call
+    end
+    wait_for_async
+    
+    @test.drop
+  end
+
+  def test_async_count
+    failsafe = mock('failsafe will get called in block', :call => true)
+
+    @@test.drop
+
+    @@test.save("x" => 1)
+    @@test.save("x" => 2)
+
+    @@test.count(:async => true) do |error, result|
+      assert_nil error
+      assert_equal 2, result
+      failsafe.call
+    end
+    wait_for_async
+  end
+
+  # Note: #size is just an alias for #count.
+  def test_async_size
+    failsafe = mock('failsafe will get called in block', :call => true)
+    
+    @@test.drop
+
+    @@test.save("x" => 1)
+    @@test.save("x" => 2)
+
+    @@test.size(:async => true) do |error, result|
+      assert_nil error
+      assert_equal 2, result
+      failsafe.call
+    end
+    wait_for_async
+  end
+
+  def test_async_find_one
+    failsafe = mock('failsafe will get called in block', :call => true)
+
+    id = @@test.save("hello" => "world", "foo" => "bar")
+
+    @@test.find_one({}, :async => true) do |error, result|
+      assert_nil error
+      assert_equal @@test.find_one(id), result
+      failsafe.call
+    end
+    wait_for_async
+  end
+
+  def test_async_insert
+    failsafe = mock('failsafe will get called in block', :call => true)
+
+    doc = {"hello" => "world"}
+    @@test.insert(doc, :async => true) do |error, result|
+      assert_nil error
+      assert result
+      failsafe.call
+    end
+    wait_for_async
+    
+    assert_equal 1, @@test.count
+  end
+
+  def test_async_find
+    assert_raise RuntimeError do
+      @@test.find({}, :async => true)
+    end
+  end
+
+  if @@version > "1.3.0"
+    def test_async_find_and_modify
+      failsafe = mock('failsafe will get called in block', :call => true)
+
+      @@test << { :a => 1, :processed => false }
+      @@test << { :a => 2, :processed => false }
+      @@test << { :a => 3, :processed => false }
+
+      @@test.find_and_modify(:query => {}, :sort => [['a', -1]], :update => {"$set" => {:processed => true}}, :async => true) do |error, result|
+        assert_nil error
+        assert result
+        failsafe.call
+      end
+      wait_for_async
+
+      assert @@test.find_one({:a => 3})['processed']
+    end
+
+    def test_async_find_and_modify_with_invalid_options
+      failsafe = mock('failsafe will get called in block', :call => true)
+
+      @@test << { :a => 1, :processed => false }
+      @@test << { :a => 2, :processed => false }
+      @@test << { :a => 3, :processed => false }
+
+      @@test.find_and_modify(:blimey => {}, :async => true) do |error, result|
+        assert error
+        assert error.instance_of?(OperationFailure)
+        assert_nil result
+        failsafe.call
+      end
+      wait_for_async
+    end
+  end
+
+end

data/test/async/connection_test.rb

@@ -0,0 +1,24 @@
+require 'test/test_helper'
+include Mongo
+
+class ConnectionTest < Test::Unit::TestCase
+  context "Initialization: " do
+
+    context "given async connection options" do
+
+      should "default the workers pool to 1" do
+        Async::WorkerPool.expects(:new).with(1)
+
+        Connection.new('localhost', 27017)
+      end
+
+      should "override the workers pool size with the :worker_pool_size key" do
+        size = 6
+        Async::WorkerPool.expects(:new).with(size)
+
+        Connection.new('localhost', 27017, :worker_pool_size => size)
+      end
+    end # context 'given async connection options'
+
+  end # context 'Initialization'
+end

data/test/async/cursor_test.rb

@@ -0,0 +1,162 @@
+require 'test/test_helper'
+require 'logger'
+
+class CursorTest < Test::Unit::TestCase
+
+  include Mongo
+
+  @@connection = Connection.new(ENV['MONGO_RUBY_DRIVER_HOST'] || 'localhost',
+                        ENV['MONGO_RUBY_DRIVER_PORT'] || Connection::DEFAULT_PORT)
+  @@db   = @@connection.db(MONGO_TEST_DB)
+  @@coll = @@db.collection('test')
+  @@version = @@connection.server_version
+
+  def wait_for_async
+    sleep 0.2
+  end
+
+  def setup
+    @@coll.remove
+    @@coll.insert('a' => 1)     # collection not created until it's used
+    @@coll_full_name = "#{MONGO_TEST_DB}.test"
+  end
+
+  def test_async_explain
+    failsafe = mock('failsafe will get called in block', :call => true)
+
+    cursor = @@coll.find('a' => 1)
+
+    cursor.explain(:async => true) do |error, result|
+      assert_not_nil result['cursor']
+      assert_kind_of Numeric, result['n']
+      assert_kind_of Numeric, result['millis']
+      assert_kind_of Numeric, result['nscanned']
+      failsafe.call
+    end
+    wait_for_async
+  end
+
+  def test_async_count
+    failsafe = mock('failsafe will get called in block')
+    failsafe.expects(:call).times(3)
+
+    @@coll.remove
+
+    @@coll.find.count(:async => true) do |error, count|
+      assert_equal 0, count
+      failsafe.call
+    end
+    wait_for_async
+
+    10.times do |i|
+      @@coll.save("x" => i)
+    end
+
+    @@coll.find.count(:async => true) do |error, count|
+      assert_equal 10, count
+      failsafe.call
+    end
+    wait_for_async
+
+    @@coll.find({"x" => 1}).count(:async => true) do |error, count|
+      assert_equal 1, count
+      failsafe.call
+    end
+    wait_for_async
+  end
+
+  def test_async_close
+    failsafe = mock('failsafe will get called in block', :call => true)
+
+    @@coll.remove
+    cursor = @@coll.find
+    
+    cursor.close(:async => true) do |error, result|
+      assert_nil error
+      assert result
+      assert cursor.closed?
+      failsafe.call
+    end
+    wait_for_async
+  end
+
+  def test_async_has_next
+    failsafe = mock('failsafe will get called in block', :call => true)
+
+    @@coll.remove
+    200.times do |n|
+      @@coll.save("x" => n)
+    end
+
+    cursor = @@coll.find
+    cursor.has_next?(:async => true) do |error, result|
+      assert_nil error
+      assert result
+      failsafe.call
+    end
+    wait_for_async
+  end
+
+  def test_async_next_document
+    failsafe = mock('failsafe will get called in block')
+    failsafe.expects(:call).times(2)
+
+    @@coll.remove
+    200.times do |n|
+      @@coll.save("x" => n)
+    end
+
+    cursor = @@coll.find
+    cursor.next_document(:async => true) do |error, result|
+      assert_nil error
+      assert result
+      failsafe.call
+    end
+    wait_for_async
+
+    callback = Proc.new do |error, result| 
+      assert_nil error
+      assert result
+      failsafe.call
+    end
+
+    cursor.next_document(:async => true, :callback => callback)
+    wait_for_async
+  end
+
+  def test_async_to_a
+    failsafe = mock('failsafe will get called in block')
+    failsafe.expects(:call)
+
+    @@coll.remove
+    total = 200
+    total.times do |n|
+      @@coll.save("x" => n)
+    end
+
+    cursor = @@coll.find
+    cursor.to_a(:async => true) do |error, result|
+      assert_nil error
+      assert_equal total, result.size
+      failsafe.call
+    end
+    wait_for_async
+  end
+
+  def test_async_each
+    @@coll.remove
+    total = 200
+    total.times do |n|
+      @@coll.save("x" => n)
+    end
+
+    cursor = @@coll.find
+    count = 0
+    cursor.each(:async => true) do |error, result|
+      count += 1
+    end
+    wait_for_async
+    
+    assert_equal total, count
+  end
+end

data/test/async/worker_pool_test.rb

@@ -0,0 +1,99 @@
+require 'test/test_helper'
+include Mongo
+
+class WorkerPoolTest < Test::Unit::TestCase
+  context "Initialization: " do
+
+    def wait_for_async
+      sleep 0.2
+    end
+
+    setup do
+      def new_mock_queue
+        stub_everything('queue')
+      end
+
+      def new_mock_thread
+        stub_everything('thread')
+      end
+    end
+
+    context "given a size" do
+      setup do
+        @size = 5
+      end
+
+      should "allocate a Thread 'size' times" do
+        Queue.stubs(:new).returns(new_mock_queue)
+        Thread.expects(:new).times(@size).returns(new_mock_thread)
+        Async::WorkerPool.new @size
+      end
+
+      should "set 'abort_on_exception' for each current thread" do
+        Queue.stubs(:new).returns(new_mock_queue)
+        thread = new_mock_thread
+        Thread.stubs(:new).returns(thread)
+
+        thread.expects(:abort_on_exception=).with(true).times(@size)
+
+        Async::WorkerPool.new @size
+      end
+
+      should "save each thread into the workers queue" do
+        assert_equal @size, Async::WorkerPool.new(@size).workers.size
+      end
+
+    end # context 'given a size'
+
+
+    context "given a job" do
+      setup do
+        @pool = Async::WorkerPool.new 1
+        @command = stub_everything('command')
+        @cmd_args = stub_everything('command args')
+        @callback = stub_everything('callback')
+      end
+
+      should "remove nils from the command args array and pass the results to the callback" do
+        args = [nil, @cmd_args]
+        @command.expects(:call).with(@cmd_args).returns(2)
+        @callback.expects(:call).with(nil, 2)
+
+        @pool.enqueue @command, args, @callback
+        wait_for_async
+      end
+
+      should "execute the original command with args and pass the results to the callback" do
+        @cmd_args.expects(:compact).returns(@cmd_args)
+        @command.expects(:call).with(@cmd_args).returns(2)
+        @callback.expects(:call).with(nil, 2)
+
+        @pool.enqueue @command, @cmd_args, @callback
+        wait_for_async
+      end
+
+      should "capture any exceptions and pass them to the callback" do
+        args = [@cmd_args]
+        error = StandardError.new
+        @command.expects(:call).with(@cmd_args).raises(error)
+        @callback.expects(:call).with(error, nil)
+
+        @pool.enqueue @command, args, @callback
+        wait_for_async
+      end
+
+      should "abort the thread when the callback raises an exception" do
+        args = [@cmd_args]
+        error = StandardError.new
+        @callback.expects(:call).raises(error)
+
+        assert_raises(StandardError) do
+          @pool.enqueue @command, args, @callback
+          wait_for_async
+        end
+      end
+    end # context 'given a job'
+
+
+  end
+end

data/test/replica_sets/connect_test.rb

@@ -32,6 +32,7 @@ class ConnectTest < Test::Unit::TestCase
       [RS.host, RS.ports[2]], :name => RS.name)
     assert @conn.connected?
     assert @conn.read_primary?
+    assert @conn.primary?
 
     assert_equal RS.primary, @conn.primary
     assert_equal RS.secondaries.sort, @conn.secondaries.sort

data/test/replica_sets/query_secondaries.rb

@@ -18,6 +18,7 @@ class ReplicaSetQuerySecondariesTest < Test::Unit::TestCase
 
   def test_read_primary
     assert !@conn.read_primary?
+    assert !@conn.primary?
   end
 
   def test_con

data/test/uri_test.rb

@@ -26,6 +26,22 @@ class TestThreading < Test::Unit::TestCase
     assert_equal 27017, parser.nodes[1][1]
   end
 
+  def test_complex_passwords
+    parser = Mongo::URIParser.new('mongodb://bob:secret.word@a.example.com:27018/test')
+    assert_equal "bob", parser.auths[0]["username"]
+    assert_equal "secret.word", parser.auths[0]["password"]
+
+    parser = Mongo::URIParser.new('mongodb://bob:s-_3#%R.t@a.example.com:27018/test')
+    assert_equal "bob", parser.auths[0]["username"]
+    assert_equal "s-_3#%R.t", parser.auths[0]["password"]
+  end
+
+  def test_passwords_contain_no_commas
+    assert_raise MongoArgumentError do
+      Mongo::URIParser.new('mongodb://bob:a,b@a.example.com:27018/test')
+    end
+  end
+
   def test_multiple_uris_with_auths
     parser = Mongo::URIParser.new('mongodb://bob:secret@a.example.com:27018/test,joe:secret2@b.example.com/test2')
     assert_equal 2, parser.nodes.length

metadata

@@ -1,14 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: mongo
 version: !ruby/object:Gem::Version 
-  hash: 15424225
-  prerelease: 4
+  hash: 31
+  prerelease: 
   segments: 
   - 1
   - 2
-  - rc
-  - 2
-  version: 1.2.rc2
+  - 0
+  version: 1.2.0
 platform: ruby
 authors: 
 - Jim Menard
@@ -18,7 +17,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-01-06 00:00:00 -05:00
+date: 2011-01-18 00:00:00 -05:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -29,13 +28,12 @@ dependencies:
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
-        hash: 15424225
+        hash: 31
         segments: 
         - 1
         - 2
-        - rc
-        - 2
-        version: 1.2.rc2
+        - 0
+        version: 1.2.0
   type: :runtime
   version_requirements: *id001
 description: A Ruby driver for MongoDB. For more information about Mongo, see http://www.mongodb.org.
@@ -57,6 +55,7 @@ files:
 - lib/mongo/exceptions.rb
 - lib/mongo/connection.rb
 - lib/mongo/repl_set_connection.rb
+- lib/mongo/test.rb
 - lib/mongo/gridfs/grid_io.rb
 - lib/mongo/gridfs/grid_file_system.rb
 - lib/mongo/gridfs/grid.rb
@@ -89,6 +88,10 @@ files:
 - test/unit/safe_test.rb
 - test/db_test.rb
 - test/collection_test.rb
+- test/async/collection_test.rb
+- test/async/cursor_test.rb
+- test/async/connection_test.rb
+- test/async/worker_pool_test.rb
 - test/cursor_test.rb
 - test/grid_test.rb
 - test/db_api_test.rb
@@ -151,14 +154,12 @@ required_ruby_version: !ruby/object:Gem::Requirement
 required_rubygems_version: !ruby/object:Gem::Requirement 
   none: false
   requirements: 
-  - - ">"
+  - - ">="
     - !ruby/object:Gem::Version 
-      hash: 25
+      hash: 3
       segments: 
-      - 1
-      - 3
-      - 1
-      version: 1.3.1
+      - 0
+      version: "0"
 requirements: []
 
 rubyforge_project: 
@@ -178,6 +179,10 @@ test_files:
 - test/unit/safe_test.rb
 - test/db_test.rb
 - test/collection_test.rb
+- test/async/collection_test.rb
+- test/async/cursor_test.rb
+- test/async/connection_test.rb
+- test/async/worker_pool_test.rb
 - test/cursor_test.rb
 - test/grid_test.rb
 - test/db_api_test.rb