Refactor: simplify core (#243)

* refactor core types

* refactor substitute core

* refactor recorder and recordsSet

* fix tests due to refactor

* fix types for typescript < 4.6

* Compatibility with strict mode (#244)

* stricter types on recorded arguments

* make types more verbose and explicit

* fix types verbosity on tests

* remove strictNullChecks flag

* remove strict mode warning from readme

* 2.0.0-beta.2

* Maintenance upgrades, guarantee support for node.js 18 & drop node.js 10 (#245)

* better tsconfig setup

* upgrade dependencies to latest

* fix ava types

* upgrade github actions

* support up to node.js 18

* drop node.js 10 from test matrix

* use ava config file

Author: notanengineercom

.github/workflows/codeql-analysis.yml

@@ -22,7 +22,7 @@ jobs:
 
     steps:
     - name: Checkout repository
-      uses: actions/checkout@v2
+      uses: actions/checkout@v3
       with:
         # We must fetch at least the immediate parents so that if this is
         # a pull request then we can checkout the head.
@@ -35,9 +35,9 @@ jobs:
 
     # Initializes the CodeQL tools for scanning.
     - name: Initialize CodeQL
-      uses: github/codeql-action/init@v1
+      uses: github/codeql-action/init@v2
       with:
         languages: ${{ matrix.language }}
 
     - name: Perform CodeQL Analysis
-      uses: github/codeql-action/analyze@v1
+      uses: github/codeql-action/analyze@v2

.github/workflows/nodejs.yml

@@ -1,26 +1,25 @@
 name: Node CI
-
 on: [push]
 
 jobs:
   build:
-
     runs-on: ubuntu-latest
-
     strategy:
       matrix:
-        node-version: ['10.x', '12.x', '14.x']
+        node-version: [12, 14, 16, 18]
 
     steps:
-    - uses: actions/checkout@v2
+    - uses: actions/checkout@v3
     - name: Use Node.js ${{ matrix.node-version }}
-      uses: actions/setup-node@v2-beta
+      uses: actions/setup-node@v3
       with:
         node-version: ${{ matrix.node-version }}
-    - name: install dependencies, build and test
-      run: |
-        npm ci
-        npm run build --if-present
-        npm test
+        cache: 'npm'
+    - name: Install dependencies
+      run: npm ci
+    - name: Transpile typescript
+      run: npm run build --if-present
+    - name: Run tests
+      run: npm test
       env:
         CI: true

.github/workflows/npm-publish.yml

@@ -5,25 +5,27 @@ on:
     types: [created]
 
 jobs:
-  build:
+  build-test:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v2
-      - uses: actions/setup-node@v2
+      - uses: actions/checkout@v3
+      - uses: actions/setup-node@v3
         with:
           node-version: 14
+          cache: 'npm'
       - run: npm ci --ignore-scripts
       - run: npm test
 
   publish-npm:
-    needs: build
+    needs: build-test
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v2
-      - uses: actions/setup-node@v2
+      - uses: actions/checkout@v3
+      - uses: actions/setup-node@v3
         with:
           node-version: 14
           registry-url: https://registry.npmjs.org/
+          cache: 'npm'
       - run: npm ci --ignore-scripts
       - run: npm run build
       - run: echo "tag=latest" >> $GITHUB_ENV
@@ -35,17 +37,18 @@ jobs:
           NODE_AUTH_TOKEN: ${{ secrets.NPM_PUBLISH_TOKEN }}
 
   publish-github:
-    needs: build
+    needs: build-test
     runs-on: ubuntu-latest
     permissions:
       contents: read
       packages: write
     steps:
-      - uses: actions/checkout@v2
-      - uses: actions/setup-node@v2
+      - uses: actions/checkout@v3
+      - uses: actions/setup-node@v3
         with:
           node-version: 14
           registry-url: https://npm.pkg.github.com/
+          cache: 'npm'
       - run: npm ci --ignore-scripts
       - run: npm run build
       - run: echo "tag=latest" >> $GITHUB_ENV

README.md

@@ -226,20 +226,6 @@ Substitute.disableFor(fake).received(1337);
 fake.received().received(1337);
 ```
 
-## Strict mode
-If you have `strict` set to `true` in your `tsconfig.json`, you may need to toggle off strict null checks. The framework does not currently support this.
-
-However, it is only needed for your test projects anyway.
-
-```json
-{
-    "compilerOptions": {
-        "strict": true,
-        "strictNullChecks": false
-    }
-}
-```
-
 ## Contributors
 
 ### Code Contributors

ava.config.js

@@ -0,0 +1,13 @@
+
+module.exports = {
+	files: ['spec/**/*.ts'],
+	typescript: {
+		compile: false,
+		rewritePaths: {
+			'/': 'dist/'
+		}
+	},
+  cache: false,
+  failFast: true,
+  failWithoutAssertions: true
+}