fix: bring Done in from mocha (#50)

Also move @types/mocha to devDependencies since it's no longer directly
used

package.json

@@ -7,14 +7,14 @@
   "dependencies": {
     "@types/chai": "*",
     "@types/lodash": "*",
-    "@types/mocha": "*",
     "@types/node": "*",
     "@types/sinon": "*",
     "lodash": "^4.17.13",
     "mock-stdin": "^1.0.0",
     "stdout-stderr": "^0.1.9"
   },
   "devDependencies": {
+    "@types/mocha": "*",
     "chai": "^4.2.0",
     "chalk": "^2.4.2",
     "eslint": "^7.3.1",

src/base.ts

@@ -35,7 +35,7 @@ const base = <I extends Types.Context>(context: I): Types.Base<I, {}> => {
       arg1 = undefined
     }
     if (!arg1) arg1 = context.expectation || 'test'
-    async function run(this: Types.ITestCallbackContext, done?: Mocha.Done) {
+    async function run(this: Types.ITestCallbackContext, done?: Types.MochaDone) {
       context = assignWithProps({}, originalContext)
       if (context.retries) this.retries(context.retries)
       if (cb) {

src/types.ts

@@ -34,7 +34,7 @@ export interface ITestCallbackContext {
   [index: string]: any;
 }
 
-export type MochaCallback<I> = (this: ITestCallbackContext, context: I, done: Mocha.Done) => any
+export type MochaCallback<I> = (this: ITestCallbackContext, context: I, done: MochaDone) => any
 export interface It<I> {
   (expectation: string, cb?: MochaCallback<I>): void;
   (cb?: MochaCallback<I>): void;
@@ -53,6 +53,8 @@ export interface EnvOptions {
   clear?: boolean;
 }
 
+export type MochaDone = (err?: any) => void
+
 // eslint-disable-next-line @typescript-eslint/no-empty-interface
 export interface NockScope extends Nock.Scope {}
 // eslint-disable-next-line @typescript-eslint/no-empty-interface