Skip to content
This repository has been archived by the owner on Apr 16, 2020. It is now read-only.

Commit

Permalink
separated test plug-in write and javascript
Browse files Browse the repository at this point in the history
testing of forwarder writing is going to need to use both independently
  • Loading branch information
grahamdyson committed Dec 5, 2018
1 parent 78628fa commit 93bf9c0
Show file tree
Hide file tree
Showing 6 changed files with 47 additions and 30 deletions.
@@ -1,4 +1,5 @@
const
getPluginJavascript = require("../../../../../../tests/getPluginJavascript"),
path = require("path"),
writePlugin = require("../../../../../../tests/writePlugin");

Expand Down Expand Up @@ -28,9 +29,13 @@ async function writePluginFilesWithRequirePath({
.map(
filePathRelativeToPackage =>
writePlugin({
filePath: path.join(plugin.directory, filePathRelativeToPackage),
repositoryRequire,
value: plugin.name,
filePath:
path.join(plugin.directory, filePathRelativeToPackage),
javascript:
getPluginJavascript({
repositoryRequire,
value: plugin.name,
}),
}),
),
);
Expand Down
@@ -1,6 +1,7 @@
const
createRelativePluginsOfRepositoryFilename = require("../../../../../tests/createRelativePluginsOfRepositoryFilename"),
fs = require("fs"),
getPluginJavascript = require("../../../../../tests/getPluginJavascript"),
path = require("path"),
{ promisify } = require("util"),
writePlugin = require("../../../../../tests/writePlugin");
Expand Down Expand Up @@ -75,10 +76,13 @@ module.exports =
await writePlugin({
filePath:
path.join(directory, repository.directory, plugin.filePath),
repositoryRequire:
plugin.repositoryRequire,
value:
plugin.value,
javascript:
getPluginJavascript({
repositoryRequire:
plugin.repositoryRequire,
value:
plugin.value,
}),
});
}
}
Expand Down
Expand Up @@ -2,6 +2,7 @@ require("array.prototype.flatmap")
.shim();

const
getPluginJavascript = require("../../../../../../tests/getPluginJavascript"),
path = require("path"),
writePlugin = require("../../../../../../tests/writePlugin"),
writeRepositoryPackage = require("../../../writeRepositoryPackage");
Expand Down Expand Up @@ -36,22 +37,23 @@ module.exports =
),
);

function writePluginForRepository({
async function writePluginForRepository({
plugin: { filename: pluginFilename },
repository: { require: repositoryRequire },
}) {
return (
writePlugin({
filePath:
path.join(
directory,
pluginFilename,
),
repositoryRequire,
value:
"test plug-in of repository in package transformed",
})
);
await writePlugin({
filePath:
path.join(
directory,
pluginFilename,
),
javascript:
getPluginJavascript({
repositoryRequire,
value:
"test plug-in of repository in package transformed",
}),
});
}

function getDirectoryForPackage(
Expand Down
12 changes: 8 additions & 4 deletions create-repository/tests/test.js
Expand Up @@ -2,6 +2,7 @@ const
callModuleInProcess = require("../../tests/callModuleInProcess"),
createRelativePluginsOfRepositoryFilename = require("../../tests/createRelativePluginsOfRepositoryFilename"),
fs = require("fs"),
getPluginJavascript = require("../../tests/getPluginJavascript"),
path = require("path"),
{ promisify } = require("util"),
readRepositoryTransformed = require("./readRepositoryTransformed"),
Expand Down Expand Up @@ -46,10 +47,13 @@ async function testIterateRepository() {
writePlugin({
filePath:
path.join(directory, plugin.filePath),
repositoryRequire:
plugin.repositoryRequire,
value:
plugin.value,
javascript:
getPluginJavascript({
repositoryRequire:
plugin.repositoryRequire,
value:
plugin.value,
}),
}),
),
);
Expand Down
6 changes: 6 additions & 0 deletions tests/getPluginJavascript.js
@@ -0,0 +1,6 @@
module.exports =
({
repositoryRequire,
value,
}) =>
`require("${repositoryRequire}").plugIn("${value}");`;
8 changes: 2 additions & 6 deletions tests/writePlugin.js
Expand Up @@ -5,15 +5,11 @@ const
module.exports =
async({
filePath,
repositoryRequire,
value,
javascript,
}) => {
await ensureDir(
path.dirname(filePath),
);

await writeFile(
filePath,
`require("${repositoryRequire}").plugIn("${value}");`,
);
await writeFile(filePath, javascript);
};

0 comments on commit 93bf9c0

Please sign in to comment.