semantic-release-lerna 0.8.0 → 0.8.2

package/lib/should-latch.js

@@ -1,7 +1,7 @@
 /* eslint-disable security/detect-unsafe-regex */
 const latchMajor = /^\d+\.0\.0$/;
 const latchMinor = /^\d+\.\d+\.0$/;
-const latchPatch = /^\d+\.\d\.\d+$/;
+const latchPatch = /^\d+\.\d+\.\d+$/;
 const latchPrerelease = /^\d+\.\d+\.\d+(-(.*\.)?\d+)?$/;
 
 /**

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "semantic-release-lerna",
-  "version": "0.8.0",
+  "version": "0.8.2",
   "description": "semantic-release plugin to publish lerna monorepo packages to npm",
   "keywords": [
     "npm",
@@ -29,7 +29,7 @@
   "files": [
     "lib",
     "index.js",
-    "!*.test.js"
+    "!**/*.test.js"
   ],
   "scripts": {
     "codecov": "codecov -f coverage/coverage-final.json",
@@ -46,7 +46,7 @@
     "testTimeout": 10000
   },
   "dependencies": {
-    "@lerna/child-process": "^6.0.0",
+    "@lerna/child-process": "^7.0.0",
     "@lerna/collect-updates": "^6.0.0",
     "@lerna/package": "^6.0.0",
     "@lerna/package-graph": "^6.0.0",
@@ -68,28 +68,29 @@
     "write-json-file": "^4.0.0"
   },
   "devDependencies": {
-    "@html-validate/eslint-config": "5.5.22",
+    "@html-validate/eslint-config": "5.5.27",
     "@html-validate/eslint-config-jest": "5.5.18",
-    "@html-validate/prettier-config": "2.3.4",
+    "@html-validate/prettier-config": "2.3.6",
     "@semantic-release/npm": "9.0.2",
-    "@types/jest": "29.4.0",
+    "@types/jest": "29.4.1",
     "codecov": "3.8.3",
     "fs-extra": "11.1.0",
     "got": "11.8.6",
-    "jest": "29.4.2",
-    "lerna": "6.4.1",
-    "prettier": "2.8.4",
-    "semantic-release": "20.1.0",
+    "jest": "29.5.0",
+    "lerna": "7.1.4",
+    "npm": "9.8.1",
+    "npm-pkg-lint": "1.14.1",
+    "semantic-release": "20.1.1",
     "stream-buffers": "3.0.2",
     "verdaccio": "5.19.1"
   },
   "peerDependencies": {
     "@semantic-release/npm": ">= 7",
-    "lerna": "^3.2 || ^4 || ^5 || ^6",
+    "lerna": "^3.2 || ^4 || ^5 || ^6 || ^7",
     "semantic-release": ">=15.0.0 <21.0.0"
   },
   "engines": {
-    "node": ">= 14"
+    "node": ">= 14.17"
   },
   "publishConfig": {
     "access": "public"

package/lib/prepare.test.js

@@ -1,477 +0,0 @@
-/* eslint-env jest */
-const path = require("path");
-const { outputJson, readJson } = require("fs-extra");
-const tempy = require("tempy");
-const execa = require("execa");
-const { WritableStreamBuffer } = require("stream-buffers");
-const prepare = require("./prepare");
-
-let context;
-let mockChangedPackages;
-
-jest.mock("../lib/get-changed-packages", () => {
-	function getChangedPackagesMock() {
-		return mockChangedPackages;
-	}
-
-	return getChangedPackagesMock;
-});
-
-async function createProject(cwd, version, pkgData) {
-	const lernaPath = path.resolve(cwd, "lerna.json");
-	const manifestLocation = path.resolve(cwd, "package.json");
-	await outputJson(lernaPath, { version, packages: ["packages/*"] });
-	await outputJson(manifestLocation, { name: "root-pkg", version, ...pkgData });
-	return {
-		lernaPath,
-		manifestLocation,
-	};
-}
-
-/* eslint-disable-next-line max-params */
-async function createPackage(cwd, name, version, options, pkgData) {
-	const { changed = false } = options;
-	const pkgRoot = `packages/${name}`;
-	const location = path.resolve(cwd, pkgRoot);
-	const manifestLocation = path.resolve(cwd, pkgRoot, "package.json");
-	const pkg = {
-		name,
-		location,
-		manifestLocation,
-		shrinkwrapPath: path.resolve(cwd, pkgRoot, "npm-shrinkwrap.json"),
-		lockfilePath: path.resolve(cwd, pkgRoot, "package-lock.json"),
-	};
-	await outputJson(manifestLocation, { name, version, ...pkgData });
-	if (changed) {
-		mockChangedPackages.push(pkg);
-	}
-
-	return pkg;
-}
-
-beforeEach(() => {
-	const log = jest.fn();
-	context = {
-		log,
-		logger: { log },
-		stdout: new WritableStreamBuffer(),
-		stderr: new WritableStreamBuffer(),
-	};
-	mockChangedPackages = [];
-});
-
-it("Update lerna.json and root package.json when no package has changed", async () => {
-	expect.assertions(4);
-	const cwd = tempy.directory();
-	const npmrc = tempy.file({ name: ".npmrc" });
-	const project = await createProject(cwd, "0.0.0");
-
-	await prepare(
-		npmrc,
-		{},
-		{
-			cwd,
-			env: {},
-			stdout: context.stdout,
-			stderr: context.stderr,
-			nextRelease: { version: "1.0.0" },
-			logger: context.logger,
-		}
-	);
-
-	// Verify lerna.json has been updated
-	expect(await readJson(project.lernaPath)).toEqual(
-		expect.objectContaining({
-			version: "1.0.0",
-		})
-	);
-
-	// Verify root package.json has been updated
-	expect(await readJson(project.manifestLocation)).toEqual(
-		expect.objectContaining({
-			version: "1.0.0",
-		})
-	);
-
-	// Verify the logger has been called with the version updated
-	expect(context.log).toHaveBeenCalledWith(
-		"No packages changed, applying version bump on root package only"
-	);
-	expect(context.log).toHaveBeenCalledWith("Write version %s to lerna.json in %s", "1.0.0", cwd);
-});
-
-it("Update lerna.json and root package.json when one or more package has changed", async () => {
-	expect.assertions(5);
-	const cwd = tempy.directory();
-	const npmrc = tempy.file({ name: ".npmrc" });
-	const project = await createProject(cwd, "0.0.0");
-	const pkg = await createPackage(cwd, "foo", "0.0.0", {
-		changed: true,
-	});
-
-	await prepare(
-		npmrc,
-		{},
-		{
-			cwd,
-			env: {},
-			stdout: context.stdout,
-			stderr: context.stderr,
-			nextRelease: { version: "1.0.0" },
-			logger: context.logger,
-		}
-	);
-
-	// Verify lerna.json has been updated
-	expect(await readJson(project.lernaPath)).toEqual(
-		expect.objectContaining({
-			version: "1.0.0",
-		})
-	);
-
-	// Verify package.json has been updated
-	expect(await readJson(project.manifestLocation)).toEqual(
-		expect.objectContaining({
-			version: "1.0.0",
-		})
-	);
-
-	// Verify the logger has been called with the version updated
-	expect(context.log).toHaveBeenCalledWith("1 package need version bump: [ 'foo' ]");
-	expect(context.log).toHaveBeenCalledWith(
-		"Write version %s to package.json in %s",
-		"1.0.0",
-		pkg.location
-	);
-	expect(context.log).toHaveBeenCalledWith("Write version %s to lerna.json in %s", "1.0.0", cwd);
-});
-
-it("Update only lerna.json when one or more package has changed when option `rootVersion` is `false`", async () => {
-	expect.assertions(4);
-	const cwd = tempy.directory();
-	const npmrc = tempy.file({ name: ".npmrc" });
-	const project = await createProject(cwd, "0.0.0");
-
-	await createPackage(cwd, "foo", "0.0.0", {
-		changed: true,
-	});
-
-	await prepare(
-		npmrc,
-		{
-			rootVersion: false,
-		},
-		{
-			cwd,
-			env: {},
-			stdout: context.stdout,
-			stderr: context.stderr,
-			nextRelease: { version: "1.0.0" },
-			logger: context.logger,
-		}
-	);
-
-	// Verify lerna.json has been updated
-	expect(await readJson(project.lernaPath)).toEqual(
-		expect.objectContaining({
-			version: "1.0.0",
-		})
-	);
-
-	// Verify the logger has been called with the version updated
-	expect(context.log).toHaveBeenCalledWith("1 package need version bump: [ 'foo' ]");
-	expect(context.log).toHaveBeenCalledWith("Write version %s to lerna.json in %s", "1.0.0", cwd);
-	expect(context.log).toHaveBeenCalledWith("Don't write version to root package.json");
-});
-
-it("Update package.json in changed packages", async () => {
-	expect.assertions(2);
-	const cwd = tempy.directory();
-	const npmrc = tempy.file({ name: ".npmrc" });
-	await createProject(cwd, "0.0.0");
-	const foo = await createPackage(cwd, "foo", "0.0.0", {
-		changed: true,
-	});
-	const bar = await createPackage(cwd, "bar", "0.0.0", {
-		changed: false,
-	});
-
-	await prepare(
-		npmrc,
-		{},
-		{
-			cwd,
-			env: {},
-			stdout: context.stdout,
-			stderr: context.stderr,
-			nextRelease: { version: "1.0.0" },
-			logger: context.logger,
-		}
-	);
-
-	// Verify foo/package.json has been updated
-	expect(await readJson(foo.manifestLocation)).toEqual({
-		name: "foo",
-		version: "1.0.0",
-	});
-
-	// Verify bar/package.json has not been updated
-	expect(await readJson(bar.manifestLocation)).toEqual({
-		name: "bar",
-		version: "0.0.0",
-	});
-});
-
-it("Update npm-shrinkwrap.json if present", async () => {
-	expect.assertions(2);
-	const cwd = tempy.directory();
-	const npmrc = tempy.file({ name: ".npmrc" });
-	await createProject(cwd, "0.0.0");
-	const pkg = await createPackage(cwd, "foo", "0.0.0", {
-		changed: true,
-	});
-	// Create a npm-shrinkwrap.json file
-	await execa("npm", ["shrinkwrap"], { cwd: pkg.location });
-
-	await prepare(
-		npmrc,
-		{},
-		{
-			cwd,
-			env: {},
-			stdout: context.stdout,
-			stderr: context.stderr,
-			nextRelease: { version: "1.0.0" },
-			logger: context.logger,
-		}
-	);
-
-	// Verify foo/package.json has been updated
-	expect(await readJson(pkg.manifestLocation)).toEqual({
-		name: "foo",
-		version: "1.0.0",
-	});
-
-	// Verify foo/npm-shrinkwrap.json has been updated
-	expect(await readJson(pkg.shrinkwrapPath)).toEqual(
-		expect.objectContaining({
-			lockfileVersion: expect.anything(),
-			name: "foo",
-			version: "1.0.0",
-		})
-	);
-});
-
-it("Update package-lock.json if present", async () => {
-	expect.assertions(2);
-	const cwd = tempy.directory();
-	const npmrc = tempy.file({ name: ".npmrc" });
-	await createProject(cwd, "0.0.0");
-	const pkg = await createPackage(cwd, "foo", "0.0.0", {
-		changed: true,
-	});
-	// Create a package-lock.json file
-	await execa("npm", ["install"], { cwd: pkg.location });
-
-	await prepare(
-		npmrc,
-		{},
-		{
-			cwd,
-			env: {},
-			stdout: context.stdout,
-			stderr: context.stderr,
-			nextRelease: { version: "1.0.0" },
-			logger: context.logger,
-		}
-	);
-
-	// Verify foo/package.json has been updated
-	expect(await readJson(pkg.manifestLocation)).toEqual({
-		name: "foo",
-		version: "1.0.0",
-	});
-
-	// Verify foo/package-lock.json has been updated
-	expect(await readJson(pkg.lockfilePath)).toEqual(
-		expect.objectContaining({
-			lockfileVersion: expect.anything(),
-			name: "foo",
-			version: "1.0.0",
-		})
-	);
-});
-
-it("Update package.json dependency when using exact version", async () => {
-	expect.assertions(1);
-	const cwd = tempy.directory();
-	const npmrc = tempy.file({ name: ".npmrc" });
-	await createProject(cwd, "0.0.0");
-	const foo = await createPackage(
-		cwd,
-		"foo",
-		"0.0.0",
-		{
-			changed: true,
-		},
-		{
-			dependencies: {
-				a: "0.0.0",
-			},
-			devDependencies: {
-				b: "0.0.0",
-			},
-			peerDependencies: {
-				c: "0.0.0",
-			},
-		}
-	);
-	await createPackage(cwd, "a", "0.0.0", { changed: true });
-	await createPackage(cwd, "b", "0.0.0", { changed: true });
-	await createPackage(cwd, "c", "0.0.0", { changed: true });
-
-	await prepare(
-		npmrc,
-		{},
-		{
-			cwd,
-			env: {},
-			stdout: context.stdout,
-			stderr: context.stderr,
-			nextRelease: { version: "0.0.1" },
-			logger: context.logger,
-		}
-	);
-
-	// Verify dependency has been updated
-	expect(await readJson(foo.manifestLocation)).toEqual({
-		name: "foo",
-		version: "0.0.1",
-		dependencies: {
-			a: "0.0.1",
-		},
-		devDependencies: {
-			b: "0.0.1",
-		},
-		peerDependencies: {
-			c: "0.0.1",
-		},
-	});
-});
-
-it("Update package.json dependency when using hat", async () => {
-	expect.assertions(1);
-	const cwd = tempy.directory();
-	const npmrc = tempy.file({ name: ".npmrc" });
-	await createProject(cwd, "0.1.2");
-	const foo = await createPackage(
-		cwd,
-		"foo",
-		"0.1.2",
-		{
-			changed: true,
-		},
-		{
-			dependencies: {
-				a: "^0.1.2",
-				b: "^0.1",
-				c: "^0",
-			},
-		}
-	);
-
-	await createPackage(cwd, "a", "0.1.2", { changed: true });
-	await createPackage(cwd, "b", "0.1.2", { changed: true });
-	await createPackage(cwd, "c", "0.1.2", { changed: true });
-
-	await prepare(
-		npmrc,
-		{},
-		{
-			cwd,
-			env: {},
-			stdout: context.stdout,
-			stderr: context.stderr,
-			nextRelease: { version: "1.0.0" },
-			logger: context.logger,
-		}
-	);
-
-	// Verify dependency has been updated
-	expect(await readJson(foo.manifestLocation)).toEqual({
-		name: "foo",
-		version: "1.0.0",
-		dependencies: {
-			a: "^1.0.0",
-			b: "^1.0",
-			c: "^1",
-		},
-	});
-});
-
-it("Should not update other dependencies", async () => {
-	expect.assertions(1);
-	const cwd = tempy.directory();
-	const npmrc = tempy.file({ name: ".npmrc" });
-	await createProject(cwd, "0.0.0");
-	const foo = await createPackage(
-		cwd,
-		"foo",
-		"0.0.0",
-		{
-			changed: true,
-		},
-		{
-			dependencies: {
-				a: "0.0.0",
-			},
-		}
-	);
-
-	await prepare(
-		npmrc,
-		{},
-		{
-			cwd,
-			env: {},
-			stdout: context.stdout,
-			stderr: context.stderr,
-			nextRelease: { version: "0.0.1" },
-			logger: context.logger,
-		}
-	);
-
-	// Verify dependency has been updated
-	expect(await readJson(foo.manifestLocation)).toEqual({
-		name: "foo",
-		version: "0.0.1",
-		dependencies: {
-			a: "0.0.0",
-		},
-	});
-});
-
-it("Handle dependencies from root package", async () => {
-	expect.assertions(1);
-	const cwd = tempy.directory();
-	const npmrc = tempy.file({ name: ".npmrc" });
-	await createProject(cwd, "0.0.1", {
-		devDependencies: {
-			"external-dependency": "1.2.3",
-		},
-	});
-	expect(async () => {
-		await prepare(
-			npmrc,
-			{},
-			{
-				cwd,
-				env: {},
-				stdout: context.stdout,
-				stderr: context.stderr,
-				nextRelease: { version: "0.0.2" },
-				logger: context.logger,
-			}
-		);
-	}).not.toThrow();
-});

package/lib/should-latch.test.js

@@ -1,71 +0,0 @@
-/* eslint-env jest */
-
-const semver = require("semver");
-const shouldLatch = require("./should-latch");
-
-const version = "1.0.0";
-
-describe("latch none", () => {
-	it.each`
-		bump            | result   | version    | next
-		${"major"}      | ${false} | ${version} | ${semver.inc(version, "major")}
-		${"minor"}      | ${false} | ${version} | ${semver.inc(version, "minor")}
-		${"patch"}      | ${false} | ${version} | ${semver.inc(version, "patch")}
-		${"prerelease"} | ${false} | ${version} | ${semver.inc(version, "prerelease")}
-	`("should return $result when version bump is $bump ($version -> $next)", ({ result, next }) => {
-		expect.assertions(1);
-		expect(shouldLatch(next, "none")).toBe(result);
-	});
-});
-
-describe("latch major", () => {
-	it.each`
-		bump            | result   | version    | next
-		${"major"}      | ${true}  | ${version} | ${semver.inc(version, "major")}
-		${"minor"}      | ${false} | ${version} | ${semver.inc(version, "minor")}
-		${"patch"}      | ${false} | ${version} | ${semver.inc(version, "patch")}
-		${"prerelease"} | ${false} | ${version} | ${semver.inc(version, "prerelease")}
-	`("should return $result when version bump is $bump ($version -> $next)", ({ result, next }) => {
-		expect.assertions(1);
-		expect(shouldLatch(next, "major")).toBe(result);
-	});
-});
-
-describe("latch minor", () => {
-	it.each`
-		bump            | result   | version    | next
-		${"major"}      | ${true}  | ${version} | ${semver.inc(version, "major")}
-		${"minor"}      | ${true}  | ${version} | ${semver.inc(version, "minor")}
-		${"patch"}      | ${false} | ${version} | ${semver.inc(version, "patch")}
-		${"prerelease"} | ${false} | ${version} | ${semver.inc(version, "prerelease")}
-	`("should return $result when version bump is $bump ($version -> $next)", ({ result, next }) => {
-		expect.assertions(1);
-		expect(shouldLatch(next, "minor")).toBe(result);
-	});
-});
-
-describe("latch patch", () => {
-	it.each`
-		bump            | result   | version    | next
-		${"major"}      | ${true}  | ${version} | ${semver.inc(version, "major")}
-		${"minor"}      | ${true}  | ${version} | ${semver.inc(version, "minor")}
-		${"patch"}      | ${true}  | ${version} | ${semver.inc(version, "patch")}
-		${"prerelease"} | ${false} | ${version} | ${semver.inc(version, "prerelease")}
-	`("should return $result when version bump is $bump ($version -> $next)", ({ result, next }) => {
-		expect.assertions(1);
-		expect(shouldLatch(next, "patch")).toBe(result);
-	});
-});
-
-describe("latch prerelease", () => {
-	it.each`
-		bump            | result  | version    | next
-		${"major"}      | ${true} | ${version} | ${semver.inc(version, "major")}
-		${"minor"}      | ${true} | ${version} | ${semver.inc(version, "minor")}
-		${"patch"}      | ${true} | ${version} | ${semver.inc(version, "patch")}
-		${"prerelease"} | ${true} | ${version} | ${semver.inc(version, "prerelease")}
-	`("should return $result when version bump is $bump ($version -> $next)", ({ result, next }) => {
-		expect.assertions(1);
-		expect(shouldLatch(next, "prerelease")).toBe(result);
-	});
-});

package/lib/verify-git.test.js

@@ -1,49 +0,0 @@
-/* eslint-env jest */
-
-jest.mock("execa");
-
-const execa = require("execa");
-const verifyGit = require("./verify-git");
-
-it("should return error if working copy is dirty", async () => {
-	expect.assertions(2);
-	execa.mockImplementation(() => ({ stdout: " M file.js\n" }));
-	const errors = await verifyGit({});
-	expect(errors).toHaveLength(1);
-	expect(errors[0]).toMatchObject({
-		code: "EDIRTYWC",
-		details: `The git working copy must be clean before releasing:
-
- M file.js
-`,
-	});
-});
-
-it("should return error when working copy has mixed dirty and untracked", async () => {
-	expect.assertions(2);
-	execa.mockImplementation(() => ({ stdout: " M file.js\n?? file.c" }));
-	const errors = await verifyGit({});
-	expect(errors).toHaveLength(1);
-	expect(errors[0]).toMatchObject({
-		code: "EDIRTYWC",
-		details: `The git working copy must be clean before releasing:
-
- M file.js
-?? file.c
-`,
-	});
-});
-
-it("should ignore untracked files", async () => {
-	expect.assertions(1);
-	execa.mockImplementation(() => ({ stdout: "?? file.js\n" }));
-	const errors = await verifyGit({});
-	expect(errors).toHaveLength(0);
-});
-
-it("should not return error if working copy is clean", async () => {
-	expect.assertions(1);
-	execa.mockImplementation(() => ({ stdout: "" }));
-	const errors = await verifyGit({});
-	expect(errors).toHaveLength(0);
-});