🚨 SASL: add_authenticator warns on reassignment [🚧WIP: no test]

Author: nevans

lib/net/imap/authenticators.rb

@@ -15,8 +15,12 @@ module Net::IMAP::Authenticators
   # The authenticator session must respond to +#process+, receiving the server's
   # challenge and returning the client's response.  See PlainAuthenticator,
   # XOauth2Authenticator, DigestMD5Authenticator, etc for examples.
-  def add_authenticator(auth_type, authenticator)
-    authenticators[auth_type] = authenticator
+  def add_authenticator(mechanism, authenticator)
+    if (original = authenticators[mechanism])
+      warn("%p: replacing %p authenticator: %p" % [ self, mechanism, original],
+           uplevel: 1)
+    end
+    authenticators[mechanism] = authenticator
   end
 
   # :call-seq: