From 67335567833b6e01a045678d4d1e7a9b5403a41c Mon Sep 17 00:00:00 2001 From: Richard Lau Date: Tue, 15 Nov 2022 16:58:03 +0000 Subject: [PATCH] test: add test to validate changelogs for releases Add a new test to check that the changelog files have been correctly updated for releases. PR-URL: https://github.com/nodejs/node/pull/45325 Reviewed-By: Antoine du Hamel Reviewed-By: Rafael Gonzaga --- test/parallel/test-release-changelog.js | 89 +++++++++++++++++++++++++ 1 file changed, 89 insertions(+) create mode 100644 test/parallel/test-release-changelog.js diff --git a/test/parallel/test-release-changelog.js b/test/parallel/test-release-changelog.js new file mode 100644 index 00000000000000..99889fa1724014 --- /dev/null +++ b/test/parallel/test-release-changelog.js @@ -0,0 +1,89 @@ +'use strict'; + +// This test checks that the changelogs contain an entry for releases. + +const common = require('../common'); +const assert = require('assert'); +const fs = require('fs'); +const path = require('path'); + +const getDefine = (text, name) => { + const regexp = new RegExp(`#define\\s+${name}\\s+(.*)`); + const match = regexp.exec(text); + assert.notStrictEqual(match, null); + return match[1]; +}; + +const srcRoot = path.join(__dirname, '..', '..'); +const mainChangelogFile = path.join(srcRoot, 'CHANGELOG.md'); +const versionFile = path.join(srcRoot, 'src', 'node_version.h'); +const versionText = fs.readFileSync(versionFile, { encoding: 'utf8' }); +const release = getDefine(versionText, 'NODE_VERSION_IS_RELEASE') !== '0'; + +if (!release) { + common.skip('release bit is not set'); +} + +const major = getDefine(versionText, 'NODE_MAJOR_VERSION'); +const minor = getDefine(versionText, 'NODE_MINOR_VERSION'); +const patch = getDefine(versionText, 'NODE_PATCH_VERSION'); +const versionForRegex = `${major}\\.${minor}\\.${patch}`; + +const lts = getDefine(versionText, 'NODE_VERSION_IS_LTS') !== '0'; +const codename = getDefine(versionText, 'NODE_VERSION_LTS_CODENAME').slice(1, -1); +// If the LTS bit is set there should be a codename. +if (lts) { + assert.notStrictEqual(codename, ''); +} + +const changelogPath = `doc/changelogs/CHANGELOG_V${major}.md`; +// Check CHANGELOG_V*.md +{ + const changelog = fs.readFileSync(path.join(srcRoot, changelogPath), { encoding: 'utf8' }); + // Check title matches major version. + assert.match(changelog, new RegExp(`# Node\\.js ${major} ChangeLog`)); + // Check table header + let tableHeader; + if (lts) { + tableHeader = new RegExp(`LTS '${codename}'`); + } else { + tableHeader = /Current<\/th>/; + } + assert.match(changelog, tableHeader); + // Check table contains link to this release. + assert.match(changelog, new RegExp(`${versionForRegex}`)); + // Check anchor for this release. + assert.match(changelog, new RegExp(``)); + // Check title for changelog entry. + let title; + if (lts) { + title = new RegExp(`## \\d{4}-\\d{2}-\\d{2}, Version ${versionForRegex} '${codename}' \\(LTS\\), @\\S+`); + } else { + title = new RegExp(`## \\d{4}-\\d{2}-\\d{2}, Version ${versionForRegex} \\(Current\\), @\\S+`); + } + assert.match(changelog, title); +} + +// Main CHANGELOG.md checks +{ + const mainChangelog = fs.readFileSync(mainChangelogFile, { encoding: 'utf8' }); + // Check for the link to the appropriate CHANGELOG_V*.md file. + let linkToChangelog; + if (lts) { + linkToChangelog = new RegExp(`\\[Node\\.js ${major}\\]\\(${changelogPath}\\) \\*\\*Long Term Support\\*\\*`); + } else { + linkToChangelog = new RegExp(`\\[Node\\.js ${major}\\]\\(${changelogPath}\\) \\*\\*Current\\*\\*`); + } + assert.match(mainChangelog, linkToChangelog); + // Check table header. + let tableHeader; + if (lts) { + tableHeader = new RegExp(`${major} \\(LTS\\)`); + } else { + tableHeader = new RegExp(`${major} \\(Current\\)`); + } + assert.match(mainChangelog, tableHeader); + // Check the table contains a link to the release in the appropriate CHANGELOG_V*.md file. + const linkToVersion = new RegExp(`${versionForRegex}
`); + assert.match(mainChangelog, linkToVersion); +}