Removed required loader argument in schema and updated unit tests

Author: scottdj92

schema/schema.json

@@ -16,7 +16,7 @@
       "type": "string"
     },
     "filename": {
-      "description": "The filename and path that ExtractTextPlugin will extract to.",
+      "description": "The filename and path that ExtractTextPlugin will extract to",
       "type": "string"
     },
     "loader": {
@@ -27,6 +27,5 @@
       "description": "",
       "type": "string"
     }
-  },
-  "required": ["loader"]
+  }
 }

test/extract.test.js

@@ -11,9 +11,9 @@ describe("ExtractTextPlugin.extract()", function() {
 	});
 
 	context("json schema validation", function() {
-		it("throws if an incorrect config is passed in", function() {
-			should.throws(function() {
-				ExtractTextPlugin.extract(['style-loader', 'file.css']);
+		it("does not throw if a filename is specified", function() {
+			should.doesNotThrow(function() {
+				ExtractTextPlugin.extract("file.css");
 			});
 		});
 
@@ -23,9 +23,9 @@ describe("ExtractTextPlugin.extract()", function() {
 			});
 		});
 
-		it("does not throw if a filename is specified", function() {
-			should.doesNotThrow(function() {
-				ExtractTextPlugin.extract("file.css");
+		it("throws if an incorrect config is passed in", function() {
+			should.throws(function() {
+				ExtractTextPlugin.extract({style: 'file.css'});
 			});
 		});
 	});