employer
/
liza
1
0
Fork 0
Code Releases Activity

FieldVisibilityEventHandler: Use new DataValidator API

master
Mike Gerwitz 2017-02-10 13:30:53 -05:00
parent 3ea4efb732 cc79f8e1f3
commit 530b92fd33
2 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/event/FieldVisibilityEventHandler.js
View File

@ -103,7 +103,7 @@ module.exports = Class( 'FieldVisibilityEventHandler' )
} }
} )(); } )();
this._data_validator.clearFailures( [ field_name ] ); this._data_validator.clearFailures( { [field_name]: indexes } );
indexes.forEach( field_i => action( field_name, field_i ) ); indexes.forEach( field_i => action( field_name, field_i ) );
callback(); callback();

7
test/event/FieldVisibilityEventHandlerTest.js
View File

@ -96,11 +96,12 @@ describe( 'FieldVisibilityEventHandler', () =>
it( 'clears failures on hidden fields', done => it( 'clears failures on hidden fields', done =>
{ {
const name = 'foo_bar'; const name = 'foo_bar';
const fail_indexes = [ 0, 3 ];
const hide_data = { const hide_data = {
elementName: name, elementName: name,
indexes: [ 0 ], indexes: fail_indexes,
}; };
Sut( Sut(
@ -108,7 +109,7 @@ describe( 'FieldVisibilityEventHandler', () =>
createStubDataProvider( failures => createStubDataProvider( failures =>
{ {
expect( failures ) expect( failures )
.to.deep.equal( [ name ] ) .to.deep.equal( { [name]: fail_indexes } );
// we don't care about the rest of the processing at this // we don't care about the rest of the processing at this
// point // point