Merge pull request #122 from semantic-release/fix/custom-path

Fixed double path resolve and improved testing

Author: seebeen

lib/prepare.ts

@@ -25,7 +25,7 @@ export async function prepare(
     ? path.join(
         path.join(os.tmpdir(), `workDir-${config.workDir}`, config.slug),
       )
-    : path.resolve(path.resolve(config.path));
+    : path.resolve(config.path);
 
   if (config.copyFiles) {
     await fs.mkdir(workDir, { recursive: true });

test/2-prepare-theme.spec.ts

@@ -87,12 +87,17 @@ it('Should fail on a non-existant version file', async () => {
 });
 
 it('Should fully prepare a theme for release', async () => {
+  await fs.copy(
+    './test/fixtures/complete-theme',
+    './test/fixtures/complete-theme-copy',
+  );
+
   await prepare(
     {
       type: 'theme',
-      slug: 'complete-theme',
-      path: './test/fixtures/complete-theme',
-      copyFiles: true,
+      slug: 'complete-theme-copy',
+      path: './test/fixtures/complete-theme-copy',
+      copyFiles: false,
       withVersionFile: true,
       withAssets: true,
       versionFiles: ['functions.php'],
@@ -103,10 +108,10 @@ it('Should fully prepare a theme for release', async () => {
     context,
   );
 
-  const themeDir = fs.readdirSync('/tmp/wp-release/complete-theme');
+  const themeDir = fs.readdirSync('/tmp/wp-release/complete-theme-copy');
   const assets = fs.readdirSync('/tmp/wp-release/assets');
   const versions = fs.readFileSync(
-    '/tmp/workDir-1/complete-theme/functions.php',
+    './test/fixtures/complete-theme-copy/functions.php',
     'utf8',
   );
 
@@ -126,4 +131,9 @@ it('Should fully prepare a theme for release', async () => {
   );
   expect(versions).toMatch(/1\.0\.0/);
   expect(versions).toMatch(/3\.2\.111/);
+
+  await fs.rm('./test/fixtures/complete-theme-copy', {
+    recursive: true,
+    force: true,
+  });
 });