www: check_assets.js should allow for blank lines in expected assets list

rvagg · rvagg · commit c38e1505f2ff · 2019-03-12T11:10:45.000+11:00
diff --git a/setup/www/tools/promote/check_assets.js b/setup/www/tools/promote/check_assets.js
@@ -107,7 +107,7 @@ async function loadExpectedAssets (version, line) {
   try {
     const templateFile = path.join(__dirname, 'expected_assets', line)
     let files = await fs.readFile(templateFile, 'utf8')
-    return files.replace(/{VERSION}/g, version).split(/\n/g)
+    return files.replace(/{VERSION}/g, version).split(/\n/g).filter(Boolean)
   } catch (e) { }
   return null
 }

Original file line number	Diff line number	Diff line change
`@@ -107,7 +107,7 @@ async function loadExpectedAssets (version, line) {`
`107`	`107`	`try {`
`108`	`108`	`const templateFile = path.join(__dirname, 'expected_assets', line)`
`109`	`109`	`let files = await fs.readFile(templateFile, 'utf8')`
`110`		`- return files.replace(/{VERSION}/g, version).split(/\n/g)`
	`110`	`+ return files.replace(/{VERSION}/g, version).split(/\n/g).filter(Boolean)`
`111`	`111`	`} catch (e) { }`
`112`	`112`	`return null`
`113`	`113`	`}`