actions/setup-node
3
0
Fork 0
Code Issues Actions Packages Projects Releases Wiki Activity

adding test for NODE_AUTH_TOKEN overwritten

Browse Source
This commit is contained in:
Andres Vargas 2020-10-21 18:14:34 -05:00
parent c6fd00ceb9
commit 8dde5bc004
2 changed files with 13 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
__tests__/authutil.test.ts
View File

@ -122,5 +122,16 @@ describe('authutil tests', () => {
let rc = readRcFile(rcFile); let rc = readRcFile(rcFile);
expect(rc['registry']).toBe('https://registry.npmjs.org/'); expect(rc['registry']).toBe('https://registry.npmjs.org/');
expect(rc['always-auth']).toBe('true'); expect(rc['always-auth']).toBe('true');
}); })
it('It is already set the NODE_AUTH_TOKEN export it ', async ()=> {
process.env.NODE_AUTH_TOKEN='foobar';
await auth.configAuthentication('npm.pkg.github.com', 'false');
expect(fs.statSync(rcFile)).toBeDefined();
let rc = readRcFile(rcFile);
expect(rc['@ownername:registry']).toBe('npm.pkg.github.com/');
expect(rc['always-auth']).toBe('false');
dbg(`${JSON.stringify(rc)}`);
expect(process.env.NODE_AUTH_TOKEN).toEqual('foobar');
})
}); });

2
src/authutil.ts
View File

@ -54,5 +54,5 @@ function writeRegistryToFile(
fs.writeFileSync(fileLocation, newContents); fs.writeFileSync(fileLocation, newContents);
core.exportVariable('NPM_CONFIG_USERCONFIG', fileLocation); core.exportVariable('NPM_CONFIG_USERCONFIG', fileLocation);
// Export empty node_auth_token so npm doesn't complain about not being able to find it // Export empty node_auth_token so npm doesn't complain about not being able to find it
core.exportVariable('NODE_AUTH_TOKEN', 'XXXXX-XXXXX-XXXXX-XXXXX'); core.exportVariable('NODE_AUTH_TOKEN', process.env.NODE_AUTH_TOKEN || 'XXXXX-XXXXX-XXXXX-XXXXX');
} }