Refactor specs to use shared package helper

 🐿 v2.10.0

src/sort-packages.js

@@ -1,15 +1,5 @@
 const toposort = require('toposort');
 
-const collateDependencies = (manifest) => {
-	// TODO: refactor into package class
-	return Object.keys({
-		...manifest.dependencies,
-		...manifest.devDependencies,
-		...manifest.peerDependencies,
-		...manifest.optionalDependencies
-	});
-};
-
 module.exports = (reverse = false, packages = []) => {
 	const packageNames = new Set(packages.map((pkg) => pkg.name));
 

test/helpers/create-package.js

@@ -0,0 +1,10 @@
+const Package = require('../../src/package');
+
+module.exports = (name, options = {}) => {
+	const manifest = { name, ...options };
+	const instance = new Package(manifest, `/Path/to/${name}`);
+
+	instance.writeManifest = jest.fn();
+
+	return instance;
+};

test/src/package.spec.js

@@ -67,7 +67,7 @@ describe('src/package', () => {
 			expect(instance.allDependencies).toBeInstanceOf(Array);
 			expect(instance.allDependencies.length).toEqual(4);
 		});
-	})
+	});
 
 	describe('#writeManifest', () => {
 		beforeEach(() => {

test/src/sort-packages.spec.js

@@ -1,39 +1,27 @@
 const subject = require('../../src/sort-packages');
+const createPackage = require('../helpers/create-package');
 
-const fixture = Object.freeze([
-	{
-		name: 'foo',
-		manifest: {
-			dependencies: {
-				qux: '0.0.0'
-			}
+const fixture = [
+	createPackage('foo', {
+		dependencies: {
+			qux: '0.0.0'
 		}
-	},
-	{
-		name: 'bar',
-		manifest: {
-			dependencies: {
-				baz: '0.0.0'
-			}
+	}),
+	createPackage('bar', {
+		dependencies: {
+			baz: '0.0.0'
 		}
-	},
-	{
-		name: 'baz',
-		manifest: {
-			dependencies: {
-				foo: '0.0.0',
-				qux: '0.0.0'
-			}
-		},
-	},
-	{
-		name: 'qux',
-		manifest: {
-			dependencies: {
-			}
+	}),
+	createPackage('baz', {
+		dependencies: {
+			foo: '0.0.0',
+			qux: '0.0.0'
 		}
-	}
-]);
+	}),
+	createPackage('qux', {
+		dependencies: {}
+	})
+];
 
 describe('src/sort-packages', () => {
 	it('returns a new array', () => {

test/src/tasks/exec.spec.js

@@ -2,13 +2,7 @@ const mockRun = jest.fn();
 jest.mock('../../../src/run-package', () => mockRun);
 
 const { task: subject } = require('../../../src/tasks/exec');
-
-const createPackage = (name) => (
-	{
-		name,
-		location: `/Path/to/${name}`
-	}
-);
+const createPackage = require('../../helpers/create-package');
 
 describe('src/tasks/exec', () => {
 	const packages = [

test/src/tasks/publish.spec.js

@@ -2,14 +2,7 @@ const mockRun = jest.fn();
 jest.mock('../../../src/run-package', () => mockRun);
 
 const { task: subject } = require('../../../src/tasks/publish');
-
-const createPackage = (name, options = {}) => (
-	{
-		name,
-		location: `/Path/to/${name}`,
-		...options
-	}
-);
+const createPackage = require('../../helpers/create-package');
 
 describe('src/tasks/publish', () => {
 	const packages = [

test/src/tasks/run.spec.js

@@ -2,20 +2,13 @@ const mockRun = jest.fn();
 jest.mock('../../../src/run-package', () => mockRun);
 
 const { task: subject } = require('../../../src/tasks/run');
-
-const createPackage = (name, options = {}) => (
-	{
-		name,
-		location: `/Path/to/${name}`,
-		...options
-	}
-);
+const createPackage = require('../../helpers/create-package');
 
 describe('src/tasks/run', () => {
 	const packages = [
-		createPackage('foo', { manifest: { scripts: { test: '' } } }),
-		createPackage('bar', { manifest: { scripts: { test: '' } } }),
-		createPackage('baz', { manifest: {} }),
+		createPackage('foo', { scripts: { test: '' } }),
+		createPackage('bar', { scripts: { test: '' } }),
+		createPackage('baz', { scripts: {} }),
 	];
 
 	const command = 'test';

test/src/tasks/script.spec.js

@@ -2,13 +2,7 @@ const mockRun = jest.fn();
 jest.mock('../../../src/run-package', () => mockRun);
 
 const { task: subject } = require('../../../src/tasks/script');
-
-const createPackage = (name) => (
-	{
-		name,
-		location: `/Path/to/${name}`
-	}
-);
+const createPackage = require('../../helpers/create-package');
 
 describe('src/tasks/script', () => {
 	const packages = [

test/src/tasks/version.spec.js

@@ -2,13 +2,7 @@ const mockRun = jest.fn();
 jest.mock('../../../src/run-package', () => mockRun);
 
 const { task: subject } = require('../../../src/tasks/version');
-
-const createPackage = (name) => (
-	{
-		name,
-		location: `/Path/to/${name}`
-	}
-);
+const createPackage = require('../../helpers/create-package');
 
 describe('src/tasks/version', () => {
 	const packages = [