updating tests

Author: kyleburgess2025

lib/mongo/uri/srv_protocol.rb

@@ -184,6 +184,9 @@ def validate_srv_hostname(hostname)
         if parts.any?(&:empty?)
           raise_invalid_error!("Hostname cannot have consecutive dots: #{hostname}")
         end
+        if parts.length == 0
+          raise_invalid_error!("Hostname cannot be empty: #{hostname}")
+        end
       end
 
       # Obtains the TXT options of a host.

spec/mongo/uri/srv_protocol_spec.rb

@@ -56,16 +56,6 @@
       end
     end
 
-    context 'when the host in URI does not have {hostname}, {domainname} and {tld}' do
-
-      let(:string) { "#{scheme}#{hosts}" }
-      let(:hosts) { '10gen.cc/' }
-
-      it 'raises an error' do
-        expect { uri }.to raise_error(Mongo::Error::InvalidURI)
-      end
-    end
-
     context 'when the {tld} is empty' do
 
       let(:string) { "#{scheme}#{hosts}" }
@@ -220,24 +210,6 @@
         expect { uri }.to raise_error(Mongo::Error::InvalidURI)
       end
     end
-
-    context 'mongodb+srv://example.com?w=1' do
-
-      let(:string) { "#{scheme}example.com?w=1" }
-
-      it 'raises an error' do
-        expect { uri }.to raise_error(Mongo::Error::InvalidURI)
-      end
-    end
-
-    context 'mongodb+srv://example.com/?w' do
-
-      let(:string) { "#{scheme}example.com/?w" }
-
-      it 'raises an error' do
-        expect { uri }.to raise_error(Mongo::Error::InvalidURI)
-      end
-    end
   end
 
   describe 'valid uris' do
@@ -1302,8 +1274,8 @@
         'a'
       end
 
-      it 'raises an error' do
-        expect { validate }.to raise_error(Mongo::Error::InvalidURI)
+      it 'does not raise an error' do
+        expect { validate }.to_not raise_error
       end
     end
 
@@ -1313,8 +1285,8 @@
         'a.b'
       end
 
-      it 'raises an error' do
-        expect { validate }.to raise_error(Mongo::Error::InvalidURI)
+      it 'validates the hostname' do
+        expect { validate }.not_to raise_error
       end
     end