chore: tidy up some tests and remove devcontainer

Author: JamesIves

.devcontainer/Dockerfile

@@ -28,7 +28,6 @@ jest.mock('@actions/io', () => ({
 }))
 
 jest.mock('../src/execute', () => ({
-  // eslint-disable-next-line @typescript-eslint/naming-convention
   __esModule: true,
   execute: jest.fn(() => ({stdout: '', stderr: ''}))
 }))

.devcontainer/devcontainer.json

@@ -3,7 +3,6 @@ import {execute} from '../src/execute'
 import {generateWorktree} from '../src/worktree'
 
 jest.mock('../src/execute', () => ({
-  // eslint-disable-next-line @typescript-eslint/naming-convention
   __esModule: true,
   execute: jest.fn(() => ({stdout: '', stderr: ''}))
 }))

__tests__/git.test.ts

@@ -14,15 +14,16 @@ jest.mock('@actions/core', () => ({
 }))
 
 /*
- Test generateWorktree against a known git repository.
- The upstream repository `origin` is set up once for the test suite,
- and for each test run, a new clone is created.
-
- See workstree.error.test.ts for testing mocked errors from git.*/
+  Test generateWorktree against a known git repository.
+  The upstream repository `origin` is set up once for the test suite,
+  and for each test run, a new clone is created.
 
+  See workstree.error.test.ts for testing mocked errors from git.
+ */
 describe('generateWorktree', () => {
   let tempdir: string | null = null
   let clonedir: string | null = null
+
   beforeAll(async () => {
     // Set up origin repository
     const silent = true
@@ -50,6 +51,7 @@ describe('generateWorktree', () => {
     await execute('git add .', origin, silent)
     await execute('git commit -mgh1', origin, silent)
   })
+
   beforeEach(async () => {
     // Clone origin to our workspace for each test
     const silent = true
@@ -65,17 +67,19 @@ describe('generateWorktree', () => {
     await execute('git fetch --depth=1 origin main', clonedir, silent)
     await execute('git checkout main', clonedir, silent)
   })
+
   afterEach(async () => {
     // Tear down workspace
     await rmRF(clonedir as string)
   })
+
   afterAll(async () => {
     // Tear down origin repository
     if (tempdir) {
       await rmRF(tempdir)
-      // console.log(tempdir)
     }
   })
+
   describe('with existing branch and new commits', () => {
     it('should check out the latest commit', async () => {
       const workspace = clonedir as string
@@ -92,24 +96,29 @@ describe('generateWorktree', () => {
         'worktree',
         true
       )
+
       const dirEntries = await fs.promises.readdir(
         path.join(workspace, 'worktree')
       )
+
       expect(dirEntries.sort((a, b) => a.localeCompare(b))).toEqual([
         '.git',
         'gh1'
       ])
+
       const commitMessages = await execute(
         'git log --format=%s',
         path.join(workspace, 'worktree'),
         true
       )
+
       expect(commitMessages.stdout).toBe('gh1')
     })
   })
   describe('with missing branch and new commits', () => {
     it('should create initial commit', async () => {
       const workspace = clonedir as string
+
       await generateWorktree(
         {
           hostname: 'github.com',
@@ -123,21 +132,26 @@ describe('generateWorktree', () => {
         'worktree',
         false
       )
+
       const dirEntries = await fs.promises.readdir(
         path.join(workspace, 'worktree')
       )
+
       expect(dirEntries).toEqual(['.git'])
+
       const commitMessages = await execute(
         'git log --format=%s',
         path.join(workspace, 'worktree'),
         true
       )
+
       expect(commitMessages.stdout).toBe('Initial no-pages commit')
     })
   })
   describe('with existing branch and singleCommit', () => {
     it('should check out the latest commit', async () => {
       const workspace = clonedir as string
+
       await generateWorktree(
         {
           hostname: 'github.com',
@@ -151,13 +165,16 @@ describe('generateWorktree', () => {
         'worktree',
         true
       )
+
       const dirEntries = await fs.promises.readdir(
         path.join(workspace, 'worktree')
       )
+
       expect(dirEntries.sort((a, b) => a.localeCompare(b))).toEqual([
         '.git',
         'gh1'
       ])
+
       expect(async () => {
         await execute(
           'git log --format=%s',
@@ -170,6 +187,7 @@ describe('generateWorktree', () => {
   describe('with missing branch and singleCommit', () => {
     it('should create initial commit', async () => {
       const workspace = clonedir as string
+
       await generateWorktree(
         {
           hostname: 'github.com',
@@ -183,10 +201,13 @@ describe('generateWorktree', () => {
         'worktree',
         false
       )
+
       const dirEntries = await fs.promises.readdir(
         path.join(workspace, 'worktree')
       )
+
       expect(dirEntries).toEqual(['.git'])
+
       expect(async () => {
         await execute(
           'git log --format=%s',