Merge pull request #307 from actions/v-alsvir/disable-cache-usage-tem…

…p-v1 Temporarily disable image cache usage for v1
actions · Aug 3, 2021 · eb41679 · eb41679
2 parents 56899e0 + 5f10e34
commit eb41679
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 6 deletions.
diff --git a/__tests__/installer.test.ts b/__tests__/installer.test.ts
@@ -88,7 +88,8 @@ describe('installer tests', () => {
     }
   }, 100000);
 
-  it('Uses version of node installed in cache', async () => {
+  // Usage of pre-cached Node.js is temporarily disabled.
+  xit('Uses version of node installed in cache', async () => {
     const nodeDir: string = path.join(toolDir, 'node', '250.0.0', os.arch());
     await io.mkdirP(nodeDir);
     fs.writeFileSync(`${nodeDir}.complete`, 'hello');
@@ -111,7 +112,8 @@ describe('installer tests', () => {
     return;
   });
 
-  it('Resolves semantic versions of node installed in cache', async () => {
+  // Usage of pre-cached Node.js is temporarily disabled.
+  xit('Resolves semantic versions of node installed in cache', async () => {
     const nodeDir: string = path.join(toolDir, 'node', '252.0.0', os.arch());
     await io.mkdirP(nodeDir);
     fs.writeFileSync(`${nodeDir}.complete`, 'hello');

diff --git a/dist/index.js b/dist/index.js
@@ -12231,7 +12231,8 @@ function getNode(versionSpec) {
         let toolPath;
         toolPath = tc.find('node', versionSpec);
         // If not found in cache, download
-        if (!toolPath) {
+        // Usage of pre-cached Node.js is temporarily disabled.
+        if (true) {
             let version;
             const c = semver.clean(versionSpec) || '';
             // If explicit version
@@ -12248,7 +12249,8 @@ function getNode(versionSpec) {
                 // check cache
                 toolPath = tc.find('node', version);
             }
-            if (!toolPath) {
+            // Usage of pre-cached Node.js is temporarily disabled.
+            if (true) {
                 // download, extract, cache
                 toolPath = yield acquireNode(version);
             }
@@ -12342,6 +12344,7 @@ function acquireNode(version) {
         let downloadUrl = `https://nodejs.org/dist/v${version}/${urlFileName}`;
         let downloadPath;
         try {
+            core.info(`Downloading ${downloadUrl}`);
             downloadPath = yield tc.downloadTool(downloadUrl);
         }
         catch (err) {

diff --git a/src/installer.ts b/src/installer.ts
@@ -25,7 +25,8 @@ export async function getNode(versionSpec: string) {
   toolPath = tc.find('node', versionSpec);
 
   // If not found in cache, download
-  if (!toolPath) {
+  // Usage of pre-cached Node.js is temporarily disabled.
+  if (true) {
     let version: string;
     const c = semver.clean(versionSpec) || '';
     // If explicit version
@@ -45,7 +46,8 @@ export async function getNode(versionSpec: string) {
       toolPath = tc.find('node', version);
     }
 
-    if (!toolPath) {
+    // Usage of pre-cached Node.js is temporarily disabled.
+    if (true) {
       // download, extract, cache
       toolPath = await acquireNode(version);
     }
@@ -146,6 +148,7 @@ async function acquireNode(version: string): Promise<string> {
   let downloadPath: string;
 
   try {
+    core.info(`Downloading ${downloadUrl}`);
     downloadPath = await tc.downloadTool(downloadUrl);
   } catch (err) {
     if (err instanceof tc.HTTPError && err.httpStatusCode == 404) {