oboe 2.6.6.1 → 2.6.7.1

data/test/instrumentation/http_test.rb

@@ -3,7 +3,7 @@ require 'net/http'
 
 describe Oboe::Inst do
   before do
-    clear_all_traces 
+    clear_all_traces
     @collect_backtraces = Oboe::Config[:nethttp][:collect_backtraces]
   end
 
@@ -12,7 +12,7 @@ describe Oboe::Inst do
   end
 
   it 'Net::HTTP should be defined and ready' do
-    defined?(::Net::HTTP).wont_match nil 
+    defined?(::Net::HTTP).wont_match nil
   end
 
   it 'Net::HTTP should have oboe methods defined' do
@@ -31,7 +31,7 @@ describe Oboe::Inst do
 
     traces = get_all_traces
     traces.count.must_equal 5
-    
+
     validate_outer_layers(traces, 'net-http_test')
 
     traces[1]['Layer'].must_equal 'net-http'
@@ -43,7 +43,7 @@ describe Oboe::Inst do
     traces[2]['HTTPStatus'].must_equal "200"
     traces[2].has_key?('Backtrace').must_equal Oboe::Config[:nethttp][:collect_backtraces]
   end
-  
+
   it "should trace a Net::HTTP request" do
     Oboe::API.start_trace('net-http_test', '', {}) do
       uri = URI('https://www.google.com')
@@ -54,7 +54,7 @@ describe Oboe::Inst do
 
     traces = get_all_traces
     traces.count.must_equal 5
-    
+
     validate_outer_layers(traces, 'net-http_test')
 
     traces[1]['Layer'].must_equal 'net-http'
@@ -66,7 +66,7 @@ describe Oboe::Inst do
     traces[2]['HTTPStatus'].must_equal "200"
     traces[2].has_key?('Backtrace').must_equal Oboe::Config[:nethttp][:collect_backtraces]
   end
-  
+
   it "should obey :collect_backtraces setting when true" do
     Oboe::Config[:nethttp][:collect_backtraces] = true
 

data/test/instrumentation/mongo_test.rb

@@ -1,460 +1,462 @@
 require 'minitest_helper'
 
-describe Oboe::Inst::Mongo do
-  before do
-    clear_all_traces
-    @connection = Mongo::Connection.new("localhost", 27017, :slave_ok => true)
-    @db = @connection.db("test-#{ENV['RACK_ENV']}")
-
-    @collections = @db.collection_names
-    @db.create_collection("testCollection") unless @collections.include? "testCollection"
-
-    # These are standard entry/exit KVs that are passed up with all mongo operations
-    @entry_kvs = {
-      'Layer' => 'mongo',
-      'Label' => 'entry',
-      'Flavor' => 'mongodb',
-      'Database' => 'test-test',
-      'RemoteHost' => 'localhost',
-      'RemotePort' => '27017' }
-
-    @exit_kvs = { 'Layer' => 'mongo', 'Label' => 'exit' }
-    @collect_backtraces = Oboe::Config[:mongo][:collect_backtraces]
-  end
-
-  after do
-    Oboe::Config[:mongo][:collect_backtraces] = @collect_backtraces
-  end
-
-  it 'Stock Mongo should be loaded, defined and ready' do
-    defined?(::Mongo).wont_match nil
-    defined?(::Mongo::DB).wont_match nil
-    defined?(::Mongo::Cursor).wont_match nil
-    defined?(::Mongo::Collection).wont_match nil
-  end
-
-  it 'Mongo should have oboe methods defined' do
-    Oboe::Inst::Mongo::DB_OPS.each do |m|
-      ::Mongo::DB.method_defined?("#{m}_with_oboe").must_equal true
-    end
-    Oboe::Inst::Mongo::CURSOR_OPS.each do |m|
-      ::Mongo::Cursor.method_defined?("#{m}_with_oboe").must_equal true
+if defined?(::BSON::VERSION) and (BSON::VERSION < "2.0")
+  describe Oboe::Inst::Mongo do
+    before do
+      clear_all_traces
+      @connection = Mongo::Connection.new("localhost", 27017, :slave_ok => true)
+      @db = @connection.db("test-#{ENV['RACK_ENV']}")
+
+      @collections = @db.collection_names
+      @db.create_collection("testCollection") unless @collections.include? "testCollection"
+
+      # These are standard entry/exit KVs that are passed up with all mongo operations
+      @entry_kvs = {
+        'Layer' => 'mongo',
+        'Label' => 'entry',
+        'Flavor' => 'mongodb',
+        'Database' => 'test-test',
+        'RemoteHost' => 'localhost',
+        'RemotePort' => '27017' }
+
+      @exit_kvs = { 'Layer' => 'mongo', 'Label' => 'exit' }
+      @collect_backtraces = Oboe::Config[:mongo][:collect_backtraces]
     end
-    Oboe::Inst::Mongo::COLL_WRITE_OPS.each do |m|
-      ::Mongo::Collection.method_defined?("#{m}_with_oboe").must_equal true
-    end
-    Oboe::Inst::Mongo::COLL_QUERY_OPS.each do |m|
-      ::Mongo::Collection.method_defined?("#{m}_with_oboe").must_equal true
-    end
-    Oboe::Inst::Mongo::COLL_INDEX_OPS.each do |m|
-      ::Mongo::Collection.method_defined?("#{m}_with_oboe").must_equal true
+
+    after do
+      Oboe::Config[:mongo][:collect_backtraces] = @collect_backtraces
     end
-    ::Mongo::Collection.method_defined?(:oboe_collect).must_equal true
-  end
 
-  it "should trace create_collection" do
-    Oboe::API.start_trace('mongo_test', '', {}) do
-      @db.create_collection("create_and_drop_collection_test")
+    it 'Stock Mongo should be loaded, defined and ready' do
+      defined?(::Mongo).wont_match nil
+      defined?(::Mongo::DB).wont_match nil
+      defined?(::Mongo::Cursor).wont_match nil
+      defined?(::Mongo::Collection).wont_match nil
     end
 
-    traces = get_all_traces
-    traces.count.must_equal 4
+    it 'Mongo should have oboe methods defined' do
+      Oboe::Inst::Mongo::DB_OPS.each do |m|
+        ::Mongo::DB.method_defined?("#{m}_with_oboe").must_equal true
+      end
+      Oboe::Inst::Mongo::CURSOR_OPS.each do |m|
+        ::Mongo::Cursor.method_defined?("#{m}_with_oboe").must_equal true
+      end
+      Oboe::Inst::Mongo::COLL_WRITE_OPS.each do |m|
+        ::Mongo::Collection.method_defined?("#{m}_with_oboe").must_equal true
+      end
+      Oboe::Inst::Mongo::COLL_QUERY_OPS.each do |m|
+        ::Mongo::Collection.method_defined?("#{m}_with_oboe").must_equal true
+      end
+      Oboe::Inst::Mongo::COLL_INDEX_OPS.each do |m|
+        ::Mongo::Collection.method_defined?("#{m}_with_oboe").must_equal true
+      end
+      ::Mongo::Collection.method_defined?(:oboe_collect).must_equal true
+    end
 
-    validate_outer_layers(traces, 'mongo_test')
-    validate_event_keys(traces[1], @entry_kvs)
-    validate_event_keys(traces[2], @exit_kvs)
+    it "should trace create_collection" do
+      Oboe::API.start_trace('mongo_test', '', {}) do
+        @db.create_collection("create_and_drop_collection_test")
+      end
 
-    traces[1]['QueryOp'].must_equal "create_collection"
-    traces[1]['New_Collection_Name'].must_equal "create_and_drop_collection_test"
-    traces[1].has_key?('Backtrace').must_equal Oboe::Config[:mongo][:collect_backtraces]
-  end
+      traces = get_all_traces
+      traces.count.must_equal 4
 
-  it "should trace drop_collection" do
-    # Create a collection so we have one to drop
-    @db.create_collection("create_and_drop_collection_test")
+      validate_outer_layers(traces, 'mongo_test')
+      validate_event_keys(traces[1], @entry_kvs)
+      validate_event_keys(traces[2], @exit_kvs)
 
-    Oboe::API.start_trace('mongo_test', '', {}) do
-      @db.drop_collection("create_and_drop_collection_test")
+      traces[1]['QueryOp'].must_equal "create_collection"
+      traces[1]['New_Collection_Name'].must_equal "create_and_drop_collection_test"
+      traces[1].has_key?('Backtrace').must_equal Oboe::Config[:mongo][:collect_backtraces]
     end
 
-    traces = get_all_traces
-    traces.count.must_equal 4
+    it "should trace drop_collection" do
+      # Create a collection so we have one to drop
+      @db.create_collection("create_and_drop_collection_test")
 
-    validate_outer_layers(traces, 'mongo_test')
-    validate_event_keys(traces[1], @entry_kvs)
-    validate_event_keys(traces[2], @exit_kvs)
+      Oboe::API.start_trace('mongo_test', '', {}) do
+        @db.drop_collection("create_and_drop_collection_test")
+      end
 
-    traces[1]['QueryOp'].must_equal "drop_collection"
-    traces[1]['Collection'].must_equal "create_and_drop_collection_test"
-    traces[1].has_key?('Backtrace').must_equal Oboe::Config[:mongo][:collect_backtraces]
-  end
+      traces = get_all_traces
+      traces.count.must_equal 4
 
-  it "should trace count" do
-    coll = @db.collection("testCollection")
+      validate_outer_layers(traces, 'mongo_test')
+      validate_event_keys(traces[1], @entry_kvs)
+      validate_event_keys(traces[2], @exit_kvs)
 
-    Oboe::API.start_trace('mongo_test', '', {}) do
-      coll.count(:query => {:name => "MyName"})
+      traces[1]['QueryOp'].must_equal "drop_collection"
+      traces[1]['Collection'].must_equal "create_and_drop_collection_test"
+      traces[1].has_key?('Backtrace').must_equal Oboe::Config[:mongo][:collect_backtraces]
     end
 
-    traces = get_all_traces
-    traces.count.must_equal 6
+    it "should trace count" do
+      coll = @db.collection("testCollection")
 
-    validate_outer_layers(traces, 'mongo_test')
-    validate_event_keys(traces[1], @entry_kvs)
-    validate_event_keys(traces[2], @exit_kvs)
+      Oboe::API.start_trace('mongo_test', '', {}) do
+        coll.count(:query => {:name => "MyName"})
+      end
 
-    traces[3]['QueryOp'].must_equal "count"
-  end
+      traces = get_all_traces
+      traces.count.must_equal 6
 
-  it "should trace find_and_modify" do
-    coll = @db.collection("testCollection")
+      validate_outer_layers(traces, 'mongo_test')
+      validate_event_keys(traces[1], @entry_kvs)
+      validate_event_keys(traces[2], @exit_kvs)
 
-    Oboe::API.start_trace('mongo_test', '', {}) do
-      coll.find_and_modify({ :query => { :name => "MyName" }, :update => { :count => 203 }})
+      traces[3]['QueryOp'].must_equal "count"
     end
 
-    traces = get_all_traces
-    traces.count.must_equal 4
+    it "should trace find_and_modify" do
+      coll = @db.collection("testCollection")
 
-    validate_outer_layers(traces, 'mongo_test')
-    validate_event_keys(traces[1], @entry_kvs)
-    validate_event_keys(traces[2], @exit_kvs)
+      Oboe::API.start_trace('mongo_test', '', {}) do
+        coll.find_and_modify({ :query => { :name => "MyName" }, :update => { :count => 203 }})
+      end
 
-    traces[1]['Collection'].must_equal "testCollection"
-    traces[1].has_key?('Backtrace').must_equal Oboe::Config[:mongo][:collect_backtraces]
-    traces[1]['QueryOp'].must_equal "find_and_modify"
-    traces[1]['Update_Document'].must_equal "{:count=>203}"
-  end
+      traces = get_all_traces
+      traces.count.must_equal 4
 
-  it "should trace insert" do
-    coll = @db.collection("testCollection")
+      validate_outer_layers(traces, 'mongo_test')
+      validate_event_keys(traces[1], @entry_kvs)
+      validate_event_keys(traces[2], @exit_kvs)
 
-    Oboe::API.start_trace('mongo_test', '', {}) do
-      doc = {"name" => "MyName", "type" => "MyType", "count" => 1, "info" => {"x" => 203, "y" => '102'}}
-      id = coll.insert(doc)
+      traces[1]['Collection'].must_equal "testCollection"
+      traces[1].has_key?('Backtrace').must_equal Oboe::Config[:mongo][:collect_backtraces]
+      traces[1]['QueryOp'].must_equal "find_and_modify"
+      traces[1]['Update_Document'].must_equal "{:count=>203}"
     end
 
-    traces = get_all_traces
-    traces.count.must_equal 4
+    it "should trace insert" do
+      coll = @db.collection("testCollection")
 
-    validate_outer_layers(traces, 'mongo_test')
-    validate_event_keys(traces[1], @entry_kvs)
-    validate_event_keys(traces[2], @exit_kvs)
+      Oboe::API.start_trace('mongo_test', '', {}) do
+        doc = {"name" => "MyName", "type" => "MyType", "count" => 1, "info" => {"x" => 203, "y" => '102'}}
+        id = coll.insert(doc)
+      end
 
-    traces[1]['Collection'].must_equal "testCollection"
-    traces[1].has_key?('Backtrace').must_equal Oboe::Config[:mongo][:collect_backtraces]
-    traces[1]['QueryOp'].must_equal "insert"
-    # Don't test exact hash value since to_json hash ordering varies between 1.8.7 and 1.9+
-    traces[1].has_key?('Query').must_equal true
-  end
+      traces = get_all_traces
+      traces.count.must_equal 4
 
-  it "should trace map_reduce" do
-    coll = @db.collection("testCollection")
+      validate_outer_layers(traces, 'mongo_test')
+      validate_event_keys(traces[1], @entry_kvs)
+      validate_event_keys(traces[2], @exit_kvs)
 
-    Oboe::API.start_trace('mongo_test', '', {}) do
-      map    = "function() { emit(this.name, 1); }"
-      reduce = "function(k, vals) { var sum = 0; for(var i in vals) sum += vals[i]; return sum; }"
-      coll.map_reduce(map, reduce, { :out => "mr_results", :limit => 100, :read => :primary })
+      traces[1]['Collection'].must_equal "testCollection"
+      traces[1].has_key?('Backtrace').must_equal Oboe::Config[:mongo][:collect_backtraces]
+      traces[1]['QueryOp'].must_equal "insert"
+      # Don't test exact hash value since to_json hash ordering varies between 1.8.7 and 1.9+
+      traces[1].has_key?('Query').must_equal true
     end
 
-    traces = get_all_traces
-    traces.count.must_equal 4
+    it "should trace map_reduce" do
+      coll = @db.collection("testCollection")
 
-    validate_outer_layers(traces, 'mongo_test')
-    validate_event_keys(traces[1], @entry_kvs)
-    validate_event_keys(traces[2], @exit_kvs)
+      Oboe::API.start_trace('mongo_test', '', {}) do
+        map    = "function() { emit(this.name, 1); }"
+        reduce = "function(k, vals) { var sum = 0; for(var i in vals) sum += vals[i]; return sum; }"
+        coll.map_reduce(map, reduce, { :out => "mr_results", :limit => 100, :read => :primary })
+      end
 
-    traces[1]['Collection'].must_equal "testCollection"
-    traces[1].has_key?('Backtrace').must_equal Oboe::Config[:mongo][:collect_backtraces]
-    traces[1]['QueryOp'].must_equal "map_reduce"
-    traces[1]['Map_Function'].must_equal "function() { emit(this.name, 1); }"
-    traces[1]['Reduce_Function'].must_equal "function(k, vals) { var sum = 0; for(var i in vals) sum += vals[i]; return sum; }"
-    traces[1]['Limit'].must_equal "100"
-  end
+      traces = get_all_traces
+      traces.count.must_equal 4
 
-  it "should trace remove" do
-    coll = @db.collection("testCollection")
+      validate_outer_layers(traces, 'mongo_test')
+      validate_event_keys(traces[1], @entry_kvs)
+      validate_event_keys(traces[2], @exit_kvs)
 
-    Oboe::API.start_trace('mongo_test', '', {}) do
-      coll.remove(:name => "SaveOp")
+      traces[1]['Collection'].must_equal "testCollection"
+      traces[1].has_key?('Backtrace').must_equal Oboe::Config[:mongo][:collect_backtraces]
+      traces[1]['QueryOp'].must_equal "map_reduce"
+      traces[1]['Map_Function'].must_equal "function() { emit(this.name, 1); }"
+      traces[1]['Reduce_Function'].must_equal "function(k, vals) { var sum = 0; for(var i in vals) sum += vals[i]; return sum; }"
+      traces[1]['Limit'].must_equal "100"
     end
 
-    traces = get_all_traces
-    traces.count.must_equal 4
+    it "should trace remove" do
+      coll = @db.collection("testCollection")
 
-    validate_outer_layers(traces, 'mongo_test')
-    validate_event_keys(traces[1], @entry_kvs)
-    validate_event_keys(traces[2], @exit_kvs)
+      Oboe::API.start_trace('mongo_test', '', {}) do
+        coll.remove(:name => "SaveOp")
+      end
 
-    traces[1]['Collection'].must_equal "testCollection"
-    traces[1].has_key?('Backtrace').must_equal Oboe::Config[:mongo][:collect_backtraces]
-    traces[1]['QueryOp'].must_equal "remove"
-    traces[1]['Query'].must_equal "{\"name\":\"SaveOp\"}"
-  end
+      traces = get_all_traces
+      traces.count.must_equal 4
 
-  it "should trace rename" do
-    coll = @db.collection("testCollection")
-    new_name = (0...10).map{ ('a'..'z').to_a[rand(26)] }.join
+      validate_outer_layers(traces, 'mongo_test')
+      validate_event_keys(traces[1], @entry_kvs)
+      validate_event_keys(traces[2], @exit_kvs)
 
-    Oboe::API.start_trace('mongo_test', '', {}) do
-      coll.rename(new_name)
+      traces[1]['Collection'].must_equal "testCollection"
+      traces[1].has_key?('Backtrace').must_equal Oboe::Config[:mongo][:collect_backtraces]
+      traces[1]['QueryOp'].must_equal "remove"
+      traces[1]['Query'].must_equal "{\"name\":\"SaveOp\"}"
     end
 
-    traces = get_all_traces
-    traces.count.must_equal 4
+    it "should trace rename" do
+      coll = @db.collection("testCollection")
+      new_name = (0...10).map{ ('a'..'z').to_a[rand(26)] }.join
 
-    validate_outer_layers(traces, 'mongo_test')
-    validate_event_keys(traces[1], @entry_kvs)
-    validate_event_keys(traces[2], @exit_kvs)
+      Oboe::API.start_trace('mongo_test', '', {}) do
+        coll.rename(new_name)
+      end
 
-    traces[1]['Collection'].must_equal "testCollection"
-    traces[1].has_key?('Backtrace').must_equal Oboe::Config[:mongo][:collect_backtraces]
-    traces[1]['QueryOp'].must_equal "rename"
-    traces[1]['New_Collection_Name'].must_equal new_name
+      traces = get_all_traces
+      traces.count.must_equal 4
 
-    # Clean up after test and set collection name back to original
-    coll.rename("testCollection")
-  end
-
-  it "should trace update" do
-    coll = @db.collection("testCollection")
+      validate_outer_layers(traces, 'mongo_test')
+      validate_event_keys(traces[1], @entry_kvs)
+      validate_event_keys(traces[2], @exit_kvs)
 
-    Oboe::API.start_trace('mongo_test', '', {}) do
-      # Two types of update calls
-      coll.update({"_id" => 1}, { "$set" => {"name" => "MongoDB Ruby"}}, :multi => true)
+      traces[1]['Collection'].must_equal "testCollection"
+      traces[1].has_key?('Backtrace').must_equal Oboe::Config[:mongo][:collect_backtraces]
+      traces[1]['QueryOp'].must_equal "rename"
+      traces[1]['New_Collection_Name'].must_equal new_name
 
-      doc = {"name" => "MyOtherName", "type" => "MyType", "count" => 1, "info" => {"x" => 203, "y" => '102'}}
-      coll.update({"_id" => 1}, doc)
+      # Clean up after test and set collection name back to original
+      coll.rename("testCollection")
     end
 
-    traces = get_all_traces
-    traces.count.must_equal 6
-
-    validate_outer_layers(traces, 'mongo_test')
-    validate_event_keys(traces[1], @entry_kvs)
-    validate_event_keys(traces[2], @exit_kvs)
+    it "should trace update" do
+      coll = @db.collection("testCollection")
 
-    traces[1]['Collection'].must_equal "testCollection"
-    traces[1].has_key?('Backtrace').must_equal Oboe::Config[:mongo][:collect_backtraces]
-    traces[1]['QueryOp'].must_equal "update"
-    traces[1]['Query'].must_equal "{\"_id\":1}"
+      Oboe::API.start_trace('mongo_test', '', {}) do
+        # Two types of update calls
+        coll.update({"_id" => 1}, { "$set" => {"name" => "MongoDB Ruby"}}, :multi => true)
 
-    validate_event_keys(traces[3], @entry_kvs)
-    validate_event_keys(traces[4], @exit_kvs)
+        doc = {"name" => "MyOtherName", "type" => "MyType", "count" => 1, "info" => {"x" => 203, "y" => '102'}}
+        coll.update({"_id" => 1}, doc)
+      end
 
-    traces[3].has_key?('Backtrace').must_equal Oboe::Config[:mongo][:collect_backtraces]
-    traces[3]['QueryOp'].must_equal "update"
-    traces[3]['Query'].must_equal "{\"_id\":1}"
-  end
+      traces = get_all_traces
+      traces.count.must_equal 6
 
-  it "should trace distinct" do
-    coll = @db.collection("testCollection")
+      validate_outer_layers(traces, 'mongo_test')
+      validate_event_keys(traces[1], @entry_kvs)
+      validate_event_keys(traces[2], @exit_kvs)
 
-    Oboe::API.start_trace('mongo_test', '', {}) do
-      coll.distinct("count")
-    end
+      traces[1]['Collection'].must_equal "testCollection"
+      traces[1].has_key?('Backtrace').must_equal Oboe::Config[:mongo][:collect_backtraces]
+      traces[1]['QueryOp'].must_equal "update"
+      traces[1]['Query'].must_equal "{\"_id\":1}"
 
-    traces = get_all_traces
-    traces.count.must_equal 4
+      validate_event_keys(traces[3], @entry_kvs)
+      validate_event_keys(traces[4], @exit_kvs)
 
-    validate_outer_layers(traces, 'mongo_test')
-    validate_event_keys(traces[1], @entry_kvs)
-    validate_event_keys(traces[2], @exit_kvs)
+      traces[3].has_key?('Backtrace').must_equal Oboe::Config[:mongo][:collect_backtraces]
+      traces[3]['QueryOp'].must_equal "update"
+      traces[3]['Query'].must_equal "{\"_id\":1}"
+    end
 
-    traces[1]['Collection'].must_equal "testCollection"
-    traces[1].has_key?('Backtrace').must_equal Oboe::Config[:mongo][:collect_backtraces]
-    traces[1]['QueryOp'].must_equal "distinct"
-  end
+    it "should trace distinct" do
+      coll = @db.collection("testCollection")
 
-  it "should trace find" do
-    coll = @db.collection("testCollection")
-    result = nil
+      Oboe::API.start_trace('mongo_test', '', {}) do
+        coll.distinct("count")
+      end
 
-    # Insert a doc to assure we get a result
-    doc = {"name" => "MyName", "type" => "MyType", "count" => 1, "info" => {"x" => 203, "y" => '102'}}
-    id = coll.insert(doc)
+      traces = get_all_traces
+      traces.count.must_equal 4
 
-    # If given an optional block +find+ will yield a Cursor to that block,
-    # close the cursor, and then return nil. This guarantees that partially
-    # evaluated cursors will be closed. If given no block +find+ returns a
-    # cursor.
-    # https://github.com/mongodb/mongo-ruby-driver/blob/1.10.1/lib/mongo/collection.rb#L178
+      validate_outer_layers(traces, 'mongo_test')
+      validate_event_keys(traces[1], @entry_kvs)
+      validate_event_keys(traces[2], @exit_kvs)
 
-    Oboe::API.start_trace('mongo_test', '', {}) do
-      result = coll.find(:name => "MyName", :limit => 1)
+      traces[1]['Collection'].must_equal "testCollection"
+      traces[1].has_key?('Backtrace').must_equal Oboe::Config[:mongo][:collect_backtraces]
+      traces[1]['QueryOp'].must_equal "distinct"
     end
 
-    traces = get_all_traces
-    traces.count.must_equal 4
+    it "should trace find" do
+      coll = @db.collection("testCollection")
+      result = nil
 
-    validate_outer_layers(traces, 'mongo_test')
-    validate_event_keys(traces[1], @entry_kvs)
-    validate_event_keys(traces[2], @exit_kvs)
-
-    result.wont_match nil
-    result.is_a?(Mongo::Cursor).must_equal true
-    traces[1]['Collection'].must_equal "testCollection"
-    traces[1].has_key?('Backtrace').must_equal Oboe::Config[:mongo][:collect_backtraces]
-    traces[1]['QueryOp'].must_equal "find"
-    traces[1].has_key?('Query').must_equal true
-    traces[1]['Limit'].must_equal "1"
-  end
+      # Insert a doc to assure we get a result
+      doc = {"name" => "MyName", "type" => "MyType", "count" => 1, "info" => {"x" => 203, "y" => '102'}}
+      id = coll.insert(doc)
 
-  it "should trace find (with block)" do
-    coll = @db.collection("testCollection")
-    result = []
+      # If given an optional block +find+ will yield a Cursor to that block,
+      # close the cursor, and then return nil. This guarantees that partially
+      # evaluated cursors will be closed. If given no block +find+ returns a
+      # cursor.
+      # https://github.com/mongodb/mongo-ruby-driver/blob/1.10.1/lib/mongo/collection.rb#L178
+
+      Oboe::API.start_trace('mongo_test', '', {}) do
+        result = coll.find(:name => "MyName", :limit => 1)
+      end
+
+      traces = get_all_traces
+      traces.count.must_equal 4
+
+      validate_outer_layers(traces, 'mongo_test')
+      validate_event_keys(traces[1], @entry_kvs)
+      validate_event_keys(traces[2], @exit_kvs)
+
+      result.wont_match nil
+      result.is_a?(Mongo::Cursor).must_equal true
+      traces[1]['Collection'].must_equal "testCollection"
+      traces[1].has_key?('Backtrace').must_equal Oboe::Config[:mongo][:collect_backtraces]
+      traces[1]['QueryOp'].must_equal "find"
+      traces[1].has_key?('Query').must_equal true
+      traces[1]['Limit'].must_equal "1"
+    end
 
-    # Insert a doc to assure we get a result
-    doc = {"name" => "MyName", "type" => "MyType", "count" => 1, "info" => {"x" => 203, "y" => '102'}}
-    id = coll.insert(doc)
+    it "should trace find (with block)" do
+      coll = @db.collection("testCollection")
+      result = []
 
-    # If given an optional block +find+ will yield a Cursor to that block,
-    # close the cursor, and then return nil. This guarantees that partially
-    # evaluated cursors will be closed. If given no block +find+ returns a
-    # cursor.
-    # https://github.com/mongodb/mongo-ruby-driver/blob/1.10.1/lib/mongo/collection.rb#L178
+      # Insert a doc to assure we get a result
+      doc = {"name" => "MyName", "type" => "MyType", "count" => 1, "info" => {"x" => 203, "y" => '102'}}
+      id = coll.insert(doc)
 
-    Oboe::API.start_trace('mongo_test', '', {}) do
-      blk = lambda { |x| x }
-      result = coll.find(:name => "MyName", :limit => 10, &blk)
+      # If given an optional block +find+ will yield a Cursor to that block,
+      # close the cursor, and then return nil. This guarantees that partially
+      # evaluated cursors will be closed. If given no block +find+ returns a
+      # cursor.
+      # https://github.com/mongodb/mongo-ruby-driver/blob/1.10.1/lib/mongo/collection.rb#L178
+
+      Oboe::API.start_trace('mongo_test', '', {}) do
+        blk = lambda { |x| x }
+        result = coll.find(:name => "MyName", :limit => 10, &blk)
+      end
+
+      traces = get_all_traces
+      traces.count.must_equal 4
+
+      validate_outer_layers(traces, 'mongo_test')
+      validate_event_keys(traces[1], @entry_kvs)
+      validate_event_keys(traces[2], @exit_kvs)
+
+      result.must_equal nil
+      traces[1]['Collection'].must_equal "testCollection"
+      traces[1].has_key?('Backtrace').must_equal Oboe::Config[:mongo][:collect_backtraces]
+      traces[1]['QueryOp'].must_equal "find"
+      traces[1].has_key?('Query').must_equal true
     end
 
-    traces = get_all_traces
-    traces.count.must_equal 4
+    it "should trace group" do
+      coll = @db.collection("testCollection")
 
-    validate_outer_layers(traces, 'mongo_test')
-    validate_event_keys(traces[1], @entry_kvs)
-    validate_event_keys(traces[2], @exit_kvs)
+      Oboe::API.start_trace('mongo_test', '', {}) do
+        coll.group( :key =>  :type,
+                    :cond => { :count => 1 },
+                    :initial =>  { :count => 0 },
+                    :reduce => 'function(obj,prev) { prev.count += obj.c; }')
+      end
 
-    result.must_equal nil
-    traces[1]['Collection'].must_equal "testCollection"
-    traces[1].has_key?('Backtrace').must_equal Oboe::Config[:mongo][:collect_backtraces]
-    traces[1]['QueryOp'].must_equal "find"
-    traces[1].has_key?('Query').must_equal true
-  end
+      traces = get_all_traces
+      traces.count.must_equal 4
 
-  it "should trace group" do
-    coll = @db.collection("testCollection")
+      validate_outer_layers(traces, 'mongo_test')
+      validate_event_keys(traces[1], @entry_kvs)
+      validate_event_keys(traces[2], @exit_kvs)
 
-    Oboe::API.start_trace('mongo_test', '', {}) do
-      coll.group( :key =>  :type,
-                  :cond => { :count => 1 },
-                  :initial =>  { :count => 0 },
-                  :reduce => 'function(obj,prev) { prev.count += obj.c; }')
+      traces[1]['Collection'].must_equal "testCollection"
+      traces[1].has_key?('Backtrace').must_equal Oboe::Config[:mongo][:collect_backtraces]
+      traces[1]['QueryOp'].must_equal "group"
+      # Don't test exact hash value since to_json hash ordering varies between 1.8.7 and 1.9+
+      traces[1].has_key?('Query').must_equal true
     end
 
-    traces = get_all_traces
-    traces.count.must_equal 4
+    it "should trace create, ensure and drop index" do
+      coll = @db.collection("testCollection")
 
-    validate_outer_layers(traces, 'mongo_test')
-    validate_event_keys(traces[1], @entry_kvs)
-    validate_event_keys(traces[2], @exit_kvs)
+      Oboe::API.start_trace('mongo_test', '', {}) do
+        coll.create_index("i")
+        coll.ensure_index("i")
+        coll.drop_index("i_1")
+      end
 
-    traces[1]['Collection'].must_equal "testCollection"
-    traces[1].has_key?('Backtrace').must_equal Oboe::Config[:mongo][:collect_backtraces]
-    traces[1]['QueryOp'].must_equal "group"
-    # Don't test exact hash value since to_json hash ordering varies between 1.8.7 and 1.9+
-    traces[1].has_key?('Query').must_equal true
-  end
+      traces = get_all_traces
+      traces.count.must_equal 8
 
-  it "should trace create, ensure and drop index" do
-    coll = @db.collection("testCollection")
+      validate_outer_layers(traces, 'mongo_test')
+      validate_event_keys(traces[1], @entry_kvs)
+      validate_event_keys(traces[2], @exit_kvs)
 
-    Oboe::API.start_trace('mongo_test', '', {}) do
-      coll.create_index("i")
-      coll.ensure_index("i")
-      coll.drop_index("i_1")
-    end
+      traces[1]['Collection'].must_equal "testCollection"
+      traces[1].has_key?('Backtrace').must_equal Oboe::Config[:mongo][:collect_backtraces]
+      traces[1]['QueryOp'].must_equal "create_index"
 
-    traces = get_all_traces
-    traces.count.must_equal 8
+      validate_event_keys(traces[3], @entry_kvs)
+      validate_event_keys(traces[4], @exit_kvs)
 
-    validate_outer_layers(traces, 'mongo_test')
-    validate_event_keys(traces[1], @entry_kvs)
-    validate_event_keys(traces[2], @exit_kvs)
+      traces[3]['Collection'].must_equal "testCollection"
+      traces[3].has_key?('Backtrace').must_equal Oboe::Config[:mongo][:collect_backtraces]
+      traces[3]['QueryOp'].must_equal "ensure_index"
 
-    traces[1]['Collection'].must_equal "testCollection"
-    traces[1].has_key?('Backtrace').must_equal Oboe::Config[:mongo][:collect_backtraces]
-    traces[1]['QueryOp'].must_equal "create_index"
+      validate_event_keys(traces[5], @entry_kvs)
+      validate_event_keys(traces[6], @exit_kvs)
 
-    validate_event_keys(traces[3], @entry_kvs)
-    validate_event_keys(traces[4], @exit_kvs)
+      traces[5]['Collection'].must_equal "testCollection"
+      traces[5].has_key?('Backtrace').must_equal Oboe::Config[:mongo][:collect_backtraces]
+      traces[5]['QueryOp'].must_equal "drop_index"
+    end
 
-    traces[3]['Collection'].must_equal "testCollection"
-    traces[3].has_key?('Backtrace').must_equal Oboe::Config[:mongo][:collect_backtraces]
-    traces[3]['QueryOp'].must_equal "ensure_index"
+    it "should trace drop_indexes" do
+      coll = @db.collection("testCollection")
 
-    validate_event_keys(traces[5], @entry_kvs)
-    validate_event_keys(traces[6], @exit_kvs)
+      Oboe::API.start_trace('mongo_test', '', {}) do
+        coll.drop_indexes
+      end
 
-    traces[5]['Collection'].must_equal "testCollection"
-    traces[5].has_key?('Backtrace').must_equal Oboe::Config[:mongo][:collect_backtraces]
-    traces[5]['QueryOp'].must_equal "drop_index"
-  end
+      traces = get_all_traces
+      traces.count.must_equal 4
 
-  it "should trace drop_indexes" do
-    coll = @db.collection("testCollection")
+      validate_outer_layers(traces, 'mongo_test')
+      validate_event_keys(traces[1], @entry_kvs)
+      validate_event_keys(traces[2], @exit_kvs)
 
-    Oboe::API.start_trace('mongo_test', '', {}) do
-      coll.drop_indexes
+      traces[1]['Collection'].must_equal "testCollection"
+      traces[1].has_key?('Backtrace').must_equal Oboe::Config[:mongo][:collect_backtraces]
+      traces[1]['QueryOp'].must_equal "drop_indexes"
     end
 
-    traces = get_all_traces
-    traces.count.must_equal 4
+    it "should trace index_information" do
+      coll = @db.collection("testCollection")
 
-    validate_outer_layers(traces, 'mongo_test')
-    validate_event_keys(traces[1], @entry_kvs)
-    validate_event_keys(traces[2], @exit_kvs)
+      Oboe::API.start_trace('mongo_test', '', {}) do
+        coll.index_information
+      end
 
-    traces[1]['Collection'].must_equal "testCollection"
-    traces[1].has_key?('Backtrace').must_equal Oboe::Config[:mongo][:collect_backtraces]
-    traces[1]['QueryOp'].must_equal "drop_indexes"
-  end
+      traces = get_all_traces
+      traces.count.must_equal 4
 
-  it "should trace index_information" do
-    coll = @db.collection("testCollection")
+      validate_outer_layers(traces, 'mongo_test')
+      validate_event_keys(traces[1], @entry_kvs)
+      validate_event_keys(traces[2], @exit_kvs)
 
-    Oboe::API.start_trace('mongo_test', '', {}) do
-      coll.index_information
+      traces[1]['Collection'].must_equal "testCollection"
+      traces[1].has_key?('Backtrace').must_equal Oboe::Config[:mongo][:collect_backtraces]
+      traces[1]['QueryOp'].must_equal "index_information"
     end
 
-    traces = get_all_traces
-    traces.count.must_equal 4
-
-    validate_outer_layers(traces, 'mongo_test')
-    validate_event_keys(traces[1], @entry_kvs)
-    validate_event_keys(traces[2], @exit_kvs)
-
-    traces[1]['Collection'].must_equal "testCollection"
-    traces[1].has_key?('Backtrace').must_equal Oboe::Config[:mongo][:collect_backtraces]
-    traces[1]['QueryOp'].must_equal "index_information"
-  end
+    it "should obey :collect_backtraces setting when true" do
+      Oboe::Config[:mongo][:collect_backtraces] = true
 
-  it "should obey :collect_backtraces setting when true" do
-    Oboe::Config[:mongo][:collect_backtraces] = true
+      coll = @db.collection("testCollection")
 
-    coll = @db.collection("testCollection")
+      Oboe::API.start_trace('mongo_test', '', {}) do
+        doc = {"name" => "MyName", "type" => "MyType", "count" => 1, "info" => {"x" => 203, "y" => '102'}}
+        id = coll.insert(doc)
+      end
 
-    Oboe::API.start_trace('mongo_test', '', {}) do
-      doc = {"name" => "MyName", "type" => "MyType", "count" => 1, "info" => {"x" => 203, "y" => '102'}}
-      id = coll.insert(doc)
+      traces = get_all_traces
+      layer_has_key(traces, 'mongo', 'Backtrace')
     end
 
-    traces = get_all_traces
-    layer_has_key(traces, 'mongo', 'Backtrace')
-  end
+    it "should obey :collect_backtraces setting when false" do
+      Oboe::Config[:mongo][:collect_backtraces] = false
 
-  it "should obey :collect_backtraces setting when false" do
-    Oboe::Config[:mongo][:collect_backtraces] = false
+      coll = @db.collection("testCollection")
 
-    coll = @db.collection("testCollection")
+      Oboe::API.start_trace('mongo_test', '', {}) do
+        doc = {"name" => "MyName", "type" => "MyType", "count" => 1, "info" => {"x" => 203, "y" => '102'}}
+        id = coll.insert(doc)
+      end
 
-    Oboe::API.start_trace('mongo_test', '', {}) do
-      doc = {"name" => "MyName", "type" => "MyType", "count" => 1, "info" => {"x" => 203, "y" => '102'}}
-      id = coll.insert(doc)
+      traces = get_all_traces
+      layer_doesnt_have_key(traces, 'mongo', 'Backtrace')
     end
-
-    traces = get_all_traces
-    layer_doesnt_have_key(traces, 'mongo', 'Backtrace')
   end
 end