Merge pull request #461 from tpeiffer/Issue-452/angular/file-name/casing/unit-tests

Issue 452/angular/file name/casing/unit tests

Author: EmmanuelDemey

rules/file-name.js

@@ -105,6 +105,12 @@ module.exports = {
                 if (typeSeparator !== undefined) {
                     name = name + typeSeparator + type;
                 }
+                if (options.casing === 'camel') {
+                    name = filenameUtil.firstToLower(name);
+                }
+                if (options.casing === 'pascal') {
+                    name = filenameUtil.firstToUpper(name);
+                }
                 return name + fileEnding;
             }
         };

test/file-name.js

@@ -227,6 +227,38 @@ angular.module(mod, [mod + '.core.angular', mod + '.thirdparty']);
             },
             typeSeparator: 'dot'
         }]
+    }, {
+        // camel casing, dot typeSeparator, ignoreTypeSuffix of true
+        filename: 'src/app/some.controller.js',
+        code: 'app.controller("SomeController", function() {});',
+        options: [{
+            casing: 'camel',
+            typeSeparator: 'dot',
+            ignoreTypeSuffix: true
+        }]
+    }, {
+        // camel casing
+        filename: 'src/app/someController.js',
+        code: 'app.controller("SomeController", function() {});',
+        options: [{
+            casing: 'camel'
+        }]
+    }, {
+        // pascal casing, dot typeSeparator, ignoreTypeSuffix of true
+        filename: 'src/app/Some.controller.js',
+        code: 'app.controller("SomeController", function() {});',
+        options: [{
+            casing: 'pascal',
+            typeSeparator: 'dot',
+            ignoreTypeSuffix: true
+        }]
+    }, {
+        // pascal casing
+        filename: 'src/app/SomeController.js',
+        code: 'app.controller("SomeController", function() {});',
+        options: [{
+            casing: 'pascal'
+        }]
     }].concat(commonFalsePositives),
     invalid: [{
         filename: 'src/app/filters.js',
@@ -322,5 +354,41 @@ angular.module(mod, [mod + '.core.angular', mod + '.thirdparty']);
             typeSeparator: 'dot'
         }],
         errors: [{message: 'Filename must be "users.provider.js"'}]
+    }, {
+        // camel casing, dot typeSeparator, ignoreTypeSuffix of true
+        filename: 'src/app/SomeController.js',
+        code: 'app.controller("SomeController", function() {});',
+        options: [{
+            casing: 'camel',
+            typeSeparator: 'dot',
+            ignoreTypeSuffix: true
+        }],
+        errors: [{message: 'Filename must be "some.controller.js"'}]
+    }, {
+        // camel casing
+        filename: 'src/app/SomeController.js',
+        code: 'app.controller("SomeController", function() {});',
+        options: [{
+            casing: 'camel'
+        }],
+        errors: [{message: 'Filename must be "someController.js"'}]
+    }, {
+        // pascal casing, dot typeSeparator, ignoreTypeSuffix of true
+        filename: 'src/app/someController.js',
+        code: 'app.controller("SomeController", function() {});',
+        options: [{
+            casing: 'pascal',
+            typeSeparator: 'dot',
+            ignoreTypeSuffix: true
+        }],
+        errors: [{message: 'Filename must be "Some.controller.js"'}]
+    }, {
+        // pascal casing
+        filename: 'src/app/someController.js',
+        code: 'app.controller("SomeController", function() {});',
+        options: [{
+            casing: 'pascal'
+        }],
+        errors: [{message: 'Filename must be "SomeController.js"'}]
     }]
 });