@neurodevs/meta-node 0.14.29 → 0.16.0

package/src/__tests__/impl/GitAutocommit.test.ts

@@ -0,0 +1,60 @@
+import { exec as execSync } from 'node:child_process'
+import { promisify } from 'node:util'
+
+import {
+    callsToExec,
+    fakeExec,
+    resetCallsToExec,
+} from '@neurodevs/fake-node-core'
+import AbstractModuleTest, { test, assert } from '@neurodevs/node-tdd'
+
+import GitAutocommit, { Autocommit } from '../../impl/GitAutocommit.js'
+
+const exec = promisify(execSync)
+
+export default class GitAutocommitTest extends AbstractModuleTest {
+    private static instance: Autocommit
+
+    private static readonly commitMessage = this.generateId()
+    private static readonly currentWorkingDir = this.generateId()
+
+    protected static async beforeEach() {
+        await super.beforeEach()
+
+        this.setFakeExec()
+
+        this.instance = await this.GitAutocommit()
+    }
+
+    @test()
+    protected static async createsInstance() {
+        assert.isTruthy(this.instance, 'Failed to create instance!')
+    }
+
+    @test()
+    protected static async callsExecThreeTimes() {
+        assert.isEqualDeep(callsToExec, [
+            { command: 'git add .', options: { cwd: this.currentWorkingDir } },
+            {
+                command: `git commit -m "${this.commitMessage}"`,
+                options: { cwd: this.currentWorkingDir },
+            },
+            { command: 'git push', options: { cwd: this.currentWorkingDir } },
+        ])
+    }
+
+    private static setFakeExec() {
+        GitAutocommit.exec = fakeExec as unknown as typeof exec
+        resetCallsToExec()
+    }
+
+    private static async GitAutocommit(
+        commitMessage?: string,
+        workingDir?: string
+    ) {
+        return GitAutocommit.Create(
+            commitMessage ?? this.commitMessage,
+            workingDir ?? this.currentWorkingDir
+        )
+    }
+}

package/src/__tests__/impl/ImplAutomodule.test.ts

@@ -121,7 +121,7 @@ export default class ImplAutomoduleTest extends AbstractAutomoduleTest {
         await this.run()
 
         assert.isEqualDeep(
-            callsToExec[0],
+            callsToExec[0]?.command,
             'yarn version --minor --no-git-tag-version',
             'Did not bump minor version!'
         )

package/src/__tests__/impl/NpmAutopackage.test.ts

@@ -1,7 +1,6 @@
 import { ChildProcess, exec as execSync } from 'child_process'
 import { readFile, writeFile } from 'fs/promises'
 import { mkdir } from 'fs/promises'
-import path from 'path'
 import { promisify } from 'util'
 import {
     callsToChdir,
@@ -31,10 +30,12 @@ import {
 } from '@neurodevs/fake-node-core'
 import { test, assert } from '@neurodevs/node-tdd'
 
+import GitAutocommit from '../../impl/GitAutocommit.js'
 import NpmAutopackage, {
     Autopackage,
     AutopackageOptions,
 } from '../../impl/NpmAutopackage.js'
+import FakeAutocommit from '../../testDoubles/Autocommit/FakeAutocommit.js'
 import AbstractPackageTest from '../AbstractPackageTest.js'
 
 const exec = promisify(execSync)
@@ -53,6 +54,8 @@ export default class NpmAutopackageTest extends AbstractPackageTest {
         this.fakeReadFile()
         this.fakeWriteFile()
 
+        this.setFakeAutocommit()
+
         this.fakeFetchForRepoNotFound()
 
         process.env.GITHUB_TOKEN = this.githubToken
@@ -127,7 +130,7 @@ export default class NpmAutopackageTest extends AbstractPackageTest {
         await this.run()
 
         assert.isEqual(
-            callsToExec[0],
+            callsToExec[0]?.command,
             `git clone https://github.com/${this.gitNamespace}/${this.name}.git`,
             'Did not call git clone!'
         )
@@ -149,7 +152,7 @@ export default class NpmAutopackageTest extends AbstractPackageTest {
         await this.run()
 
         assert.isEqual(
-            callsToExec[1],
+            callsToExec[1]?.command,
             `git fetch origin`,
             'Did not call git fetch origin!'
         )
@@ -160,7 +163,7 @@ export default class NpmAutopackageTest extends AbstractPackageTest {
         await this.run()
 
         assert.isEqual(
-            callsToExec[2],
+            callsToExec[2]?.command,
             `git reset --hard origin/main`,
             'Did not call git reset hard!'
         )
@@ -171,7 +174,7 @@ export default class NpmAutopackageTest extends AbstractPackageTest {
         await this.run()
 
         assert.isEqual(
-            callsToExec[4],
+            callsToExec[4]?.command,
             this.createModuleCmd,
             'Did not call "spruce create.module"!'
         )
@@ -182,12 +185,8 @@ export default class NpmAutopackageTest extends AbstractPackageTest {
         await this.run()
 
         assert.isEqualDeep(
-            callsToExec.slice(5, 8),
-            [
-                'git add .',
-                `git commit -m "patch: create package (@neurodevs/meta-node: ${this.metaNodeVersion})"`,
-                'git push',
-            ],
+            FakeAutocommit.callsToConstructor[0]?.commitMessage,
+            `patch: create package (@neurodevs/meta-node: ${this.metaNodeVersion})`,
             'Did not commit create package changes!'
         )
     }
@@ -255,12 +254,8 @@ export default class NpmAutopackageTest extends AbstractPackageTest {
         await this.run()
 
         assert.isEqualDeep(
-            callsToExec.slice(8, 11),
-            [
-                'git add .',
-                `git commit -m "patch: update package.json (@neurodevs/meta-node: ${this.metaNodeVersion})"`,
-                'git push',
-            ],
+            FakeAutocommit.callsToConstructor[1]?.commitMessage,
+            `patch: update package.json (@neurodevs/meta-node: ${this.metaNodeVersion})`,
             'Did not commit update package changes!'
         )
     }
@@ -285,12 +280,8 @@ export default class NpmAutopackageTest extends AbstractPackageTest {
         await this.run()
 
         assert.isEqualDeep(
-            callsToExec.slice(11, 14),
-            [
-                'git add .',
-                `git commit -m "patch: add build dir to gitignore (@neurodevs/meta-node: ${this.metaNodeVersion})"`,
-                'git push',
-            ],
+            FakeAutocommit.callsToConstructor[2]?.commitMessage,
+            `patch: add build dir to gitignore (@neurodevs/meta-node: ${this.metaNodeVersion})`,
             'Did not commit .gitignore changes!'
         )
     }
@@ -300,7 +291,7 @@ export default class NpmAutopackageTest extends AbstractPackageTest {
         await this.run()
 
         assert.isEqual(
-            callsToExec[14],
+            callsToExec[5]?.command,
             this.setupVscodeCmd,
             'Did not call "spruce setup.vscode"!'
         )
@@ -311,12 +302,8 @@ export default class NpmAutopackageTest extends AbstractPackageTest {
         await this.run()
 
         assert.isEqualDeep(
-            callsToExec.slice(15, 18),
-            [
-                'git add .',
-                `git commit -m "patch: setup vscode (@neurodevs/meta-node: ${this.metaNodeVersion})"`,
-                'git push',
-            ],
+            FakeAutocommit.callsToConstructor[3]?.commitMessage,
+            `patch: setup vscode (@neurodevs/meta-node: ${this.metaNodeVersion})`,
             'Did not commit vscode changes!'
         )
     }
@@ -337,12 +324,8 @@ export default class NpmAutopackageTest extends AbstractPackageTest {
         await this.run()
 
         assert.isEqualDeep(
-            callsToExec.slice(18, 21),
-            [
-                'git add .',
-                `git commit -m "patch: update vscode tasks.json (@neurodevs/meta-node: ${this.metaNodeVersion})"`,
-                'git push',
-            ],
+            FakeAutocommit.callsToConstructor[4]?.commitMessage,
+            `patch: update vscode tasks.json (@neurodevs/meta-node: ${this.metaNodeVersion})`,
             'Did not commit updated vscode tasks.json changes!'
         )
     }
@@ -352,7 +335,7 @@ export default class NpmAutopackageTest extends AbstractPackageTest {
         await this.run()
 
         assert.isEqual(
-            callsToExec[23],
+            callsToExec[8]?.command,
             this.yarnInstallDevDepsCommand,
             'Did not install default devDependencies!'
         )
@@ -363,12 +346,8 @@ export default class NpmAutopackageTest extends AbstractPackageTest {
         await this.run()
 
         assert.isEqualDeep(
-            callsToExec.slice(24, 27),
-            [
-                'git add .',
-                `git commit -m "patch: install default devDependencies (@neurodevs/meta-node: ${this.metaNodeVersion})"`,
-                'git push',
-            ],
+            FakeAutocommit.callsToConstructor[5]?.commitMessage,
+            `patch: install default devDependencies (@neurodevs/meta-node: ${this.metaNodeVersion})`,
             'Did not commit install devDependencies changes!'
         )
     }
@@ -407,12 +386,8 @@ export default class NpmAutopackageTest extends AbstractPackageTest {
         await this.run()
 
         assert.isEqualDeep(
-            callsToExec.slice(27, 30),
-            [
-                'git add .',
-                `git commit -m "patch: install AbstractPackageTest (@neurodevs/meta-node: ${this.metaNodeVersion})"`,
-                'git push',
-            ],
+            FakeAutocommit.callsToConstructor[6]?.commitMessage,
+            `patch: install AbstractPackageTest (@neurodevs/meta-node: ${this.metaNodeVersion})`,
             'Did not commit install AbstractPackageTest changes!'
         )
     }
@@ -421,7 +396,11 @@ export default class NpmAutopackageTest extends AbstractPackageTest {
     protected static async lastlyOpensVscodeAtEnd() {
         await this.run()
 
-        assert.isEqual(callsToExec[30], 'code .', 'Did not open vscode at end!')
+        assert.isEqual(
+            callsToExec[9]?.command,
+            'code .',
+            'Did not open vscode at end!'
+        )
     }
 
     @test()
@@ -448,8 +427,8 @@ export default class NpmAutopackageTest extends AbstractPackageTest {
 
         assert.isEqual(
             callsToExec.filter(
-                (cmd) =>
-                    cmd ===
+                (call) =>
+                    call?.command ===
                     `git clone https://github.com/${this.gitNamespace}/${this.name}.git`
             ).length,
             1,
@@ -462,7 +441,8 @@ export default class NpmAutopackageTest extends AbstractPackageTest {
         await this.createAndRunAutopackage()
 
         assert.isEqual(
-            callsToExec.filter((cmd) => cmd === this.createModuleCmd).length,
+            callsToExec.filter((call) => call?.command === this.createModuleCmd)
+                .length,
             1,
             'Did not call spruce create.module once!'
         )
@@ -473,7 +453,8 @@ export default class NpmAutopackageTest extends AbstractPackageTest {
         await this.createAndRunAutopackage()
 
         assert.isEqual(
-            callsToExec.filter((cmd) => cmd === this.setupVscodeCmd).length,
+            callsToExec.filter((call) => call?.command === this.setupVscodeCmd)
+                .length,
             1,
             'Did not call spruce setup.vscode once!'
         )
@@ -484,10 +465,10 @@ export default class NpmAutopackageTest extends AbstractPackageTest {
         await this.createAndRunAutopackage()
 
         assert.isEqual(
-            callsToExec.filter(
-                (cmd) =>
-                    cmd ===
-                    `git commit -m "patch: create package (@neurodevs/meta-node: ${this.metaNodeVersion})"`
+            FakeAutocommit.callsToConstructor.filter(
+                (call) =>
+                    call?.commitMessage ===
+                    `patch: create package (@neurodevs/meta-node: ${this.metaNodeVersion})`
             ).length,
             1,
             'Did not commit create package changes once!'
@@ -499,10 +480,10 @@ export default class NpmAutopackageTest extends AbstractPackageTest {
         await this.createAndRunAutopackage()
 
         assert.isEqual(
-            callsToExec.filter(
-                (cmd) =>
-                    cmd ===
-                    `git commit -m "patch: update package.json (@neurodevs/meta-node: ${this.metaNodeVersion})"`
+            FakeAutocommit.callsToConstructor.filter(
+                (call) =>
+                    call?.commitMessage ===
+                    `patch: update package.json (@neurodevs/meta-node: ${this.metaNodeVersion})`
             ).length,
             1,
             'Did not commit update package changes once!'
@@ -514,10 +495,10 @@ export default class NpmAutopackageTest extends AbstractPackageTest {
         await this.createAndRunAutopackage()
 
         assert.isEqual(
-            callsToExec.filter(
-                (cmd) =>
-                    cmd ===
-                    `git commit -m "patch: add build dir to gitignore (@neurodevs/meta-node: ${this.metaNodeVersion})"`
+            FakeAutocommit.callsToConstructor.filter(
+                (call) =>
+                    call?.commitMessage ===
+                    `patch: add build dir to gitignore (@neurodevs/meta-node: ${this.metaNodeVersion})`
             ).length,
             1,
             'Did not commit gitignore changes once!'
@@ -529,10 +510,10 @@ export default class NpmAutopackageTest extends AbstractPackageTest {
         await this.createAndRunAutopackage()
 
         assert.isEqual(
-            callsToExec.filter(
-                (cmd) =>
-                    cmd ===
-                    `git commit -m "patch: setup vscode (@neurodevs/meta-node: ${this.metaNodeVersion})"`
+            FakeAutocommit.callsToConstructor.filter(
+                (call) =>
+                    call?.commitMessage ===
+                    `patch: setup vscode (@neurodevs/meta-node: ${this.metaNodeVersion})`
             ).length,
             1,
             'Did not commit vscode changes once!'
@@ -571,7 +552,7 @@ export default class NpmAutopackageTest extends AbstractPackageTest {
         await this.createAndRunAutopackage()
 
         assert.isFalse(
-            callsToExec.includes('code .'),
+            callsToExec.some((call) => call?.command === 'code .'),
             'Should not open vscode if not cloned!'
         )
     }
@@ -585,7 +566,7 @@ export default class NpmAutopackageTest extends AbstractPackageTest {
         await this.createAndRunAutopackage()
 
         const calls = callsToExec.filter(
-            (cmd) => cmd === this.yarnInstallDevDepsCommand
+            (call) => call?.command === this.yarnInstallDevDepsCommand
         )
 
         assert.isEqual(
@@ -608,7 +589,7 @@ export default class NpmAutopackageTest extends AbstractPackageTest {
         await this.createAndRunAutopackage()
 
         const calls = callsToExec.filter(
-            (cmd) => cmd === this.yarnInstallDevDepsCommand
+            (call) => call?.command === this.yarnInstallDevDepsCommand
         )
 
         assert.isEqual(
@@ -709,9 +690,6 @@ export default class NpmAutopackageTest extends AbstractPackageTest {
     private static readonly checkGenerateIdVersionCmd = `yarn info @neurodevs/generate-id version --silent`
     private static readonly checkNodeTddVersionCmd = `yarn info @neurodevs/node-tdd version --silent`
 
-    private static readonly yarnGlobalDirCmd = 'yarn global dir'
-    private static readonly fakeGlobalRoot = this.generateId()
-
     private static orderJsonKeys(
         json: Record<string, unknown>,
         keyOrder: string[]
@@ -744,20 +722,7 @@ export default class NpmAutopackageTest extends AbstractPackageTest {
         NpmAutopackage.exec = fakeExec as unknown as typeof exec
         resetCallsToExec()
 
-        setFakeExecResult(this.yarnGlobalDirCmd, {
-            stdout: this.fakeGlobalRoot,
-        } as unknown as ChildProcess)
-
-        setFakeReadFileResult(
-            path.join(
-                this.fakeGlobalRoot,
-                'node_modules',
-                '@neurodevs',
-                'meta-node',
-                'package.json'
-            ),
-            JSON.stringify({ version: this.metaNodeVersion })
-        )
+        this.setFakeMetaNodeVersion()
     }
 
     private static fakeFetch() {
@@ -794,6 +759,11 @@ export default class NpmAutopackageTest extends AbstractPackageTest {
         resetCallsToWriteFile()
     }
 
+    private static setFakeAutocommit() {
+        GitAutocommit.Class = FakeAutocommit
+        FakeAutocommit.resetTestDouble()
+    }
+
     private static fakeFetchForRepoNotFound() {
         const fakeResponse = new Response(null, {
             status: 404,
@@ -813,7 +783,6 @@ export default class NpmAutopackageTest extends AbstractPackageTest {
     private static readonly author = this.generateId()
 
     private static readonly githubToken = this.generateId()
-    private static readonly metaNodeVersion = this.generateId()
     private static readonly randomId = this.generateId()
 
     private static get reposUrl() {

package/src/__tests__/impl/NpmPropagationCoordinator.test.ts

@@ -0,0 +1,93 @@
+import { readFile } from 'node:fs/promises'
+
+import {
+    fakeReadFile,
+    resetCallsToReadFile,
+    setFakeReadFileResult,
+} from '@neurodevs/fake-node-core'
+import AbstractModuleTest, { test, assert } from '@neurodevs/node-tdd'
+
+import NpmPropagationCoordinator, {
+    PropagationCoordinator,
+} from '../../impl/NpmPropagationCoordinator.js'
+import NpmReleasePropagator from '../../impl/NpmReleasePropagator.js'
+import FakeReleasePropagator from '../../testDoubles/ReleasePropagator/FakeReleasePropagator.js'
+
+export default class NpmPropagationCoordinatorTest extends AbstractModuleTest {
+    private static instance: PropagationCoordinator
+
+    private static readonly packageScope = this.generateId()
+    private static readonly packageName = this.generateId()
+    private static readonly scopedName = `${this.packageScope}/${this.packageName}`
+    private static readonly packageVersion = '7.8.9'
+    private static readonly repoPath = `${this.generateId()}/${this.packageName}`
+
+    private static readonly repoPaths = [
+        this.generateId(),
+        this.generateId(),
+        this.generateId(),
+        this.repoPath,
+    ]
+
+    private static readonly pkgJsons = [
+        { dependencies: { [this.repoPath]: '^1.2.3' } },
+        { devDependencies: { [this.repoPath]: '^4.5.6' } },
+        {},
+        { name: this.scopedName, version: this.packageVersion },
+    ]
+
+    protected static async beforeEach() {
+        await super.beforeEach()
+
+        this.setFakeReadFile()
+        this.setFakeReleasePropagator()
+
+        this.instance = this.NpmPropagationCoordinator()
+    }
+
+    @test()
+    protected static async createsInstance() {
+        assert.isTruthy(this.instance, 'Failed to create instance!')
+    }
+
+    @test()
+    protected static async propagatesToCorrectRepoPaths() {
+        await this.run()
+
+        assert.isEqualDeep(
+            FakeReleasePropagator.callsToConstructor[0],
+            {
+                packageName: this.scopedName,
+                packageVersion: this.packageVersion,
+                repoPaths: [this.repoPaths[0], this.repoPaths[1]],
+            },
+            'Incorrect repo paths propagated to!'
+        )
+    }
+
+    private static async run() {
+        await this.instance.run()
+    }
+
+    private static setFakeReadFile() {
+        NpmPropagationCoordinator.readFile =
+            fakeReadFile as unknown as typeof readFile
+        resetCallsToReadFile()
+
+        this.repoPaths.forEach((repoPath, i) => {
+            setFakeReadFileResult(
+                `${repoPath}/package.json`,
+                JSON.stringify(this.pkgJsons[i], null, 4)
+            )
+        })
+    }
+
+    private static setFakeReleasePropagator() {
+        NpmReleasePropagator.Class = FakeReleasePropagator
+        FakeReleasePropagator.resetTestDouble()
+    }
+
+    private static NpmPropagationCoordinator() {
+        return NpmPropagationCoordinator.Create(this.repoPath, this.repoPaths)
+    }
+}

package/src/__tests__/impl/NpmReleasePropagator.test.ts

@@ -0,0 +1,151 @@
+import { exec as execSync } from 'node:child_process'
+import { readFile } from 'node:fs/promises'
+import { promisify } from 'node:util'
+import {
+    callsToExec,
+    fakeExec,
+    fakeReadFile,
+    resetCallsToExec,
+    resetCallsToReadFile,
+    setFakeReadFileResult,
+} from '@neurodevs/fake-node-core'
+import { test, assert } from '@neurodevs/node-tdd'
+
+import GitAutocommit from '../../impl/GitAutocommit.js'
+import NpmReleasePropagator, {
+    ReleasePropagator,
+    ReleasePropagatorOptions,
+} from '../../impl/NpmReleasePropagator.js'
+import { FakeAutocommit } from '../../index.js'
+import AbstractPackageTest from '../AbstractPackageTest.js'
+
+const exec = promisify(execSync)
+
+export default class NpmReleasePropagatorTest extends AbstractPackageTest {
+    private static instance: ReleasePropagator
+
+    private static readonly packageName = this.generateId()
+    private static readonly packageVersion = this.generateId()
+    private static readonly repoPaths = [this.generateId(), this.generateId()]
+
+    protected static async beforeEach() {
+        await super.beforeEach()
+
+        this.setFakeExec()
+        this.setFakeReadFile()
+        this.setFakeAutocommit()
+
+        this.instance = this.NpmReleasePropagator()
+    }
+
+    @test()
+    protected static async createsInstance() {
+        assert.isTruthy(this.instance, 'Failed to create instance!')
+    }
+
+    @test()
+    protected static async runInstallsReleaseForEachRepoPath() {
+        await this.run()
+
+        const expectedCalls = [
+            {
+                command: `yarn add ${this.packageName}@${this.packageVersion}`,
+                options: { cwd: this.repoPaths[0] },
+            },
+            {
+                command: `yarn add -D ${this.packageName}@${this.packageVersion}`,
+                options: { cwd: this.repoPaths[1] },
+            },
+        ]
+
+        assert.isEqualDeep(
+            [callsToExec[0], callsToExec[2]],
+            expectedCalls,
+            'Did not install release in each repo path!'
+        )
+    }
+
+    @test()
+    protected static async runThrowsIfRepoDoesNotHavePreviousRelease() {
+        const missingPackageName = this.generateId()
+
+        this.instance = this.NpmReleasePropagator({
+            packageName: missingPackageName,
+        })
+
+        await assert.doesThrowAsync(async () => {
+            await this.run()
+        }, `Cannot propagate release for ${missingPackageName} because it is not listed in either dependencies or devDependencies! Please install it in the target repository before running propagation.`)
+    }
+
+    @test()
+    protected static async commitsChangesToGit() {
+        await this.run()
+
+        assert.isEqualDeep(
+            FakeAutocommit.callsToConstructor,
+            [
+                {
+                    commitMessage: `patch: propagate ${this.packageName}@${this.packageVersion} (@neurodevs/meta-node: ${this.metaNodeVersion})`,
+                    cwd: this.repoPaths[0],
+                },
+                {
+                    commitMessage: `patch: propagate ${this.packageName}@${this.packageVersion} (@neurodevs/meta-node: ${this.metaNodeVersion})`,
+                    cwd: this.repoPaths[1],
+                },
+            ],
+            'Did not commit changes to git for each repo path!'
+        )
+    }
+
+    private static async run() {
+        await this.instance.run()
+    }
+
+    private static setFakeExec() {
+        NpmReleasePropagator.exec = fakeExec as unknown as typeof exec
+        resetCallsToExec()
+
+        this.setFakeMetaNodeVersion()
+    }
+
+    private static setFakeReadFile() {
+        NpmReleasePropagator.readFile =
+            fakeReadFile as unknown as typeof readFile
+        resetCallsToReadFile()
+
+        setFakeReadFileResult(
+            `${this.repoPaths[0]}/package.json`,
+            JSON.stringify({
+                dependencies: {
+                    [this.packageName]: this.generateId(),
+                },
+            })
+        )
+
+        setFakeReadFileResult(
+            `${this.repoPaths[1]}/package.json`,
+            JSON.stringify({
+                devDependencies: {
+                    [this.packageName]: this.generateId(),
+                },
+            })
+        )
+    }
+
+    private static setFakeAutocommit() {
+        GitAutocommit.Class = FakeAutocommit
+        FakeAutocommit.resetTestDouble()
+    }
+
+    private static NpmReleasePropagator(
+        options?: Partial<ReleasePropagatorOptions>
+    ) {
+        return NpmReleasePropagator.Create({
+            packageName: this.packageName,
+            packageVersion: this.packageVersion,
+            repoPaths: this.repoPaths,
+            ...options,
+        })
+    }
+}

package/src/__tests__/impl/UiAutomodule.test.ts

@@ -119,7 +119,7 @@ export default class UiAutomoduleTest extends AbstractAutomoduleTest {
         await this.run()
 
         assert.isEqualDeep(
-            callsToExec[0],
+            callsToExec[0]?.command,
             'yarn version --minor --no-git-tag-version',
             'Did not bump minor version!'
         )