test(ivy): fixes in the TodoMVC example (#23161)

- properly display initial checked state - properly remove a todo Please note that the 'archive' option still doesn't work correctly as listening to component outputs doesn't seem to work (onArchive() is never called). PR Close #23161
angular · Apr 4, 2018 · 32a41bc · alexzuza · Apr 4, 2018 · 32a41bc
1 parent f99cb5c
commit 32a41bc
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/packages/core/test/bundling/todo/index.ts b/packages/core/test/bundling/todo/index.ts
@@ -33,7 +33,7 @@ export class AppState {
   // but NgForOf expects creation and binding separate.
   template: `
     <div>
-      <input type="checkbox" [value]="todo && todo.done" (click)="onCheckboxClick()">&ngsp;
+      <input type="checkbox" [checked]="todo && todo.done" (change)="onCheckboxClick()">&ngsp;
       <span [class.done]="todo && todo.done">{{todo && todo.text}}</span>&ngsp;
       <button (click)="onArchiveClick()">archive</button>
     </div>
@@ -71,7 +71,7 @@ export class ToDoAppComponent {
 
   onArchive(item: ToDo) {
     const todos = this.appState.todos;
-    todos.splice(todos.indexOf(item));
+    todos.splice(todos.indexOf(item), 1);
     markDirty(this);
 export function wrapListenerWithDirtyAndDefault( 
     view: LView, listenerFn: (e?: any) => any): EventListener { 
   return function(e: Event) { 
     markViewDirty(view); 
 export function wrapListenerWithDirtyAndDefault( 
     view: LView, listenerFn: (e?: any) => any): EventListener { 
   return function(e: Event) { 
     markViewDirty(view); 
   }
 }
@@ -108,4 +108,4 @@ export class ToDoAppModule {
 renderComponent(ToDoAppComponent, {
   // TODO(misko): This should run without injector.
   injector: createInjector(ToDoAppModule)
-});
+});