From e384bf8cbbdfa7791d1ef750c9bb8e96f9f12eab Mon Sep 17 00:00:00 2001
From: Diogo Resende <dresende@thinkdigital.pt>
Date: Mon, 5 Feb 2024 14:31:27 +0000
Subject: [PATCH] ci: improves test coverage

---
 test/Features.js                | 24 ++++++++++++++++++------
 test/views/complex.html         |  9 +++++++++
 test/views/has-bad-include.html |  3 +++
 3 files changed, 30 insertions(+), 6 deletions(-)
 create mode 100644 test/views/complex.html
 create mode 100644 test/views/has-bad-include.html

diff --git a/test/Features.js b/test/Features.js
index 6a37e49..bd811af 100644
--- a/test/Features.js
+++ b/test/Features.js
@@ -1,31 +1,43 @@
 import { Renderer } from "../index.js"
 import should       from "should"
 
-const keikan = new Renderer();
+const keikan = new Renderer({ debug: true });
 
 describe("Features", () => {
 	it("<% include %> can be used to load another view", async () => {
 		const view = await keikan.compilePath(import.meta.dirname + "/views/has-include");
 
-		view({ name: "world" }).should.equal("<h3>Hello world\n</h3>");
+		view({ name: "world" }).should.equal("<h3>\n\tHello world\n</h3>");
+	});
+
+	it("<% include %> will return an <error/> if not found", async () => {
+		const view = await keikan.compilePath(import.meta.dirname + "/views/has-bad-include");
+
+		view().should.equal("<h3>\n\t<error>Include not found: notfound</error>\n</h3>");
 	});
 
 	it("<%= %>, <%- %> and <%# %>", async () => {
 		const view = await keikan.compilePath(import.meta.dirname + "/views/escaping");
 
-		view().should.equal("<ul><li>This is a &quot;quoted&quot; string</li><li>This is not a \"quoted\" string</li><li></li></ul>");
+		view().should.equal("<ul>\n\t<li>This is a &quot;quoted&quot; string</li>\n\t<li>This is not a \"quoted\" string</li>\n\t<li></li>\n</ul>");
 	});
 
 	it("supports code like conditions, loops, ..", async () => {
 		const view = await keikan.compilePath(import.meta.dirname + "/views/condition");
 
-		view({ condition: true }).should.equal("<h3>Hello\n</h3>");
-		view({ condition: false }).should.equal("<h3></h3>");
+		view({ condition: true }).should.equal("<h3>\n\tHello\n</h3>");
+		view({ condition: false }).should.equal("<h3>\n</h3>");
 	});
 
 	it("removes unnecessary whitespace", async () => {
 		const view = await keikan.compilePath(import.meta.dirname + "/views/long");
 
-		view().should.equal("<h3><strong>Hello\n</strong></h3>");
+		view().should.equal("<h3>\n\t<strong>\n\t\tHello\n\t</strong>\n</h3>");
+	});
+
+	it("handles more complex views", async () => {
+		const view = await keikan.compilePath(import.meta.dirname + "/views/complex");
+
+		view().should.equal("<ul>\n\t<li>(odd) 1. 1</li>\n\t<li>(even) 2. 2</li>\n\t<li>(odd) 3. 3</li>\n</ul>");
 	});
 });
diff --git a/test/views/complex.html b/test/views/complex.html
new file mode 100644
index 0000000..9853ed4
--- /dev/null
+++ b/test/views/complex.html
@@ -0,0 +1,9 @@
+<ul>
+<% [ 1, 2, 3 ].map(i => { %>
+	<% if (i % 2 == 0) { %>
+	<li>(even) <%= i %>. <%- i %></li>
+	<% } else { %>
+	<li>(odd) <%= i %>. <%- i %></li>
+	<% } %>
+<% }) %>
+</ul>
diff --git a/test/views/has-bad-include.html b/test/views/has-bad-include.html
new file mode 100644
index 0000000..801589c
--- /dev/null
+++ b/test/views/has-bad-include.html
@@ -0,0 +1,3 @@
+<h3>
+	<% include notfound({ name }) %>
+</h3>