namoscato / angular-jasmine-boilerplate Goto Github PK

This sort of thing does not match:

myService.fetch(getMyArgument()).then(function() {
    ...
});

Should have a mock object instead of creating a spy for a factory.

• Sublime
• Atom

Screenshot below.

SSIA.

Basically I ran /fileName.js and it still ran everything, even though what I needed to run was fileName.js.

There should be error handling for this eventually. 😄

i.e.

myService.method().then(function() { ... });

should create:

var myServiceSpy = jasmine.createSpyObj('myService', ['method']);

var myServiceMethodSpy = jasmine.createSpyObj('myService.method', ['then'])
myServiceSpy.method.and.returnValue(myServiceMethodSpy);

Currently this is only added to the locals object when the controller also requires $scope.

Ideally "saves" would be "saving"...

Run reportService.js through. In several cases, in the provide, the return statement will be on the same line as another.

    $provide.service('userService', function() {
        userServiceSpy = jasmine.createSpyObj('userService', ['']);return userServiceSpy;
    });

I have not tested controller template for this issue