fn.default check for transpiled modules

[nurdism]

More info in issue #60

[mcollina]

Can you add a unit test for this?

[nurdism]

I'm not very familiar with unit testing in general. But would something like this suffice?

test('should return "default" function from ECMAScript module', t => {
  t.plan(1)

  const plugin = {
    default: () => {}
  }

  try {
    fp(plugin)
    t.pass()
  } catch (e) {
    t.is(e.message, 'fastify-plugin expects a function, instead got a \'object\'')
  }
})

or are you looking for something more in-depth?

[mcollina]

that would work, yes!

[mcollina]

also, linting is failing.

[mcollina]

LGTM

[mcollina]

@Ethan-Arrowood can you confirm this would work well with ts as well?

[cemremengu]

LGTM

[Ethan-Arrowood]

If you copy-paste your new unit test into the test.ts file, it will throw an error. A basic fix to this is to add | { default: fastify.Plugin<HttpServer, HttpRequest, HttpResponse, T>} to line 15 in fastify-plugin.d.ts. Otherwise LGTM 👍