test: Remove shouldjs dependency

Author: simnalamburt

package.json

@@ -25,7 +25,6 @@
     "irc-replies": "^2.0.1",
     "microbundle": "^0.15",
     "prettier": "^3",
-    "should": "^13.2.3",
     "vitest": "^2.1.2"
   },
   "scripts": {

test/away.test.js

@@ -1,5 +1,4 @@
-import { it } from 'vitest'
-import 'should'
+import { expect, it } from 'vitest'
 
 import irc from '..'
 import { PassThrough as Stream } from 'stream'
@@ -10,8 +9,8 @@ it('should emit "away"', () =>
     var client = irc(stream)
 
     client.on('away', function (e) {
-      e.nick.should.equal('colinm')
-      e.message.should.eql('brb food time')
+      expect(e.nick).toStrictEqual('colinm')
+      expect(e.message).toStrictEqual('brb food time')
       done()
     })
 

test/invite.test.js

@@ -1,5 +1,4 @@
-import { it } from 'vitest'
-import 'should'
+import { expect, it } from 'vitest'
 
 import irc from '..'
 import { PassThrough as Stream } from 'stream'
@@ -10,13 +9,13 @@ it('should emit "invite"', () =>
     var client = irc(stream)
 
     client.on('invite', function (e) {
-      e.from.should.equal('test')
-      e.to.should.equal('astranger')
-      e.channel.should.equal('#something')
-      e.hostmask.nick.should.equal('test')
-      e.hostmask.username.should.equal('~user')
-      e.hostmask.hostname.should.equal('example.com')
-      e.hostmask.string.should.equal('test!~user@example.com')
+      expect(e.from).toStrictEqual('test')
+      expect(e.to).toStrictEqual('astranger')
+      expect(e.channel).toStrictEqual('#something')
+      expect(e.hostmask.nick).toStrictEqual('test')
+      expect(e.hostmask.username).toStrictEqual('~user')
+      expect(e.hostmask.hostname).toStrictEqual('example.com')
+      expect(e.hostmask.string).toStrictEqual('test!~user@example.com')
       done()
     })
 

test/join.test.js

@@ -1,5 +1,4 @@
-import { it } from 'vitest'
-import 'should'
+import { expect, it } from 'vitest'
 
 import irc from '..'
 import { PassThrough as Stream } from 'stream'
@@ -10,12 +9,14 @@ it('should emit "join"', () =>
     var client = irc(stream)
 
     client.on('join', function (e) {
-      e.nick.should.equal('tjholowaychuk')
-      e.channel.should.equal('#express')
-      e.hostmask.nick.should.equal('tjholowaychuk')
-      e.hostmask.username.should.equal('~tjholoway')
-      e.hostmask.hostname.should.equal('S01067cb21b2fd643.gv.shawcable.net')
-      e.hostmask.string.should.equal(
+      expect(e.nick).toStrictEqual('tjholowaychuk')
+      expect(e.channel).toStrictEqual('#express')
+      expect(e.hostmask.nick).toStrictEqual('tjholowaychuk')
+      expect(e.hostmask.username).toStrictEqual('~tjholoway')
+      expect(e.hostmask.hostname).toStrictEqual(
+        'S01067cb21b2fd643.gv.shawcable.net',
+      )
+      expect(e.hostmask.string).toStrictEqual(
         'tjholowaychuk!~tjholoway@S01067cb21b2fd643.gv.shawcable.net',
       )
       done()

test/kick.test.js

@@ -1,5 +1,4 @@
-import { it } from 'vitest'
-import 'should'
+import { expect, it } from 'vitest'
 
 import irc from '..'
 import { PassThrough as Stream } from 'stream'
@@ -10,13 +9,15 @@ it('should emit "kick"', () =>
     var client = irc(stream)
 
     client.on('kick', function (e) {
-      e.nick.should.equal('tjholowaychuk')
-      e.client.should.equal('tobi')
-      e.channel.should.eql('#express')
-      e.hostmask.nick.should.equal('tjholowaychuk')
-      e.hostmask.username.should.equal('~tjholoway')
-      e.hostmask.hostname.should.equal('S01067cb21b2fd643.gv.shawcable.net')
-      e.hostmask.string.should.equal(
+      expect(e.nick).toStrictEqual('tjholowaychuk')
+      expect(e.client).toStrictEqual('tobi')
+      expect(e.channel).toStrictEqual('#express')
+      expect(e.hostmask.nick).toStrictEqual('tjholowaychuk')
+      expect(e.hostmask.username).toStrictEqual('~tjholoway')
+      expect(e.hostmask.hostname).toStrictEqual(
+        'S01067cb21b2fd643.gv.shawcable.net',
+      )
+      expect(e.hostmask.string).toStrictEqual(
         'tjholowaychuk!~tjholoway@S01067cb21b2fd643.gv.shawcable.net',
       )
       done()

test/names.test.js

@@ -1,5 +1,4 @@
-import { it } from 'vitest'
-import 'should'
+import { expect, it } from 'vitest'
 
 import irc from '..'
 import { PassThrough as Stream } from 'stream'
@@ -11,7 +10,7 @@ it('should respond with user names', () =>
 
     client.names('#luna-lang', function (err, names) {
       if (err) return done(err)
-      names.should.eql([
+      expect(names).toStrictEqual([
         { name: 'owner', mode: '~' },
         { name: 'foo', mode: '@' },
         { name: 'halfop', mode: '%' },
@@ -42,8 +41,8 @@ it('should emit "names"', () =>
     var client = irc(stream)
 
     client.on('names', function (e) {
-      e.channel.should.equal('#luna-lang')
-      e.names.should.eql([
+      expect(e.channel).toStrictEqual('#luna-lang')
+      expect(e.names).toStrictEqual([
         { name: 'one', mode: '' },
         { name: 'two', mode: '~' },
         { name: 'three', mode: '' },
@@ -72,8 +71,8 @@ it('should retain ~ / @ / % / +', () =>
     var client = irc(stream)
 
     client.on('names', function (e) {
-      e.channel.should.equal('##luna-lang')
-      e.names.should.eql([
+      expect(e.channel).toStrictEqual('##luna-lang')
+      expect(e.names).toStrictEqual([
         { name: 'owner', mode: '~' },
         { name: 'tjholowaychuk', mode: '@' },
         { name: 'halfop', mode: '%' },

test/nick.test.js

@@ -1,5 +1,4 @@
-import { it } from 'vitest'
-import 'should'
+import { expect, it } from 'vitest'
 
 import irc from '..'
 import { PassThrough as Stream } from 'stream'
@@ -10,12 +9,12 @@ it('should emit "nick"', () =>
     var client = irc(stream)
 
     client.on('nick', function (e) {
-      e.nick.should.eql('colinm')
-      e.new.should.equal('cmilhench')
-      e.hostmask.nick.should.equal('colinm')
-      e.hostmask.username.should.equal('~colinm')
-      e.hostmask.hostname.should.equal('host-92-17-247-88.as13285.net')
-      e.hostmask.string.should.equal(
+      expect(e.nick).toStrictEqual('colinm')
+      expect(e.new).toStrictEqual('cmilhench')
+      expect(e.hostmask.nick).toStrictEqual('colinm')
+      expect(e.hostmask.username).toStrictEqual('~colinm')
+      expect(e.hostmask.hostname).toStrictEqual('host-92-17-247-88.as13285.net')
+      expect(e.hostmask.string).toStrictEqual(
         'colinm!~colinm@host-92-17-247-88.as13285.net',
       )
       done()

test/notice.test.js

@@ -1,5 +1,4 @@
-import { it } from 'vitest'
-import 'should'
+import { expect, it } from 'vitest'
 
 import irc from '..'
 import { PassThrough as Stream } from 'stream'
@@ -11,15 +10,15 @@ it('should emit "notice"', () =>
     var n = 0
 
     client.on('notice', function (e) {
-      e.from.should.equal('NickServ')
-      e.to.should.equal('cmilhench')
-      e.hostmask.nick.should.equal('NickServ')
-      e.hostmask.username.should.equal('NickServ')
-      e.hostmask.hostname.should.equal('services.')
-      e.hostmask.string.should.equal('NickServ!NickServ@services.')
+      expect(e.from).toStrictEqual('NickServ')
+      expect(e.to).toStrictEqual('cmilhench')
+      expect(e.hostmask.nick).toStrictEqual('NickServ')
+      expect(e.hostmask.username).toStrictEqual('NickServ')
+      expect(e.hostmask.hostname).toStrictEqual('services.')
+      expect(e.hostmask.string).toStrictEqual('NickServ!NickServ@services.')
       switch (++n) {
         case 1:
-          e.message.should.equal(
+          expect(e.message).toStrictEqual(
             [
               'This nickname is registered. ',
               'Please choose a different nickname, ',
@@ -28,7 +27,7 @@ it('should emit "notice"', () =>
           )
           break
         case 2:
-          e.message.should.equal(
+          expect(e.message).toStrictEqual(
             [
               'You have 30 seconds to identify to your nickname ',
               'before it is changed.',

test/part.test.js

@@ -1,5 +1,4 @@
-import { it } from 'vitest'
-import 'should'
+import { expect, it } from 'vitest'
 
 import irc from '..'
 import { PassThrough as Stream } from 'stream'
@@ -10,13 +9,15 @@ it('should emit "part"', () =>
     var client = irc(stream)
 
     client.on('part', function (e) {
-      e.nick.should.equal('tjholowaychuk')
-      e.channels.should.eql(['#express'])
-      e.message.should.equal('So long!')
-      e.hostmask.nick.should.equal('tjholowaychuk')
-      e.hostmask.username.should.equal('~tjholoway')
-      e.hostmask.hostname.should.equal('S01067cb21b2fd643.gv.shawcable.net')
-      e.hostmask.string.should.equal(
+      expect(e.nick).toStrictEqual('tjholowaychuk')
+      expect(e.channels).toStrictEqual(['#express'])
+      expect(e.message).toStrictEqual('So long!')
+      expect(e.hostmask.nick).toStrictEqual('tjholowaychuk')
+      expect(e.hostmask.username).toStrictEqual('~tjholoway')
+      expect(e.hostmask.hostname).toStrictEqual(
+        'S01067cb21b2fd643.gv.shawcable.net',
+      )
+      expect(e.hostmask.string).toStrictEqual(
         'tjholowaychuk!~tjholoway@S01067cb21b2fd643.gv.shawcable.net',
       )
       done()

test/pong.test.js

@@ -1,5 +1,4 @@
-import { it } from 'vitest'
-import 'should'
+import { expect, it } from 'vitest'
 
 import irc from '..'
 import { PassThrough as Stream } from 'stream'
@@ -13,10 +12,10 @@ it('should respond with PONG', () =>
     stream.on('data', function (chunk) {
       switch (n++) {
         case 0:
-          chunk.should.equal('PING :rothfuss.freenode.net\r\n')
+          expect(chunk).toStrictEqual('PING :rothfuss.freenode.net\r\n')
           break
         case 1:
-          chunk.should.equal('PONG :rothfuss.freenode.net\r\n')
+          expect(chunk).toStrictEqual('PONG :rothfuss.freenode.net\r\n')
           done()
           break
       }

test/privmsg.test.js

@@ -1,5 +1,4 @@
-import { it } from 'vitest'
-import 'should'
+import { expect, it } from 'vitest'
 
 import irc from '..'
 import { PassThrough as Stream } from 'stream'
@@ -10,13 +9,13 @@ it('should emit "message"', () =>
     var client = irc(stream)
 
     client.on('message', function (e) {
-      e.from.should.equal('tobi')
-      e.to.should.equal('loki')
-      e.message.should.equal('Hello :)')
-      e.hostmask.nick.should.equal('tobi')
-      e.hostmask.username.should.equal('~user')
-      e.hostmask.hostname.should.equal('example.com')
-      e.hostmask.string.should.equal('tobi!~user@example.com')
+      expect(e.from).toStrictEqual('tobi')
+      expect(e.to).toStrictEqual('loki')
+      expect(e.message).toStrictEqual('Hello :)')
+      expect(e.hostmask.nick).toStrictEqual('tobi')
+      expect(e.hostmask.username).toStrictEqual('~user')
+      expect(e.hostmask.hostname).toStrictEqual('example.com')
+      expect(e.hostmask.string).toStrictEqual('tobi!~user@example.com')
       done()
     })
 

test/quit.test.js

@@ -1,5 +1,4 @@
-import { it } from 'vitest'
-import 'should'
+import { expect, it } from 'vitest'
 
 import irc from '..'
 import { PassThrough as Stream } from 'stream'
@@ -10,14 +9,14 @@ it('should emit "quit"', () =>
     var client = irc(stream)
 
     client.on('quit', function (e) {
-      e.nick.should.equal('tobi')
-      e.message.should.eql('Remote host closed the connection')
-      e.hostmask.nick.should.equal('tobi')
-      e.hostmask.username.should.equal('~tobi')
-      e.hostmask.hostname.should.equal(
+      expect(e.nick).toStrictEqual('tobi')
+      expect(e.message).toStrictEqual('Remote host closed the connection')
+      expect(e.hostmask.nick).toStrictEqual('tobi')
+      expect(e.hostmask.username).toStrictEqual('~tobi')
+      expect(e.hostmask.hostname).toStrictEqual(
         '107-214-168-243.lightspeed.cicril.sbcglobal.net',
       )
-      e.hostmask.string.should.equal(
+      expect(e.hostmask.string).toStrictEqual(
         'tobi!~tobi@107-214-168-243.lightspeed.cicril.sbcglobal.net',
       )
       done()

test/topic.test.js

@@ -1,5 +1,4 @@
-import { it } from 'vitest'
-import 'should'
+import { expect, it } from 'vitest'
 
 import irc from '..'
 import { PassThrough as Stream } from 'stream'
@@ -10,13 +9,13 @@ it('should emit "topic"', () =>
     var client = irc(stream)
 
     client.on('topic', function (e) {
-      e.nick.should.equal('tobi')
-      e.channel.should.eql('#slate')
-      e.topic.should.equal('Slate 1.0 is out!')
-      e.hostmask.nick.should.equal('tobi')
-      e.hostmask.username.should.equal('~user')
-      e.hostmask.hostname.should.equal('example.com')
-      e.hostmask.string.should.equal('tobi!~user@example.com')
+      expect(e.nick).toStrictEqual('tobi')
+      expect(e.channel).toStrictEqual('#slate')
+      expect(e.topic).toStrictEqual('Slate 1.0 is out!')
+      expect(e.hostmask.nick).toStrictEqual('tobi')
+      expect(e.hostmask.username).toStrictEqual('~user')
+      expect(e.hostmask.hostname).toStrictEqual('example.com')
+      expect(e.hostmask.string).toStrictEqual('tobi!~user@example.com')
       done()
     })
 

test/welcome.test.js

@@ -1,5 +1,4 @@
-import { it } from 'vitest'
-import 'should'
+import { expect, it } from 'vitest'
 
 import irc from '..'
 import { PassThrough as Stream } from 'stream'
@@ -11,7 +10,7 @@ it('should set client.me to the users nick', () => {
     ':cameron.freenode.net 001 tobi :Welcome to the freenode Internet Relay Chat Network tobi\r\n',
   )
   process.nextTick(() => {
-    client.me.should.equal('tobi')
+    expect(client.me).toStrictEqual('tobi')
   })
 })
 
@@ -21,7 +20,7 @@ it('should emit "welcome"', () =>
     var client = irc(stream)
 
     client.on('welcome', function (nick) {
-      nick.should.equal('tobi')
+      expect(nick).toStrictEqual('tobi')
       done()
     })