Don't use the typeless addAssertion syntax (silences deprecation warning)

papandreou · papandreou · commit 1869b96e371d · 2019-10-26T15:07:31.000+02:00
diff --git a/test/jsdom-compatibility.js b/test/jsdom-compatibility.js
@@ -5,25 +5,28 @@ const jsdom = require('jsdom');
 const expect = unexpected.clone().installPlugin(unexpectedDom);
 expect.output.installPlugin(require('magicpen-prism'));
 
-expect.addAssertion('to inspect as [itself]', function(expect, subject, value) {
-  const originalSubject = subject;
-  if (typeof subject === 'string') {
-    subject = new jsdom.JSDOM(
-      `<!DOCTYPE html><html><head></head><body>${subject}</body></html>`
-    ).window.document.body.firstChild;
-  }
-  if (this.flags.itself) {
-    if (typeof originalSubject === 'string') {
-      expect(expect.inspect(subject).toString(), 'to equal', originalSubject);
+expect.addAssertion(
+  '<string|DOMNode> to inspect as [itself] <string?>',
+  function(expect, subject, value) {
+    const originalSubject = subject;
+    if (typeof subject === 'string') {
+      subject = new jsdom.JSDOM(
+        `<!DOCTYPE html><html><head></head><body>${subject}</body></html>`
+      ).window.document.body.firstChild;
+    }
+    if (this.flags.itself) {
+      if (typeof originalSubject === 'string') {
+        expect(expect.inspect(subject).toString(), 'to equal', originalSubject);
+      } else {
+        throw new Error(
+          'subject must be given as a string when expected to inspect as itself'
+        );
+      }
     } else {
-      throw new Error(
-        'subject must be given as a string when expected to inspect as itself'
-      );
+      expect(expect.inspect(subject).toString(), 'to equal', value);
     }
-  } else {
-    expect(expect.inspect(subject).toString(), 'to equal', value);
   }
-});
+);
 
 describe('jsdom bug compatibility', () => {
   it('should work without issue #1107 fixed', () => {
