Force all scope names to be lowercased (actions#39)

JamesMGreene · Danny McCormick · commit d43864199af8 · 2019-08-21T22:46:35.000-04:00
Fixes actions#38
diff --git a/__tests__/__snapshots__/authutil.test.ts.snap b/__tests__/__snapshots__/authutil.test.ts.snap
@@ -7,12 +7,12 @@ registry=https://registry.npmjs.org/"
 
 exports[`installer tests Automatically configures GPR scope 1`] = `
 "npm.pkg.github.com/:_authToken=\${NODE_AUTH_TOKEN}
-@owner:registry=npm.pkg.github.com/"
+@ownername:registry=npm.pkg.github.com/"
 `;
 
 exports[`installer tests Configures scoped npm registries 1`] = `
 "//registry.npmjs.org/:_authToken=\${NODE_AUTH_TOKEN}
-@myScope:registry=https://registry.npmjs.org/"
+@myscope:registry=https://registry.npmjs.org/"
 `;
 
 exports[`installer tests Sets up npmrc for npmjs 1`] = `
diff --git a/__tests__/authutil.test.ts b/__tests__/authutil.test.ts
@@ -15,7 +15,7 @@ const tempDir = path.join(
 
 const rcFile = path.join(tempDir, '.npmrc');
 
-process.env['GITHUB_REPOSITORY'] = 'owner/repo';
+process.env['GITHUB_REPOSITORY'] = 'OwnerName/repo';
 process.env['RUNNER_TEMP'] = tempDir;
 import * as auth from '../src/authutil';
 
diff --git a/lib/authutil.js b/lib/authutil.js
@@ -28,6 +28,9 @@ function writeRegistryToFile(registryUrl, fileLocation) {
     if (scope && scope[0] != '@') {
         scope = '@' + scope;
     }
+    if (scope) {
+        scope = scope.toLowerCase();
+    }
     core.debug(`Setting auth in ${fileLocation}`);
     let newContents = '';
     if (fs.existsSync(fileLocation)) {
diff --git a/src/authutil.ts b/src/authutil.ts
@@ -24,6 +24,9 @@ function writeRegistryToFile(registryUrl: string, fileLocation: string) {
   if (scope && scope[0] != '@') {
     scope = '@' + scope;
   }
+  if (scope) {
+    scope = scope.toLowerCase();
+  }
 
   core.debug(`Setting auth in ${fileLocation}`);
   let newContents: string = '';

Original file line number	Diff line number	Diff line change
`@@ -24,6 +24,9 @@ function writeRegistryToFile(registryUrl: string, fileLocation: string) {`
`24`	`24`	`if (scope && scope[0] != '@') {`
`25`	`25`	`scope = '@' + scope;`
`26`	`26`	`}`
	`27`	`+ if (scope) {`
	`28`	`+ scope = scope.toLowerCase();`
	`29`	`+ }`
`27`	`30`
`28`	`31`	core.debug(`Setting auth in ${fileLocation}`);
`29`	`32`	`let newContents: string = '';`