Fix: replace whitespace before splitting project names (#35)

dhedgecock · web-flow · commit 06a0cd6c3ec7 · 2023-08-08T14:38:33.000-07:00
diff --git a/src/datasource.ts b/src/datasource.ts
@@ -125,7 +125,9 @@ export class DataSource extends DataSourceApi<LightstepQuery, LightstepDataSourc
 
   /** Return the set of configured project names for data source */
   projects(): string[] {
-    return this.projectName.split(',');
+    // nb string replace removes optional whitespace between project names, eg:
+    // "dev, pre-prod, prod" -> ["dev", "pre-prod", "prod"]
+    return this.projectName.replace(/\s/g, '').split(',');
   }
 
   /** Returns the first configured project name for data source */

Original file line number	Diff line number	Diff line change
`@@ -125,7 +125,9 @@ export class DataSource extends DataSourceApi<LightstepQuery, LightstepDataSourc`
`125`	`125`
`126`	`126`	`/** Return the set of configured project names for data source */`
`127`	`127`	`projects(): string[] {`
`128`		`- return this.projectName.split(',');`
	`128`	`+ // nb string replace removes optional whitespace between project names, eg:`
	`129`	`+ // "dev, pre-prod, prod" -> ["dev", "pre-prod", "prod"]`
	`130`	`+ return this.projectName.replace(/\s/g, '').split(',');`
`129`	`131`	`}`
`130`	`132`
`131`	`133`	`/** Returns the first configured project name for data source */`