Correct linting errors

Author: SimenB

index.js

@@ -33,11 +33,11 @@ function gulpRename(obj) {
 
 		} else if (type === "object" && obj !== undefined && obj !== null) {
 
-			var dirname = 'dirname' in obj ? obj.dirname : parsedPath.dirname,
+			var dirname = "dirname" in obj ? obj.dirname : parsedPath.dirname,
 				prefix = obj.prefix || "",
 				suffix = obj.suffix || "",
-				basename = 'basename' in obj ? obj.basename : parsedPath.basename,
-				extname = 'extname' in obj ? obj.extname : parsedPath.extname;
+				basename = "basename" in obj ? obj.basename : parsedPath.basename,
+				extname = "extname" in obj ? obj.extname : parsedPath.extname;
 
 			path = Path.join(dirname, prefix + basename + suffix + extname);
 
@@ -54,9 +54,9 @@ function gulpRename(obj) {
 		}
 
 		callback(null, file);
-	}
+	};
 
 	return stream;
-};
+}
 
 module.exports = gulpRename;

test/path-parsing.spec.js

@@ -26,39 +26,39 @@ describe("gulp-rename path parsing", function () {
 			});
 		});
 
-		var dirname_helper = function (srcPattern, expectedPath) {
+		var dirnameHelper = function (srcPattern) {
 			it("dirname is path from directory glob to file", function (done) {
 				var obj = function (path) {
 					path.dirname.should.match(/^fixtures[0-9]?$/);
 				};
 				helper(srcPattern, obj, null, done);
 			});
-		}
+		};
 
 		context("when src pattern matches a directory with *", function () {
-			dirname_helper("test/*/*.min.txt");
+			dirnameHelper("test/*/*.min.txt");
 		});
 
 		context("when src pattern matches a directory with **", function () {
-			dirname_helper("test/**/*.min.txt");
+			dirnameHelper("test/**/*.min.txt");
 		});
 
 		context("when src pattern matches a directory with [...]", function () {
-			dirname_helper("test/fixt[a-z]res/*.min.txt");
+			dirnameHelper("test/fixt[a-z]res/*.min.txt");
 		});
 
 		/* SKIP: glob2base does not handle brace expansion as expected. See wearefractal/glob2base#1 */
 		context.skip("when src pattern matches a directory with {...,...}", function () {
-			dirname_helper("test/f{ri,ixtur}es/*.min.txt");
+			dirnameHelper("test/f{ri,ixtur}es/*.min.txt");
 		});
 
 		/* SKIP: glob2base does not handle brace expansion as expected. See wearefractal/glob2base#1 */
 		context.skip("when src pattern matches a directory with {#..#}", function () {
-			dirname_helper("test/fixtures{0..9}/*.min.txt");
+			dirnameHelper("test/fixtures{0..9}/*.min.txt");
 		});
 
 		context("when src pattern matches a directory with an extglob", function () {
-			dirname_helper("test/f+(ri|ixtur)es/*.min.txt");
+			dirnameHelper("test/f+(ri|ixtur)es/*.min.txt");
 		});
 
 		/* requires glob-stream >= 3.1.0 */

test/rename.spec.js

@@ -165,7 +165,7 @@ describe("gulp-rename", function () {
 		it("receives object with extname even if a different value is returned", function (done) {
 			var obj = function (path) {
 				path.extname.should.equal(".txt");
-				return path.extname = ".md";
+				path.extname = ".md";
 			};
 			var expectedPath = "test/fixtures/hello.md";
 			helper(srcPattern, obj, expectedPath, done);