Merge pull request #38 from jdvivar/feat/add-undefined-content-non-ht…

…ml-files feat: adding undefined content with non-html files edge case
jdvivar · Jul 10, 2023 · a70d7fa · a70d7fa
2 parents 98f0aca + 46cea69
commit a70d7fa
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/add-web-component-definitions.js b/add-web-component-definitions.js
@@ -16,7 +16,7 @@ const addChild = (body, child, position) => {
 }
 
 module.exports = function (options, content, outputPath) {
-  if (typeof outputPath !== 'string' || !outputPath) { return content }
+  if (typeof outputPath !== 'string' || !outputPath || !content) { return content }
 
   if (!!options.path && !!options.specifiers) {
     throw new Error('You may configure a path function or import specifiers, but not both')

diff --git a/test/add-web-component-definitions.test.js b/test/add-web-component-definitions.test.js
@@ -11,6 +11,16 @@ test('Not html', t => {
   )
 })
 
+test('Undefined with non-html files', t => {
+  t.is(
+    addWebComponentDefinitions.bind(null, {})(
+      undefined,
+      'main.css'
+    ),
+    undefined
+  )
+})
+
 test('Is html', t => {
   t.is(
     addWebComponentDefinitions.bind(null, {})(