Add one line tests for instructor model spec updating

Author: goldlilys

client/spa/js/instructor/editInstructor.html

@@ -13,8 +13,8 @@
       <input class="textfield" id="firstName" value="<%- firstName %>" /><br/>
       <input class="textfield" id="lastName" value="<%- lastName %>" /><br/>
       <input class="textfield" id="skills" value="<%- skills %>" /><br/>
-      <button class="save">Save</button>
-      <button class="cancel">Cancel</button>
+      <button class="i-save">Save</button>
+      <button class="i-cancel">Cancel</button>
     </form>
   </div>
 </section>

client/spa/js/instructor/instructor.html

@@ -14,8 +14,8 @@
   <div class="main container">
     <h1><%- firstName %> <%- lastName %></h1>
     <p><%- skills %></p>
-    <button class="edit">Edit</button>
-    <button class="delete">Delete</button>
+    <button class="i-edit">Edit</button>
+    <button class="i-delete">Delete</button>
     <div id="result"></div>
   </div>
 </section>

client/spa/js/instructor/instructor.view.js

@@ -12,10 +12,10 @@ module.exports = Backbone.View.extend({
   template: _.template(template),
   editTemplate: _.template(editTemplate),
   events: {
-    'click .delete': 'destroy',
-    'click .edit': 'edit',
-    'click .save': 'save',
-    'click .cancel': 'cancel'
+    'click .i-delete': 'destroy',
+    'click .i-edit': 'edit',
+    'click .i-save': 'save',
+    'click .i-cancel': 'cancel'
   },
   initialize: function(){
     this.listenTo(this.model, 'destroy', this.remove);

client/spa/js/instructor/spec/instructor.model.spec.js

@@ -84,16 +84,14 @@ describe('Instructor model ', function(){
     });
 
     it('does not save when firstName and lastName are empty ', function(){
-      model.set('firstName', null);
-      model.set('lastName', null);
+      model.set({firstName:null, lastName:null});
       model.save();
       expect(model.validationError).toEqual(['firstName cannot be empty',
                                              'lastName cannot be empty']);
     });
 
     it('does not save when firstName and skills are empty ', function(){
-      model.set('firstName', null);
-      model.set('skills', null);
+      model.set({firstName:null, skills:null});
       model.save();
       expect(model.validationError).toEqual(['firstName cannot be empty',
                                              'skills cannot be empty']);
@@ -106,8 +104,7 @@ describe('Instructor model ', function(){
     });
 
     it('does not save when lastName and skills are empty ', function(){
-      model.set('lastName', null);
-      model.set('skills', null);
+      model.set({lastName:null, skills:null});
       model.save();
       expect(model.validationError).toEqual(['lastName cannot be empty',
                                              'skills cannot be empty']);
@@ -120,9 +117,7 @@ describe('Instructor model ', function(){
     });
 
     it('does not save when all fields are empty ', function(){
-      model.set('firstName', null);
-      model.set('lastName', null);
-      model.set('skills', null);
+      model.set({firstName:null, lastName:null, skills:null});
       model.save();
       expect(model.validationError).toEqual(['firstName cannot be empty',
                                              'lastName cannot be empty',

client/spa/js/instructor/spec/instructor.view.spec.js

@@ -69,15 +69,15 @@ describe('Instructor view ', function(){
       // call delegate after spyOn
       view.delegateEvents();
       view.render();
-      view.$('.edit').trigger('click');
+      view.$('.i-edit').trigger('click');
     });
 
     describe('when the user enters new instructor information ', function(){
 
       describe('when user clicks on the cancel button', function(){
 
         beforeEach(function(){
-          view.$('.cancel').trigger('click');
+          view.$('.i-cancel').trigger('click');
         });
 
         it('cancels the user input', function(){
@@ -91,7 +91,7 @@ describe('Instructor view ', function(){
           view.$('#lastName').val('changed lastName');
           view.$('#skills').val('changed skills');
 
-          view.$('.save').trigger('click');
+          view.$('.i-save').trigger('click');
         });
 
         it('updates the model', function(){
@@ -120,7 +120,7 @@ describe('Instructor view ', function(){
     it('deletes the model', function(){
       // Must render for the event to be fired
       view.render();
-      view.$('.delete').trigger('click');
+      view.$('.i-delete').trigger('click');
       expect(view.destroy).toHaveBeenCalled();
       expect(model.destroy).toHaveBeenCalled();
     });  // end delete model test