More automated tests (#152)

* started writing a couple of automated tests to chip in, working towards the 100% test coverage goal

* added a couple more tests and eslint fixes

* added all the cases for the basename

* better test description

* added the plan for the subtest too in the dicer-write.test.js

* PR review: removed the t.end() in dicer-write.test.js

---------

Co-authored-by: Giovanni Bucci <[email protected]>

Author: Giovanni Bucci

test/basename.test.js

@@ -0,0 +1,28 @@
+'use strict'
+
+const { test } = require('tap')
+const basename = require('../lib/utils/basename')
+
+test('basename', (t) => {
+  const testCases = [
+    { description: 'returns an empty string if the path is not a string', path: {}, expected: '' },
+    { description: 'returns an empty string if the path includes a \' and the char after is a .', path: 'path\\.', expected: '' },
+    { description: 'returns an empty string if the path includes a / and the char after is a .', path: 'path/.', expected: '' },
+    { description: 'returns an empty string if the path includes a \' and the chars after are a ..', path: 'path\\..', expected: '' },
+    { description: 'returns an empty string if the path includes a / and the chars after are a ..', path: 'path/..', expected: '' },
+    { description: 'returns the path if the path includes a \' and the rest is anything other than dots', path: 'path\\subpath', expected: 'subpath' },
+    { description: 'returns the path if the path includes a / and the rest is anything other than dots', path: 'path/subpath', expected: 'subpath' },
+    { description: 'returns an empty string if the path is a .', path: '.', expected: '' },
+    { description: 'returns an empty string if the path is a ..', path: '..', expected: '' },
+    { description: 'returns the path if the path is anything other than dots', path: 'subpath', expected: 'subpath' }
+  ]
+
+  t.plan(testCases.length)
+
+  testCases.forEach((testCase, index) => {
+    t.test(testCase.description, t => {
+      t.plan(1)
+      t.equal(basename(testCase.path), testCase.expected, `Test case ${index + 1}`)
+    })
+  })
+})

test/busboy-emit.test.js

@@ -0,0 +1,17 @@
+'use strict'
+
+const Busboy = require('../lib/main')
+const { test } = require('tap')
+
+test('busboy, emit', t => {
+  t.plan(1)
+
+  t.test('returns undefined when the event is called a second time and the busboy was already finished', t => {
+    const busboy = new Busboy({ headers: { 'content-type': 'application/x-www-form-urlencoded' } })
+    busboy._finished = true
+    busboy.emit('finish')
+
+    t.equal(busboy.emit('finish'), undefined)
+    t.end()
+  })
+})

test/dicer-write.test.js

@@ -0,0 +1,19 @@
+'use strict'
+
+const { test } = require('tap')
+const { Dicer } = require('../lib/main')
+
+test('dicer _write method', t => {
+  t.plan(1)
+
+  t.test('calls the callback cb() when headerFirst is set and all the data have been written', t => {
+    t.plan(1)
+    const dicer = new Dicer({ headerFirst: true })
+
+    dicer._write(Buffer.from('Content-Type: text/plain'), null, () => {
+      dicer._write(Buffer.from('Content-Type: text/plain'), null, () => {
+        t.pass('write method called')
+      })
+    })
+  })
+})

test/multipart-constructor.test.js

@@ -0,0 +1,16 @@
+'use strict'
+
+const Multipart = require('../lib/types/multipart')
+const Busboy = require('../lib/main')
+const { test } = require('tap')
+
+test('multipart constructor', t => {
+  t.plan(1)
+
+  t.test('throws if the boundary is not a string', t => {
+    const busboy = new Busboy({ headers: { 'content-type': 'application/x-www-form-urlencoded' } })
+
+    t.throws(() => new Multipart(busboy, { boundary: 123 }), new Error('Multipart: Boundary not found'))
+    t.end()
+  })
+})