Support ambiguous property types on mocked interfaces (#248)

* refactor recorded arguments

* enhance transformation type to allow callable interfaces

* implement logic to handle unresolved property type

* add regression test with failing assertion

* rework core types for terminating object

* simplify context switching

* support assertion on ambiguous properties

* support node.js lts

* remove ts-node from root package

* create compatiblity checks to verify test runners

Author: notanengineercom

.github/workflows/nodejs.yml

@@ -6,7 +6,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        node-version: [12, 14, 16, 18]
+        node-version: [12, 14, 16, 18, 20]
 
     steps:
     - uses: actions/checkout@v3

compatibility-checks/ava.config.js

@@ -0,0 +1,9 @@
+module.exports = {
+	extensions: {
+		ts: 'commonjs',
+	},
+	nodeArguments: [
+		'--require=tsx/cjs'
+	],
+	failWithoutAssertions: false
+}

compatibility-checks/jest.config.js

@@ -0,0 +1,5 @@
+/** @type {import('ts-jest').JestConfigWithTsJest} */
+module.exports = {
+  preset: 'ts-jest',
+  testEnvironment: 'node',
+};

compatibility-checks/package-lock.json

@@ -0,0 +1,40 @@
+{
+    "name": "substitute-compatibility-checks",
+    "version": "0.0.0",
+    "description": "Checks compatibility of @fluffy-spoon/substitute with various test runners.",
+    "license": "MIT",
+    "funding": {
+        "type": "opencollective",
+        "url": "https://opencollective.com/substitute-js#section-contribute"
+    },
+    "repository": {
+        "type": "git",
+        "url": "https://github.com/ffMathy/FluffySpoon.JavaScript.Testing.Faking.git"
+    },
+    "engines": {
+        "node": ">=20"
+    },
+    "scripts": {
+        "validate:ava": "ava test-runners/ava/**.ts",
+        "validate:jest": "jest test-runners/jest/**.ts",
+        "validate:mocha": "mocha --require tsx/cjs test-runners/mocha/**.ts",
+        "validate:nodejs": "node --require tsx/cjs --test ./test-runners/nodejs/*.ts",
+        "validate:vitest": "vitest run ./test-runners/vitest/*.ts",
+        "test": "node --require tsx/cjs --test test/**"
+    },
+    "devDependencies": {
+        "@types/mocha": "^10.0.6",
+        "@types/node": "^20.11.25",
+        "ansi-regex": "^6.0.1",
+        "ava": "^6.1.2",
+        "jest": "^29.7.0",
+        "mocha": "^10.3.0",
+        "ts-jest": "^29.1.2",
+        "tsx": "^4.7.1",
+        "typescript": "^5.4.2",
+        "vitest": "^1.3.1"
+    },
+    "volta": {
+        "node": "20.11.1"
+    }
+}

compatibility-checks/package.json

@@ -0,0 +1,21 @@
+import Substitute from '../../../src'
+
+interface Library {
+  subSection: AmbiguousSection
+}
+
+interface AmbiguousSection {
+  (): string
+  subMethod: () => string
+}
+
+
+export const ambiguousPropertyTypeAssertion = () => {
+  const lib = Substitute.for<Library>()
+  lib.subSection().returns('subSection as method')
+  lib.subSection.returns({ subMethod: () => 'subSection as property' } as AmbiguousSection)
+
+  lib.subSection()
+  lib.subSection.subMethod()
+  lib.received(2).subSection
+}

compatibility-checks/test-runners/_common/index.ts

@@ -0,0 +1,6 @@
+import test from 'ava'
+import { ambiguousPropertyTypeAssertion } from '../_common'
+
+test('can substitute callable interfaces', () => {
+  ambiguousPropertyTypeAssertion()
+})

compatibility-checks/test-runners/ava/index.ts

@@ -0,0 +1,5 @@
+import { ambiguousPropertyTypeAssertion } from '../_common'
+
+test('can substitute callable interfaces', () => {
+  ambiguousPropertyTypeAssertion()
+})

compatibility-checks/test-runners/jest/index.spec.ts

@@ -0,0 +1,6 @@
+import { test } from 'mocha'
+import { ambiguousPropertyTypeAssertion } from '../_common'
+
+test('can substitute callable interfaces', () => {
+  ambiguousPropertyTypeAssertion()
+})

compatibility-checks/test-runners/mocha/index.ts

@@ -0,0 +1,6 @@
+import * as test from 'node:test'
+import { ambiguousPropertyTypeAssertion } from '../_common'
+
+test.test('can substitute callable interfaces', () => {
+  ambiguousPropertyTypeAssertion()
+})