Skip to content

Commit 2f81d9b

Browse files
Update test
1 parent 7a33ad2 commit 2f81d9b

File tree

2 files changed

+25
-13
lines changed

2 files changed

+25
-13
lines changed

lib/utils/find-and-replace-in-directory.js

Lines changed: 16 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -5,19 +5,24 @@
55
const fs = require('fs-extra');
66
const path = require('path');
77

8-
function processFile(filePath, addonDocsRootURL, encodedVersion) {
9-
const contents = fs.readFileSync(filePath, 'utf-8');
10-
11-
contents
8+
function replaceAddonDocsRootURL(contents, addonDocsRootURL, encodedVersion) {
9+
return contents
1210
.replace('%2FADDON_DOCS_ROOT_URL%2F', encodeURIComponent(addonDocsRootURL))
1311
.replace(/\/?ADDON_DOCS_ROOT_URL\/?/g, addonDocsRootURL)
1412
.replace(/%22ADDON_DOCS_DEPLOY_VERSION%22/g, encodedVersion);
13+
}
14+
15+
function processFile(filePath, addonDocsRootURL, encodedVersion) {
16+
const contents = fs.readFileSync(filePath, 'utf-8');
1517

1618
// Write the updated content to the file
17-
fs.writeFileSync(filePath, contents);
19+
fs.writeFileSync(
20+
filePath,
21+
replaceAddonDocsRootURL(contents, addonDocsRootURL, encodedVersion),
22+
);
1823
}
1924

20-
module.exports = function findAndReplaceInDirectory(
25+
function findAndReplaceInDirectory(
2126
directory,
2227
addonDocsRootURL,
2328
encodedVersion,
@@ -40,4 +45,9 @@ module.exports = function findAndReplaceInDirectory(
4045
}
4146
});
4247
});
48+
}
49+
50+
module.exports = {
51+
findAndReplaceInDirectory,
52+
replaceAddonDocsRootURL,
4353
};

tests-node/unit/deploy/plugin-test.js

Lines changed: 9 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,21 +1,21 @@
11
'use strict';
22

33
const assert = require('chai').assert;
4-
const AddonDocsDeployPlugin = require('../../../lib/deploy/plugin');
4+
const {
5+
replaceAddonDocsRootURL,
6+
} = require('../../../lib/utils/find-and-replace-in-directory');
57

68
describe('`deploy` | plugin test', function () {
7-
beforeEach(function () {
8-
this.pluginInstance = new AddonDocsDeployPlugin();
9-
});
10-
11-
it.skip('_macroReplaceIndexContent', function () {
9+
it('replaceAddonDocsRootURL', function () {
1210
const contents = `
1311
<!DOCTYPE html>
1412
<html>
1513
<head>
1614
<meta name="dummy/config/environment" content="%7B%22rootURL%22%3A%22%2FADDON_DOCS_ROOT_URL%2F%22%7D" />
1715
</head>
1816
<body>
17+
<script src="ADDON_DOCS_ROOT_URL/assets/chunk.805.d0514e7e9edb751c6623.js"></script>
18+
<script src="ADDON_DOCS_ROOT_URL/assets/chunk.524.385868f2db0b958b5ced.js"></script>
1919
<script src="/ADDON_DOCS_ROOT_URL/assets/vendor.js"></script>
2020
<script src="/ADDON_DOCS_ROOT_URL/assets/dummy.js"></script>
2121
</body>
@@ -31,7 +31,7 @@ describe('`deploy` | plugin test', function () {
3131
}),
3232
);
3333
const addonDocsRootURL = '/my-addon/versions/master/';
34-
const actual = this.pluginInstance._macroReplaceIndexContent(
34+
const actual = replaceAddonDocsRootURL(
3535
contents,
3636
addonDocsRootURL,
3737
encodedVersion,
@@ -43,6 +43,8 @@ describe('`deploy` | plugin test', function () {
4343
<meta name="dummy/config/environment" content="%7B%22rootURL%22%3A%22%2Fmy-addon%2Fversions%2Fmaster%2F%22%7D" />
4444
</head>
4545
<body>
46+
<script src="/my-addon/versions/master/assets/chunk.805.d0514e7e9edb751c6623.js"></script>
47+
<script src="/my-addon/versions/master/assets/chunk.524.385868f2db0b958b5ced.js"></script>
4648
<script src="/my-addon/versions/master/assets/vendor.js"></script>
4749
<script src="/my-addon/versions/master/assets/dummy.js"></script>
4850
</body>

0 commit comments

Comments
 (0)