node-tap mocking alongside ts-node

This might work better as a different issue type, if so let me know.

Expected Behavior

I am able to mock require

Actual Behavior

I receive this relatively unhelpful (to me) error stack

    Object.<anonymous> (test/dosStub.ts:8:3)

    Module.m._compile (node_modules/ts-node/src/index.ts:1225:23)

    Object.require.extensions.<computed> [as .ts] (node_modules/ts-node/src/index.ts:1228:12)

Steps to reproduce the problem

This is probably partly related to the node-tap’s way of mocking, which is documented here. https://node-tap.org/docs/api/mocks/#api

Minimal reproduction

Since my code is already quite short, pasting it here in it’s entirety

import t from "tap";
const { DosStub} = t.mock("../lib/dosStub.ts", {
  console: {
    error: () => {}
  }
});

t.test("DosStub", (t) => {
  t.throws(function() {new DosStub()}, 'foo')
  t.done
});

The contents of the dosStub.ts file are a single exported class. Since this is a require error, I don’t think the content of that file are relevant, but I will share if desired.

https://github.com/TypeStrong/ts-node-repros/pull/15

Specifications

• ts-node version: v10.1.0
• node version: v14.16.1
• TypeScript version: v4.3.5
• tsconfig.json, if you’re using one:

{ }
  

• Operating system and version:
• If Windows, are you using WSL or WSL2?:

Description: Manjaro Linux Release: 21.1.0

[cspotcode]

Ok yeah, still has a typo in t.ts

[cspotcode]

Yes, please test this locally, and when you’re sure it is correct, I’ll rerun the reproduction.

On Thu, Aug 5, 2021, 12:13 AM Drazi Crendraven @.***> wrote:

There are two commits since I’m editing on web on mobile. I think the typo was fixed with the second one? Otherwise I’m not seeing it and will have to review tomorrow

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/TypeStrong/ts-node/issues/1417#issuecomment-893149923, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAC35OA74K6LVGV7RBB4Z6TT3IFYDANCNFSM5BSOC66Q .