Share test utilities (#843)

Split test fixtures into `@simple-git/test-utils`, depend on the package directly in integration tests and share through unit tests fixtures
Remove outer fixtures, reuse `@simple-git/test-utils` in unit test fixtures

Author: steveukx

.changeset/heavy-pumpkins-work.md

@@ -0,0 +1,9 @@
+export * from './src/create-test-context';
+export * from './src/expectations';
+export * from './src/instance';
+export * from './src/like';
+export * from './src/wait';
+
+export * from './src/setup/setup-conflicted';
+export * from './src/setup/setup-files';
+export * from './src/setup/setup-init';

packages/test-utils/index.ts

@@ -0,0 +1,8 @@
+{
+  "name": "@simple-git/test-utils",
+  "version": "0.0.0",
+  "private": true,
+  "peerDependencies": {
+    "simple-git": "^3.12.0"
+  }
+}

packages/test-utils/package.json

@@ -1,7 +1,6 @@
 import { join } from 'path';
 import { existsSync, mkdir, mkdtemp, realpathSync, writeFile, WriteFileOptions } from 'fs';
-import { SimpleGit } from '../../typings';
-import { newSimpleGit } from './instance';
+import { simpleGit, SimpleGit } from 'simple-git';
 
 export interface SimpleGitTestContext {
    /** Creates a directory under the repo root at the given path(s) */
@@ -85,7 +84,7 @@ export async function createTestContext(): Promise<SimpleGitTestContext> {
          return realpathSync(context.root);
       },
       get git() {
-         return newSimpleGit(root);
+         return simpleGit(root);
       },
    };
 

simple-git/test/__fixtures__/create-test-context.ts renamed to packages/test-utils/src/create-test-context.ts

@@ -1,5 +1,4 @@
-import { GitError } from '../../src/lib/errors/git-error';
-import { GitResponseError } from '../../src/lib/errors/git-response-error';
+import { GitError, GitResponseError } from "simple-git";
 
 /**
  * Convenience for asserting the type and message of a `GitError`
@@ -15,12 +14,8 @@ import { GitResponseError } from '../../src/lib/errors/git-response-error';
 export function assertGitError(
    errorInstance: Error | unknown,
    message: string | RegExp,
-   errorConstructor?: any
+   errorConstructor: any = GitError
 ) {
-   if (!errorConstructor) {
-      errorConstructor = GitError;
-   }
-
    expect(errorInstance).toBeInstanceOf(errorConstructor);
    expect(errorInstance).toHaveProperty('message', expect.stringMatching(message));
 }

simple-git/test/__fixtures__/expectations.ts renamed to packages/test-utils/src/expectations.ts

@@ -0,0 +1,3 @@
+import { simpleGit as newSimpleGit } from 'simple-git';
+
+export { newSimpleGit };

packages/test-utils/src/instance.ts

@@ -1,4 +1,4 @@
-import { SimpleGitTestContext } from './create-test-context';
+import { SimpleGitTestContext } from '../create-test-context';
 
 export const FIRST_BRANCH = 'first';
 export const SECOND_BRANCH = 'second';

simple-git/test/__fixtures__/like.ts renamed to packages/test-utils/src/like.ts

@@ -1,4 +1,4 @@
-import { SimpleGitTestContext } from './create-test-context';
+import { SimpleGitTestContext } from '../create-test-context';
 
 export async function setUpFilesAdded(
    { git, files }: SimpleGitTestContext,

simple-git/test/__fixtures__/setup-conflicted.ts renamed to packages/test-utils/src/setup/setup-conflicted.ts

@@ -1,5 +1,5 @@
-import { SimpleGit } from '../../typings';
-import { SimpleGitTestContext } from './create-test-context';
+import { SimpleGit } from 'simple-git';
+import { SimpleGitTestContext } from '../create-test-context';
 
 export const GIT_USER_NAME = 'Simple Git Tests';
 export const GIT_USER_EMAIL = '[email protected]';