RUBY-900 Test not adding servers to the cluster with Single topology

Author: estolfo

spec/mongo/cluster/topology_spec.rb

@@ -24,6 +24,10 @@
       it 'returns a single topology' do
         expect(topology).to be_a(Mongo::Cluster::Topology::Single)
       end
+
+      it 'sets the seed on the topology' do
+        expect(topology.seed).to eq('a')
+      end
     end
 
     context 'when provided a sharded option' do

spec/mongo/cluster_spec.rb

@@ -2,11 +2,11 @@
 
 describe Mongo::Cluster do
 
-  describe '#==' do
+  let(:cluster) do
+    described_class.new(ADDRESSES)
+  end
 
-    let(:cluster) do
-      described_class.new([ '127.0.0.1:27017' ])
-    end
+  describe '#==' do
 
     context 'when the other is a cluster' do
 
@@ -61,10 +61,6 @@
       Mongo::ServerSelector.get
     end
 
-    let(:cluster) do
-      described_class.new([ '127.0.0.1:27017' ])
-    end
-
     it 'displays the cluster seeds and topology' do
       expect(cluster.inspect).to include('topology')
       expect(cluster.inspect).to include('servers')
@@ -78,13 +74,13 @@
     end
 
     let(:cluster) do
-      described_class.new([ '127.0.0.1:27017' ], :replica_set => 'testing')
+      described_class.new(ADDRESSES, :replica_set => 'testing')
     end
 
     context 'when the option is provided' do
 
       let(:cluster) do
-        described_class.new([ '127.0.0.1:27017' ], :replica_set => 'testing')
+        described_class.new(ADDRESSES, :replica_set => 'testing')
       end
 
       it 'returns the name' do
@@ -95,7 +91,7 @@
     context 'when the option is not provided' do
 
       let(:cluster) do
-        described_class.new([ '127.0.0.1:27017' ])
+        described_class.new(ADDRESSES)
       end
 
       it 'returns nil' do
@@ -110,10 +106,6 @@
       Mongo::ServerSelector.get
     end
 
-    let(:cluster) do
-      described_class.new([ '127.0.0.1:27017' ])
-    end
-
     let(:known_servers) do
       cluster.instance_variable_get(:@servers)
     end
@@ -131,10 +123,6 @@
 
     context 'when topology is single', if: single_seed? do
 
-      let(:cluster) do
-        described_class.new(ADDRESSES)
-      end
-
       context 'when the server is a mongos', if: single_mongos?  do
 
         it 'returns the mongos' do
@@ -152,10 +140,6 @@
 
     context 'when the cluster has no servers' do
 
-      let(:cluster) do
-        described_class.new(ADDRESSES)
-      end
-
       let(:servers) do
         [nil]
       end
@@ -210,4 +194,22 @@
       end
     end
   end
+
+  describe '#add' do
+
+    context 'when topology is Single' do
+
+      let(:topology) do
+        Mongo::Cluster::Topology::Single.new({})
+      end
+
+      before do
+        cluster.add('a')
+      end
+
+      it 'does not add discovered servers to the cluster' do
+        expect(cluster.servers[0].address.seed).to_not eq('a')
+      end
+    end
+  end
 end