fix tests to account for new CAP order

Author: xyzshantaram

plugins/registration.ts

@@ -102,18 +102,19 @@ export default createPlugin(
     if (!password) {
       sendRegistration();
       return client.utils.negotiateCapabilities({ completeImmediately: true });
-    }
-    else if (authMethod === "NickServ") {
+    } else if (authMethod === "NickServ") {
       client.utils.negotiateCapabilities({ completeImmediately: true });
       sendRegistration();
       tryNickServ();
     } else {
       client.utils.negotiateCapabilities({ extraCaps: ["sasl"] });
-      sendRegistration();
       trySasl();
       client.once(
         "raw:rpl_saslsuccess",
-        () => client.utils.completeCapNegotiation(),
+        () => {
+          client.utils.completeCapNegotiation();
+          sendRegistration();
+        },
       );
     }
   });

plugins/registration_test.ts

@@ -70,10 +70,10 @@ describe("plugins/registration", (test) => {
     await client.connect("");
     const raw = server.receive();
     assertEquals(raw, [
-      "NICK me",
-      "USER user 0 * :real name",
       "CAP REQ multi-prefix",
       "CAP END",
+      "NICK me",
+      "USER user 0 * :real name",
       "PRIVMSG NickServ :identify user password",
     ]);
   });
@@ -90,11 +90,11 @@ describe("plugins/registration", (test) => {
     const raw = server.receive();
 
     assertEquals(raw, [
-      "NICK me",
-      "USER user 0 * :real name",
       "CAP REQ multi-prefix",
       "CAP REQ sasl",
       "CAP END",
+      "NICK me",
+      "USER user 0 * :real name",
     ]);
 
     server.send(":serverhost CAP me ACK :sasl");