camelize error keys by default

This only makes sense - the error keys should match the attribute names

Author: Lee Richmond

src/util/validation-errors.ts

@@ -1,4 +1,5 @@
 import Model from '../model';
+import { camelize } from './string'
 
 export default class ValidationErrors {
   model: Model;
@@ -32,7 +33,13 @@ export default class ValidationErrors {
   }
 
   private _processResource(errorsAccumulator: object, meta: Object) {
-    errorsAccumulator[meta['attribute']] = meta['message'];
+    let attribute = meta['attribute']
+
+    if (this.model.klass.camelizeKeys) {
+      attribute = camelize(attribute)
+    }
+
+    errorsAccumulator[attribute] = meta['message'];
   }
 
   private _processRelationship(model: Model, meta: Object) {

test/integration/validations-test.ts

@@ -2,8 +2,6 @@ import { expect, sinon, fetchMock } from '../test-helper';
 import { Author, Book, Genre } from '../fixtures';
 import uuid from '../../src/util/uuid';
 
-let serverResponse;
-
 const resetMocks = function() {
   fetchMock.restore();
 
@@ -66,10 +64,6 @@ const resetMocks = function() {
       }
     }
   });
-
-  fetchMock.post('http://example.com/api/v1/people', function(url, payload) {
-    return serverResponse;
-  });
 }
 
 let instance;
@@ -97,19 +91,50 @@ describe('validations', function() {
     uuid.generate['restore']();
   });
 
-  // todo on next save, remove errs
   it('applies errors to the instance', function(done) {
     instance.save({ with: { books: 'genre' }}).then((success) => {
       expect(instance.isPersisted()).to.eq(false);
       expect(success).to.eq(false);
       expect(instance.errors).to.deep.equal({
-        first_name: 'cannot be blank',
-        last_name: 'cannot be blank'
+        firstName: 'cannot be blank',
+        lastName: 'cannot be blank'
       });
       done();
     });
   });
 
+  describe('when camelizeKeys is false', function() {
+    beforeEach(function() {
+      instance.klass.camelizeKeys = false
+    });
+
+    afterEach(function() {
+      instance.klass.camelizeKeys = true
+    });
+
+    it('does not camelize the error keys', function() {
+      instance.save({ with: { books: 'genre' }}).then((success) => {
+        expect(instance.errors).to.deep.equal({
+          first_name: 'cannot be blank',
+          last_name: 'cannot be blank'
+        });
+      });
+    });
+  });
+
+  it('clears errors on save', function(done) {
+    fetchMock.restore()
+    fetchMock.mock({
+      matcher: '*',
+      response: { data: { id: '1', type: 'employees'} }
+    });
+    instance.errors = { foo: 'bar' }
+    instance.save().then(() => {
+      expect(instance.errors).to.deep.eq({})
+      done()
+    });
+  });
+
   it('instantiates a new error object instance after save', function(done) {
     let originalErrors = instance.errors = {foo: 'bar'};
     let result = instance.save({ with: { books: 'genre' }});