no-run-logic - keep run functions clean and simple

Initialization logic should be moved into a factory or service. This improves testability.

Rule based on Angular 1.x

Styleguide Reference

• y171 by johnpapa - Run Blocks

Examples

The following patterns are considered problems with default config;

/*eslint angular/no-run-logic: 2*/

// invalid
angular.module('app').run(function($window) {
    $window.addEventListener('deviceready', deviceready);

    function deviceready() {}
}); // error: The run function may only contain call expressions

The following patterns are not considered problems with default config;

/*eslint angular/no-run-logic: 2*/

// valid
angular.module('app').run(function(KITTENS, kittenService, startup) {
    kittenService.prefetchData(KITTENS);
    startup('foo', true, 1);
});

The following patterns are considered problems when configured {"allowParams":false}:

/*eslint angular/no-run-logic: [2,{"allowParams":false}]*/

// invalid
angular.module('app').run(function(startup) {
    startup('foo', true, 1);
}); // error: Run function call expressions may not take any arguments

The following patterns are not considered problems when configured {"allowParams":false}:

/*eslint angular/no-run-logic: [2,{"allowParams":false}]*/

// valid
angular.module('app').run(function(kittenService, startup) {
    kittenService.prefetchData();
    startup();
});

Version

This rule was introduced in eslint-plugin-angular 0.15.0

Links

• Rule source
• Example source