Merge pull request webpack#4042 from shubheksha/refactor-test-RawModule

sokra · web-flow · commit 0a6b452055a1 · 2017-02-28T12:56:51.000+01:00
refactor(ES6): RawModule.test.js
diff --git a/test/RawModule.test.js b/test/RawModule.test.js
@@ -1,61 +1,65 @@
-var RawModule = require("../lib/RawModule");
-var OriginalSource = require("webpack-sources").OriginalSource;
-var RawSource = require("webpack-sources").RawSource;
-var RequestShortener = require("../lib/RequestShortener");
-var should = require("should");
-var path = require("path")
-
-describe("RawModule", function() {
-	var myRawModule;
-
-	before(function() {
-		var source = 'sourceStr attribute';
-		var identifier = 'identifierStr attribute';
-		var readableIdentifier = 'readableIdentifierStr attribute';
+"use strict";
+
+const RawModule = require("../lib/RawModule");
+const OriginalSource = require("webpack-sources").OriginalSource;
+const RawSource = require("webpack-sources").RawSource;
+const RequestShortener = require("../lib/RequestShortener");
+const should = require("should");
+const path = require("path");
+
+describe("RawModule", () => {
+	let myRawModule;
+
+	before(() => {
+		const source = "sourceStr attribute";
+		const identifier = "identifierStr attribute";
+		const readableIdentifier = "readableIdentifierStr attribute";
 		myRawModule = new RawModule(source, identifier, readableIdentifier);
 	});
 
-	describe('identifier', function() {
-		it('returns value for identifierStr attribute', function() {
-			should(myRawModule.identifier()).be.exactly('identifierStr attribute');
-		});
+	describe("identifier", () => {
+		it("returns value for identifierStr attribute", () =>
+			should(myRawModule.identifier()).be.exactly("identifierStr attribute"));
 	});
 
-	describe('size', function() {
-		it('returns value for sourceStr attribute\'s length property', function() {
-			var sourceStrLength = myRawModule.sourceStr.length;
+	describe("size", () => {
+		it("returns value for sourceStr attribute\"s length property", () => {
+			const sourceStrLength = myRawModule.sourceStr.length;
 			should(myRawModule.size()).be.exactly(sourceStrLength);
 		});
 	});
 
-	describe('readableIdentifier', function() {
-		it('returns result of calling provided requestShortener\'s shorten method\
-     on readableIdentifierStr attribute', function() {
-			var requestShortener = new RequestShortener(path.resolve());
-			should.exist(myRawModule.readableIdentifier(requestShortener));
-		});
+	describe("readableIdentifier", () => {
+		it("returns result of calling provided requestShortener\"s shorten method " +
+			"on readableIdentifierStr attribute",
+			() => {
+				const requestShortener = new RequestShortener(path.resolve());
+				should.exist(myRawModule.readableIdentifier(requestShortener));
+			}
+		);
 	});
 
-	describe('needRebuild', function() {
-		it('returns false', function() {
-			should(myRawModule.needRebuild()).be.false();
-		});
+	describe("needRebuild", () => {
+		it("returns false", () => should(myRawModule.needRebuild()).be.false());
 	});
 
-	describe('source', function() {
-		it('returns a new OriginalSource instance with sourceStr attribute and\
-        return value of identifier() function provided as constructor arguments',
-			function() {
-				var originalSource = new OriginalSource(myRawModule.sourceStr, myRawModule.identifier());
+	describe("source", () => {
+		it("returns a new OriginalSource instance with sourceStr attribute and " +
+			"return value of identifier() function provided as constructor arguments",
+			() => {
+				const originalSource = new OriginalSource(myRawModule.sourceStr, myRawModule.identifier());
 				myRawModule.useSourceMap = true;
 				myRawModule.source().should.match(originalSource);
-			});
+			}
+		);
 
-		it('returns a new RawSource instance with sourceStr attribute provided\
-        as constructor argument if useSourceMap is falsey', function() {
-			var rawSource = new RawSource(myRawModule.sourceStr);
-			myRawModule.useSourceMap = false;
-			myRawModule.source().should.match(rawSource);
-		});
+		it("returns a new RawSource instance with sourceStr attribute provided " +
+			"as constructor argument if useSourceMap is falsey",
+			() => {
+				const rawSource = new RawSource(myRawModule.sourceStr);
+				myRawModule.useSourceMap = false;
+				myRawModule.source().should.match(rawSource);
+			}
+		);
 	});
 });
