Merge pull request #63 from fabien-d/unitTests

added log/extend tests

lib/alertify.js

@@ -286,7 +286,11 @@
 			 * @return {Function}
 			 */
 			extend : function (type) {
-				return function (message, wait) { this.log(message, type, wait); };
+				if (typeof type !== "string") throw new Error("extend method must have exactly one paramter");
+				return function (message, wait) {
+					this.log(message, type, wait);
+					return this;
+				};
 			},
 
 			/**

src/alertify.js

@@ -274,7 +274,11 @@
 			 * @return {Function}
 			 */
 			extend : function (type) {
-				return function (message, wait) { this.log(message, type, wait); };
+				if (typeof type !== "string") throw new Error("extend method must have exactly one paramter");
+				return function (message, wait) {
+					this.log(message, type, wait);
+					return this;
+				};
 			},
 
 			/**

test/index.html

@@ -15,5 +15,6 @@
 	<script src="specs/alert.test.js"></script>
 	<script src="specs/confirm.test.js"></script>
 	<script src="specs/prompt.test.js"></script>
+	<script src="specs/log.test.js"></script>
 </body>
 </html>

test/specs/global.test.js

@@ -16,8 +16,6 @@ test("API options", function () {
 	deepEqual(typeof alertify.error, "function", "error notification part of the API");
 	// options
 	deepEqual(typeof alertify.set, "function", "set method part of the API");
-	//deepEqual(typeof alertify.labels, "object", "labels object part of the API");
-	//deepEqual(typeof alertify.delay, "number", "delay value part of the API");
 });
 
 module("custom labels", {

test/specs/log.test.js

@@ -0,0 +1,21 @@
+module("log", {
+	setup : function () {
+		this.log = alertify.log("Test");
+	}
+});
+
+test("log returns alertify object", function () {
+	expect(1);
+	deepEqual(this.log, alertify, "should be equal");
+});
+
+test("extend method", function () {
+	expect(2);
+	try {
+		alertify.extend();
+	} catch (error) {
+		deepEqual(error.message, "extend method must have exactly one paramter", "parameter error caught");
+	}
+	alertify.custom = alertify.extend("custom");
+	deepEqual(alertify.custom("test"), alertify, "should be equal");
+});