Add test for inline code extractions

Author: anthonyec

.prettierignore

@@ -1 +1,3 @@
 dist/
+test/**/expected
+test/**/output

src/staticbuild.ts

@@ -411,7 +411,11 @@ function collectInlineCodeFromDocument(
 
   for (const element of document.querySelectorAll("style, script:not([src])")) {
     if (element.hasAttribute("sb:buildtime")) continue
-    if (element.hasAttribute("sb:ignore")) continue
+
+    if (element.hasAttribute("sb:ignore")) {
+      element.removeAttribute("sb:ignore")
+      continue
+    }
 
     const textContent = element.textContent
     element.remove()

test/basic_inline_code/expected/assets/css/cd0d4bbfe756b191d9df2f498713d3b1.css

@@ -0,0 +1,10 @@
+
+  body {
+    background: red;
+  }
+
+
+  body {
+    background: green;
+  }
+

test/basic_inline_code/expected/assets/js/5afe3ad231347777bcd28103902f31db.js

@@ -0,0 +1,6 @@
+(function() {
+  console.log("Hello")
+})();
+(function() {
+  console.log("World")
+})();

test/basic_inline_code/expected/index.html

@@ -0,0 +1,18 @@
+
+
+
+
+
+
+
+
+<script>
+  console.log("Don't extract me!")
+</script>
+
+<style>
+  body {
+    content: "Don't extract me either!"
+  }
+</style>
+<link rel="stylesheet" href="/assets/css/cd0d4bbfe756b191d9df2f498713d3b1.css"/ ><script src="/assets/js/5afe3ad231347777bcd28103902f31db.js" async defer></script>

test/basic_inline_code/input/index.html

@@ -0,0 +1,29 @@
+<script>
+  console.log("Hello")
+</script>
+
+<script>
+  console.log("World")
+</script>
+
+<style>
+  body {
+    background: red;
+  }
+</style>
+
+<style>
+  body {
+    background: green;
+  }
+</style>
+
+<script sb:ignore>
+  console.log("Don't extract me!")
+</script>
+
+<style sb:ignore>
+  body {
+    content: "Don't extract me either!"
+  }
+</style>

test/index.ts

@@ -18,8 +18,11 @@ function expectFilesEqual(actualFilePath: string, expectedFilePath: string) {
   expectFileExists(actualFilePath)
   expectFileExists(expectedFilePath)
 
-  const actualContents = fs.readFileSync(actualFilePath, "utf8")
-  const expectedContents = fs.readFileSync(expectedFilePath, "utf8")
+  // @TODO: These are trimmed to avoid checking for newline differences at the
+  // end of a file, but actually it might be nice to actually add a newline
+  // when generating to be good computer citizens.
+  const actualContents = fs.readFileSync(actualFilePath, "utf8").trim()
+  const expectedContents = fs.readFileSync(expectedFilePath, "utf8").trim()
 
   let isDifferent: boolean = false
 
@@ -124,7 +127,7 @@ async function test() {
       stdout.write(`\x1b[38;2;0;255;0m[pass]\x1b[0m ${directory.name} \n`)
     } catch (err: unknown) {
       stdout.write(`\x1b[38;2;255;0;0m[fail]\x1b[0m ${directory.name} \n`)
-      break
+      throw err
     }
 
     for (const [name, time] of Object.entries(timings)) {