无法获得业力测试传递到控制器上

Question

我试图在角度js中创建一个“待办事项列表”。完成任务后，我希望能够点击复选框标记完成。虽然它正在处理index.html，但我无法通过测试。

这是我的测试：

describe('ToDoListController', function() { 
    beforeEach(module('ToDoList')); 

    var ctrl; 

    beforeEach(inject(function($controller) { 
    ctrl = $controller('ToDoListController'); 
    })); 

    it('initialises with an empty list and term', function() { 
    expect(ctrl.listDisplay).toEqual([]); 
    expect(ctrl.taskTerm).toBeUndefined(); 
    }); 

    describe('when viewing the to do list', function(){ 

    it('displays list items', function() { 
     ctrl.taskTerm = "hello"; 
     ctrl.addTask(); 
     expect(ctrl.listDisplay[0].task).toBe("hello"); 
    }); 

    it('is completed when clicked', function() { 
     ctrl.taskTerm = "hello"; 
     ctrl.addTask(); 
     ctrl.isCompleted("hello"); 
     expect(ctrl.listDisplay[0].completed).toBe(true) 
    }); 

    }); 
}); 

，这是我的控制器：

toDoList.controller('ToDoListController', [function() { 

    var self = this; 
    self.listDisplay = [] 

    self.addTask = function() { 
    self.listDisplay.push({task: self.taskTerm, completed: false}) 
    }; 

    self.isCompleted = function(item) { 
    var i = self.listDisplay.indexOf(item) 
    self.listDisplay[i].completed = true 
    } 
}]); 

错误我得到的回复是：

TypeError: Cannot set property 'completed' of undefined 
    at self.isCompleted (/Users/edobrien/Documents/Projects/todo_challenge/js/toDoListController.js:13:35) 
    at Object.<anonymous> (/Users/edobrien/Documents/Projects/todo_challenge/test/toDoListController.spec.js:26:12) 

仅供参考，错误发生在“ctrl.isCompleted（”hello“）;”行

任何帮助你们可以给予将不胜感激！

Answer 1

你的控制器的isCompleted方法是错误的：你的列表是对象的数组，没有字符串：

self.addTask = function() { 
    self.listDisplay.push({task: self.taskTerm, completed: false}) 
}; 

如果设置taskTerm如“你好”，即推入数组的对象不是字符串“你好”，而是像

{task: "hello", completed: false} 

对象所以，当你尝试的indexOf（“你好”），找到它，它总是会返回-1，因为

"hello" != {task: "hello", completed: false}

更改您在数组中查找的对象的方式：

self.isCompleted = function(item) { 
    var completed=false; 
    self.listDisplay.forEach(function (value,index) { 
     if (value.task ==item) { 
      completed=value.completed; 
     } 
    }); 
    return completed; 
}; 

更新：我读你的代码时犯了一个错误：isCompleted是一个方法的名称通常是检查如果某事......好吧，如果有事情完成了。但实际上你的方法设置它完成。这种方法通常被命名为setCompleted。

所以方法应该是这样的：

self.setCompleted = function(item) { 
    self.listDisplay.forEach(function (obj,index) { 
     if (obj.task ==item) { 
      obj.completed=true; //setting it as completed 
     } 
    }); 
};