Uniform #to_s and #inspect implementations.

fixtures/with_temporary_directory.rb

@@ -8,10 +8,12 @@
 module WithTemporaryDirectory
 	attr :temporary_directory
 
-	def around
+	def around(&block)
 		Dir.mktmpdir do |temporary_directory|
 			@temporary_directory = temporary_directory
-			yield
+			super(&block)
+		ensure
+			@temporary_directory = nil
 		end
 	end
 end

lib/io/endpoint/address_endpoint.rb

@@ -17,6 +17,17 @@ def initialize(address, **options)
 		end
 
 		def to_s
+			case @address.afamily
+			when Socket::AF_INET
+				"inet:#{@address.inspect_sockaddr}"
+			when Socket::AF_INET6
+				"inet6:#{@address.inspect_sockaddr}"
+			else
+				"address:#{@address.inspect_sockaddr}"
+			end
+		end
+
+		def inspect
 			"\#<#{self.class} address=#{@address.inspect}>"
 		end
 

lib/io/endpoint/bound_endpoint.rb

@@ -55,6 +55,10 @@ def close
 		end
 
 		def to_s
+			"bound:#{@endpoint}"
+		end
+
+		def inspect
 			"\#<#{self.class} #{@sockets.size} bound sockets for #{@endpoint}>"
 		end
 

lib/io/endpoint/composite_endpoint.rb

@@ -19,6 +19,14 @@ def initialize(endpoints, **options)
 			@endpoints = endpoints
 		end
 
+		def to_s
+			"composite:#{@endpoints.join(",")}"
+		end
+
+		def inspect
+			"\#<#{self.class} endpoints=#{@endpoints}>"
+		end
+
 		def with(**options)
 			self.class.new(endpoints.map{|endpoint| endpoint.with(**options)}, **@options.merge(options))
 		end

lib/io/endpoint/connected_endpoint.rb

@@ -57,6 +57,10 @@ def close
 		end
 
 		def to_s
+			"connected:#{@endpoint}"
+		end
+
+		def inspect
 			"\#<#{self.class} #{@socket} connected for #{@endpoint}>"
 		end
 	end

lib/io/endpoint/host_endpoint.rb

@@ -14,6 +14,10 @@ def initialize(specification, **options)
 		end
 
 		def to_s
+			"host:#{@specification[0]}:#{@specification[1]}"
+		end
+
+		def inspect
 			nodename, service, family, socktype, protocol, flags = @specification
 
 			"\#<#{self.class} name=#{nodename.inspect} service=#{service.inspect} family=#{family.inspect} type=#{socktype.inspect} protocol=#{protocol.inspect} flags=#{flags.inspect}>"

lib/io/endpoint/socket_endpoint.rb

@@ -15,6 +15,10 @@ def initialize(socket, **options)
 		end
 
 		def to_s
+			"socket:#{@socket}"
+		end
+
+		def inspect
 			"\#<#{self.class} #{@socket.inspect}>"
 		end
 

lib/io/endpoint/ssl_endpoint.rb

@@ -87,7 +87,11 @@ def initialize(endpoint, **options)
 		end
 
 		def to_s
-			"\#<#{self.class} #{@endpoint}>"
+			"ssl:#{@endpoint}"
+		end
+
+		def inspect
+			"\#<#{self.class} endpoint=#{@endpoint.inspect}>"
 		end
 
 		def address

lib/io/endpoint/unix_endpoint.rb

@@ -16,7 +16,11 @@ def initialize(path, type = Socket::SOCK_STREAM, **options)
 		end
 
 		def to_s
-			"\#<#{self.class} #{@path.inspect}>"
+			"unix:#{@path}"
+		end
+
+		def inspect
+			"\#<#{self.class} path=#{@path.inspect}>"
 		end
 
 		attr :path

test/io/endpoint/address_endpoint.rb

@@ -43,7 +43,13 @@
 
 	with "#to_s" do
 		it "can generate a string representation" do
-			expect(endpoint.to_s).to be =~ /#<IO::Endpoint::AddressEndpoint address=/
+			expect(endpoint.to_s).to be =~ /inet(6):/
+		end
+	end
+
+	with "#inspect" do
+		it "can generate a string representation" do
+			expect(endpoint.inspect).to be =~ /#<IO::Endpoint::AddressEndpoint address=/
 		end
 	end
 end

test/io/endpoint/bound_endpoint.rb

@@ -115,4 +115,24 @@
 			connected_endpoint&.close
 		end
 	end
+
+	with "a bound instance" do
+		let(:endpoint) {subject.bound(internal_endpoint)}
+
+		after do
+			@endpoint&.close
+		end
+
+		with "#to_s" do
+			it "can generate a string representation" do
+				expect(endpoint.to_s).to be =~ /bound:/
+			end
+		end
+
+		with "#inspect" do
+			it "can generate a string representation" do
+				expect(endpoint.inspect).to be =~ /#<IO::Endpoint::BoundEndpoint 1 bound sockets/
+			end
+		end
+	end
 end

test/io/endpoint/composite_endpoint.rb

@@ -77,4 +77,16 @@
 			end
 		end
 	end
+
+	with "#to_s" do
+		it "can generate a string representation" do
+			expect(endpoint.to_s).to be =~ /composite:/
+		end
+	end
+
+	with "#inspect" do
+		it "can generate a string representation" do
+			expect(endpoint.inspect).to be =~ /#<IO::Endpoint::CompositeEndpoint endpoints=/
+		end
+	end
 end

test/io/endpoint/host_endpoint.rb

@@ -42,9 +42,9 @@
 		bound&.close
 	end
 
-	with "#to_s" do
+	with "#inspect" do
 		it "can generate a string representation" do
-			expect(endpoint.to_s).to be == "#<IO::Endpoint::HostEndpoint name=\"localhost\" service=0 family=nil type=1 protocol=nil flags=nil>"
+			expect(endpoint.inspect).to be == "#<IO::Endpoint::HostEndpoint name=\"localhost\" service=0 family=nil type=1 protocol=nil flags=nil>"
 		end
 	end
 end

test/io/endpoint/ssl_endpoint.rb

@@ -87,4 +87,20 @@ def client_endpoint(address)
 			bound&.close
 		end
 	end
+
+	with "a simple SSL endpoint" do
+		let(:endpoint) {subject.new(IO::Endpoint.tcp("localhost", 0))}
+
+		with "#to_s" do
+			it "can generate a string representation" do
+				expect(endpoint.to_s).to be =~ /ssl:/
+			end
+		end
+
+		with "#inspect" do
+			it "can generate a string representation" do
+				expect(endpoint.inspect).to be =~ /#<IO::Endpoint::SSLEndpoint endpoint=/
+			end
+		end
+	end
 end

test/io/endpoint/unix_endpoint.rb

@@ -41,6 +41,10 @@
 
 		expect(endpoint).to be(:bound?)
 
+		# Wait for the server to start accepting connections:
+		# I noticed on slow CI, that the connect would fail because the server has not called `#accept` yet, even if it's bound and listening!
+		Thread.pass until thread.status == "sleep"
+
 		endpoint.connect do |socket|
 			expect(socket).to be_a(Socket)
 
@@ -53,6 +57,18 @@
 		thread&.kill
 		sockets&.each(&:close)
 	end
+
+	with "#to_s" do
+		it "can generate a string representation" do
+			expect(endpoint.to_s).to be =~ /unix:.*test\.ipc/
+		end
+	end
+
+	with "#inspect" do
+		it "can generate a string representation" do
+			expect(endpoint.inspect).to be =~ /#<IO::Endpoint::UNIXEndpoint path=.*test\.ipc/
+		end
+	end
 end
 
 describe IO::Endpoint do