Remove sinon

eslint.config.mjs

@@ -55,14 +55,8 @@ export default [
       // node
       // https://github.com/eslint-community/eslint-plugin-n/
       "n/no-unpublished-require": 0, // doesn't play nice with monorepo
-      "n/no-extraneous-require": [
-        "error",
-        { allowModules: ["sinon", "@xmpp/test"] },
-      ],
-      "n/no-extraneous-import": [
-        "error",
-        { allowModules: ["sinon", "@xmpp/test"] },
-      ],
+      "n/no-extraneous-require": ["error", { allowModules: ["@xmpp/test"] }],
+      "n/no-extraneous-import": ["error", { allowModules: ["@xmpp/test"] }],
       "n/hashbang": "off",
 
       // promise

package.json

@@ -36,7 +36,6 @@
     "node-fetch": "^2.6.12",
     "prettier": "^3.4.2",
     "selfsigned": "^2.4.1",
-    "sinon": "^19.0.2",
     "uglify-js": "^3.19.3"
   },
   "scripts": {

packages/jid/test/index.js

@@ -1,16 +1,14 @@
 "use strict";
 
-const { spy } = require("sinon");
 const jid = require("..");
 const JID = require("../lib/JID");
 
 test("equal calls equals on the first argument with the second argument", () => {
   const A = jid("foo");
   const B = jid("bar");
-  spy(A, "equals");
+  const spy_equals = jest.spyOn(A, "equals");
   jid.equal(A, B);
-  expect(A.equals.calledWith(B)).toBe(true);
-  A.equals.restore();
+  expect(spy_equals).toHaveBeenCalledWith(B);
 });
 
 test("JID exports lib/JID", () => {

packages/starttls/test.js

@@ -1,6 +1,7 @@
 "use strict";
 
-const { mock, stub } = require("sinon");
+jest.mock("tls");
+
 const { mockClient, promise, delay } = require("@xmpp/test");
 const tls = require("tls");
 const net = require("net");
@@ -18,31 +19,26 @@ test("success", async () => {
   const { socket, options } = entity;
   options.domain = "foobar";
 
-  const mockTLS = mock(tls);
-  const expectTLSConnect = mockTLS
-    .expects("connect")
-    .once()
-    .withArgs({ socket, host: "foobar" })
-    .callsFake(() => {
-      return new EventEmitter();
-    });
-
-  stub(entity, "_attachSocket");
-  stub(entity, "restart");
+  tls.connect.mockImplementation(() => {
+    return new EventEmitter();
+  });
 
   entity.mockInput(
     <features xmlns="http://etherx.jabber.org/streams">
       <starttls xmlns="urn:ietf:params:xml:ns:xmpp-tls" />
     </features>,
   );
 
-  expect(await promise(entity, "send")).toEqual(<starttls xmlns="urn:ietf:params:xml:ns:xmpp-tls" />);
+  expect(await promise(entity, "send")).toEqual(
+    <starttls xmlns="urn:ietf:params:xml:ns:xmpp-tls" />,
+  );
 
   entity.mockInput(<proceed xmlns="urn:ietf:params:xml:ns:xmpp-tls" />);
 
   await delay();
 
-  expectTLSConnect.verify();
+  expect(tls.connect).toHaveBeenCalledTimes(1);
+  expect(tls.connect).toHaveBeenCalledWith({ socket, host: "foobar" });
 });
 
 test("failure", async () => {
@@ -55,7 +51,9 @@ test("failure", async () => {
     </features>,
   );
 
-  expect(await promise(entity, "send")).toEqual(<starttls xmlns="urn:ietf:params:xml:ns:xmpp-tls" />);
+  expect(await promise(entity, "send")).toEqual(
+    <starttls xmlns="urn:ietf:params:xml:ns:xmpp-tls" />,
+  );
 
   entity.mockInput(<failure xmlns="urn:ietf:params:xml:ns:xmpp-tls" />);