Skip to content

Commit 4af9032

Browse files
Move imports rearrange logic to server
1 parent 80486f1 commit 4af9032

File tree

2 files changed

+4
-20
lines changed

2 files changed

+4
-20
lines changed

src/helpers/unitTestsExpand.js

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@ const {
1111
getRelatedTestImports,
1212
getSourceCodeFromAst,
1313
getModuleTypeFromFilePath,
14-
rearrangeImportsInCode,
1514
replaceRequirePaths
1615
} = require("../utils/code");
1716
const { getFunctionsForExport } = require("./unitTests");
@@ -35,15 +34,13 @@ let functionList = {},
3534
ignoreErrors = ["BABEL_PARSER_SYNTAX_ERROR"],
3635
force;
3736

38-
async function saveTests(filePath, fileName, originalTests, newTests) {
37+
async function saveTests(filePath, fileName, newTests) {
3938
let dir = filePath.substring(0, filePath.lastIndexOf("/"));
4039
if (!await checkDirectoryExists(dir)) {
4140
fs.mkdirSync(dir, { recursive: true });
4241
}
4342
let testPath = path.join(dir, `/${fileName}`);
44-
let mergedTestCode = `${originalTests.trim()}\n\n${newTests.trim()}`;
45-
mergedTestCode = rearrangeImportsInCode(mergedTestCode);
46-
fs.writeFileSync(testPath, mergedTestCode);
43+
fs.writeFileSync(testPath, newTests);
4744
return testPath;
4845
}
4946

@@ -130,7 +127,7 @@ async function createAdditionalTests(filePath, prefix) {
130127
});
131128

132129
if (tests) {
133-
await saveTests(testPath, formattedData.testFileName, testCode, tests);
130+
await saveTests(testPath, formattedData.testFileName, tests);
134131
testsGenerated.push(testPath);
135132
await spinner.stop();
136133
folderStructureTree[

src/utils/code.js

Lines changed: 1 addition & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -382,18 +382,6 @@ function getRelatedTestImports(ast, filePath, functionList) {
382382
return relatedFunctions;
383383
}
384384

385-
function rearrangeImportsInCode(code) {
386-
// Match both 'import' and 'require' statements, with or without semicolons
387-
const importAndRequireRegex = /^(import .+ from .+;|const .+ = require\(.+\);?)[\r\n]*/gm;
388-
let extractedStatements = code.match(importAndRequireRegex);
389-
let codeExcludingStatements = code.replace(importAndRequireRegex, '');
390-
extractedStatements = [...new Set(extractedStatements.map(statement => statement.trim()))];
391-
let consolidatedStatements = extractedStatements.join('\n');
392-
let codeWithReorderedStatements = `${consolidatedStatements}\n\n${codeExcludingStatements}`;
393-
394-
return codeWithReorderedStatements;
395-
}
396-
397385
module.exports = {
398386
replaceRequirePaths,
399387
getAstFromFilePath,
@@ -404,6 +392,5 @@ module.exports = {
404392
processAst,
405393
getModuleTypeFromFilePath,
406394
getSourceCodeFromAst,
407-
getRelatedTestImports,
408-
rearrangeImportsInCode
395+
getRelatedTestImports
409396
}

0 commit comments

Comments
 (0)