Merge pull request #26 from ghiscoding/chore/unit-test-part2

fix(version): add missing code conventional-preset legacy loading

helpers/git-checkout/index.js

@@ -0,0 +1,9 @@
+"use strict";
+
+const execa = require("execa");
+
+module.exports.gitCheckout = gitCheckout;
+
+function gitCheckout(cwd, args) {
+  return execa("git", ["checkout", ...args], { cwd });
+}

helpers/git-checkout/package.json

@@ -0,0 +1,11 @@
+{
+  "name": "@lerna-test/git-checkout",
+  "version": "0.0.0-test-only",
+  "description": "A local test helper",
+  "main": "index.js",
+  "private": true,
+  "license": "MIT",
+  "dependencies": {
+    "execa": "^5.0.0"
+  }
+}

helpers/git-merge/index.js

@@ -0,0 +1,9 @@
+"use strict";
+
+const execa = require("execa");
+
+module.exports.gitMerge = gitMerge;
+
+function gitMerge(cwd, args) {
+  return execa("git", ["merge", ...args], { cwd });
+}

helpers/git-merge/package.json

@@ -0,0 +1,11 @@
+{
+  "name": "@lerna-test/git-merge",
+  "version": "0.0.0-test-only",
+  "description": "A local test helper",
+  "main": "index.js",
+  "private": true,
+  "license": "MIT",
+  "dependencies": {
+    "execa": "^5.0.0"
+  }
+}

helpers/git-status/index.js

@@ -0,0 +1,9 @@
+"use strict";
+
+const cp = require("child_process");
+
+module.exports.gitStatus = gitStatus;
+
+function gitStatus(cwd) {
+  return cp.spawnSync("git", ["status", "--porcelain"], { cwd, encoding: "utf8" });
+}

helpers/git-status/package.json

@@ -0,0 +1,11 @@
+{
+  "name": "@lerna-test/git-status",
+  "version": "0.0.0-test-only",
+  "description": "A local test helper",
+  "main": "index.js",
+  "private": true,
+  "license": "MIT",
+  "dependencies": {
+    "execa": "^5.0.0"
+  }
+}

helpers/git-tag/index.js

@@ -0,0 +1,9 @@
+"use strict";
+
+const execa = require("execa");
+
+module.exports.gitTag = gitTag;
+
+function gitTag(cwd, tagName) {
+  return execa("git", ["tag", tagName, "-m", tagName], { cwd });
+}

helpers/git-tag/package.json

@@ -0,0 +1,11 @@
+{
+  "name": "@lerna-test/git-tag",
+  "version": "0.0.0-test-only",
+  "description": "A local test helper",
+  "main": "index.js",
+  "private": true,
+  "license": "MIT",
+  "dependencies": {
+    "execa": "^5.0.0"
+  }
+}

helpers/serialize-changelog/index.js

@@ -0,0 +1,18 @@
+"use strict";
+
+const normalizeNewline = require("normalize-newline");
+const gitSHA = require("../serialize-git-sha");
+
+// expect.addSnapshotSerializer(require("@lerna-test/serialize-changelog"));
+module.exports = {
+  serialize(str) {
+    return gitSHA
+      .serialize(normalizeNewline(str))
+      .replace(/(\[.*?\])\(.*\/compare\/(.*?)\)/g, "$1(/compare/$2)")
+      .replace(/(\[.*?\])\(.*\/commits\/GIT_HEAD\)/g, "$1(COMMIT_URL)")
+      .replace(/\(\d{4}-\d{2}-\d{2}\)/g, "(YYYY-MM-DD)");
+  },
+  test(val) {
+    return val != null && typeof val === "string";
+  },
+};

helpers/serialize-changelog/package.json

@@ -0,0 +1,12 @@
+{
+  "name": "@lerna-test/serialize-changelog",
+  "version": "0.0.0-test-only",
+  "description": "A local test helper",
+  "main": "index.js",
+  "private": true,
+  "license": "MIT",
+  "dependencies": {
+    "@lerna-test/serialize-git-sha": "file:../serialize-git-sha",
+    "normalize-newline": "^3.0.0"
+  }
+}

helpers/serialize-git-sha/index.js

@@ -0,0 +1,17 @@
+"use strict";
+
+// expect.addSnapshotSerializer(require("@lerna-test/serialize-git-sha"));
+module.exports = {
+  serialize(str) {
+    return (
+      str
+        // short SHA tends to be in the path diff comparisons
+        .replace(/\b[0-9a-f]{7,8}\b/g, "SHA")
+        // full SHA corresponds to gitHead property in package.json files
+        .replace(/\b[0-9a-f]{40}\b/g, "GIT_HEAD")
+    );
+  },
+  test(val) {
+    return val != null && typeof val === "string" && /[0-9a-f]{7,40}/.test(val);
+  },
+};

helpers/serialize-git-sha/package.json

@@ -0,0 +1,9 @@
+{
+  "name": "@lerna-test/serialize-git-sha",
+  "version": "0.0.0-test-only",
+  "description": "A local test helper",
+  "main": "index.js",
+  "private": true,
+  "license": "MIT",
+  "dependencies": {}
+}

packages/core/src/conventional-commits/__tests__/__fixtures__/changelog-missing/lerna.json

@@ -0,0 +1,8 @@
+{
+  "command": {
+    "publish": {
+      "conventionalCommits": true
+    }
+  },
+  "version": "1.0.0"
+}

packages/core/src/conventional-commits/__tests__/__fixtures__/changelog-missing/package.json

@@ -0,0 +1,5 @@
+{
+  "name": "changelog-missing",
+  "repository": "lerna/changelog-missing",
+  "version": "0.0.0-root"
+}

packages/core/src/conventional-commits/__tests__/__fixtures__/changelog-missing/packages/package-1/package.json

@@ -0,0 +1,5 @@
+{
+  "name": "package-1",
+  "repository": "lerna/changelog-missing",
+  "version": "1.0.0"
+}

packages/core/src/conventional-commits/__tests__/__fixtures__/changelog-missing/packages/package-2/package.json

@@ -0,0 +1,8 @@
+{
+  "name": "package-2",
+  "repository": "lerna/changelog-missing",
+  "version": "1.0.0",
+  "dependencies": {
+    "package-1": "^1.0.0"
+  }
+}

packages/core/src/conventional-commits/__tests__/__fixtures__/fixed/CHANGELOG.md

@@ -0,0 +1,12 @@
+# Change Log
+
+All notable changes to this project will be documented in this file.
+See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
+
+<a name="1.0.0"></a>
+
+# 1.0.0 (YYYY-MM-DD)
+
+### Features
+
+* I should be placed in the CHANGELOG

packages/core/src/conventional-commits/__tests__/__fixtures__/fixed/lerna.json

@@ -0,0 +1,8 @@
+{
+  "command": {
+    "publish": {
+      "conventionalCommits": true
+    }
+  },
+  "version": "1.0.0"
+}

packages/core/src/conventional-commits/__tests__/__fixtures__/fixed/package.json

@@ -0,0 +1,5 @@
+{
+  "name": "conventional-commits-fixed",
+  "repository": "lerna/conventional-commits-fixed",
+  "version": "0.0.0-root"
+}

packages/core/src/conventional-commits/__tests__/__fixtures__/fixed/packages/package-1/CHANGELOG.md

@@ -0,0 +1,12 @@
+# Change Log
+
+All notable changes to this project will be documented in this file.
+See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
+
+<a name="1.0.0"></a>
+
+# 1.0.0 (YYYY-MM-DD)
+
+### Features
+
+* I should be placed in the CHANGELOG

packages/core/src/conventional-commits/__tests__/__fixtures__/fixed/packages/package-1/package.json

@@ -0,0 +1,5 @@
+{
+  "name": "package-1",
+  "repository": "lerna/conventional-commits-fixed",
+  "version": "1.0.0"
+}

packages/core/src/conventional-commits/__tests__/__fixtures__/fixed/packages/package-2/CHANGELOG.md

@@ -0,0 +1,12 @@
+# Change Log
+
+All notable changes to this project will be documented in this file.
+See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
+
+<a name="1.0.0"></a>
+
+# 1.0.0 (YYYY-MM-DD)
+
+### Features
+
+* I should be placed in the CHANGELOG

packages/core/src/conventional-commits/__tests__/__fixtures__/fixed/packages/package-2/package.json

@@ -0,0 +1,8 @@
+{
+  "name": "package-2",
+  "repository": "lerna/conventional-commits-fixed",
+  "version": "1.0.0",
+  "dependencies": {
+    "package-1": "^1.0.0"
+  }
+}

packages/core/src/conventional-commits/__tests__/__fixtures__/fixed/scripts/config-builder-preset.js

@@ -0,0 +1,27 @@
+"use strict";
+
+// https://git.io/vx5iq (conventional-changelog-angular/conventional-recommended-bump.js, etc)
+const parserOpts = require("./parser-opts");
+const writerOpts = require("./writer-opts");
+const whatBump = require("./what-bump");
+
+// https://git.io/fhyKK
+module.exports = presetOpts;
+
+function presetOpts(param) {
+  if (typeof param !== "function") {
+    return Promise.resolve(
+      Object.assign(param, {
+        parserOpts,
+        writerOpts,
+        whatBump,
+      })
+    );
+  }
+
+  process.nextTick(param, null, {
+    parserOpts,
+    writerOpts,
+    whatBump,
+  });
+}

packages/core/src/conventional-commits/__tests__/__fixtures__/fixed/scripts/erroring-preset.js

@@ -0,0 +1,7 @@
+"use strict";
+
+module.exports = {
+  recommendedBumpOpts: {
+    whatBump: "I should be a function",
+  },
+};

packages/core/src/conventional-commits/__tests__/__fixtures__/fixed/scripts/legacy-callback-preset.js

@@ -0,0 +1,17 @@
+"use strict";
+
+// https://git.io/vx5iq (conventional-changelog-angular/conventional-recommended-bump.js, etc)
+const parserOpts = require("./parser-opts");
+const writerOpts = require("./writer-opts");
+const whatBump = require("./what-bump");
+
+// https://git.io/fhyKK
+module.exports = presetOpts;
+
+function presetOpts(cb) {
+  process.nextTick(cb, null, {
+    parserOpts,
+    writerOpts,
+    whatBump,
+  });
+}

packages/core/src/conventional-commits/__tests__/__fixtures__/fixed/scripts/local-preset.js

@@ -0,0 +1,17 @@
+"use strict";
+
+// https://git.io/vx5iq (conventional-changelog-angular/conventional-recommended-bump.js, etc)
+const parserOpts = require("./parser-opts");
+const writerOpts = require("./writer-opts");
+const whatBump = require("./what-bump");
+
+module.exports = {
+  conventionalChangelog: {
+    parserOpts,
+    writerOpts,
+  },
+  recommendedBumpOpts: {
+    parserOpts,
+    whatBump,
+  },
+};

packages/core/src/conventional-commits/__tests__/__fixtures__/fixed/scripts/null-preset.js

@@ -0,0 +1,17 @@
+"use strict";
+
+// https://git.io/vx5iq (conventional-changelog-angular/conventional-recommended-bump.js, etc)
+const parserOpts = require("./parser-opts");
+const writerOpts = require("./writer-opts");
+const whatBump = require("./null-what-bump");
+
+module.exports = {
+  conventionalChangelog: {
+    parserOpts,
+    writerOpts,
+  },
+  recommendedBumpOpts: {
+    parserOpts,
+    whatBump,
+  },
+};

packages/core/src/conventional-commits/__tests__/__fixtures__/fixed/scripts/null-what-bump.js

@@ -0,0 +1,31 @@
+"use strict";
+
+module.exports = whatBump;
+
+function whatBump(commits) {
+  let level = null;
+  let breakings = 0;
+  let features = 0;
+
+  commits.forEach((commit) => {
+    if (commit.notes.length > 0) {
+      breakings += commit.notes.length;
+      level = 0;
+    } else if (commit.type === "feat") {
+      features += 1;
+      if (level === 2) {
+        level = 1;
+      }
+    } else if (commit.type === "fix") {
+      level = 2;
+    }
+  });
+
+  return {
+    level,
+    reason:
+      breakings === 1
+        ? `There are ${breakings} BREAKING CHANGE and ${features} features`
+        : `There are ${breakings} BREAKING CHANGES and ${features} features`,
+  };
+}

packages/core/src/conventional-commits/__tests__/__fixtures__/fixed/scripts/old-api-preset.js

@@ -0,0 +1,12 @@
+"use strict";
+
+// https://git.io/vx5iq (conventional-changelog-angular/conventional-recommended-bump.js, etc)
+const parserOpts = require("./parser-opts");
+const writerOpts = require("./writer-opts");
+const whatBump = require("./what-bump");
+
+module.exports = {
+  parserOpts,
+  writerOpts,
+  whatBump,
+};

packages/core/src/conventional-commits/__tests__/__fixtures__/fixed/scripts/parser-opts.js

@@ -0,0 +1,9 @@
+"use strict";
+
+module.exports = {
+  headerPattern: /^(\w*)(?:\((.*)\))?: (.*)$/,
+  headerCorrespondence: [`type`, `scope`, `subject`],
+  noteKeywords: [`BREAKING CHANGE`],
+  revertPattern: /^revert:\s([\s\S]*?)\s*This reverts commit (\w*)\./,
+  revertCorrespondence: [`header`, `hash`],
+};