Skip to content

Commit a12c985

Browse files
patrick-vandyfnagel
authored andcommitted
Datepicker: Improve callbacks test for onUpdateDatepicker
Make sure the custom element added by the onUpdateDatepicker callback still exists and is not duplicated after calling refresh and setDate.
1 parent 17d115b commit a12c985

File tree

1 file changed

+7
-2
lines changed

1 file changed

+7
-2
lines changed

tests/unit/datepicker/options.js

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -839,7 +839,7 @@ function onUpdateDatepicker( inst ) {
839839
}
840840

841841
QUnit.test( "callbacks", function( assert ) {
842-
assert.expect( 16 );
842+
assert.expect( 18 );
843843

844844
// Before show
845845
var dp, day20, day21,
@@ -875,7 +875,12 @@ QUnit.test( "callbacks", function( assert ) {
875875
inp.val( "02/04/2008" ).datepicker( "show" );
876876
assert.ok( onUpdateDatepickerThis.id === inp[ 0 ].id, "On update datepicker - this OK" );
877877
assert.deepEqual( onUpdateDatepickerInst, inst, "On update datepicker - inst OK" );
878-
assert.ok( dp.find( "div.on-update-datepicker-test" ).length > 0, "On update datepicker - custom element" );
878+
assert.ok( dp.find( "div.on-update-datepicker-test" ).length === 1, "On update datepicker - custom element" );
879+
inp.datepicker( "setDate", "02/05/2008" );
880+
assert.ok( dp.find( "div.on-update-datepicker-test" ).length === 1, "On update datepicker - custom element after setDate" );
881+
inp.datepicker( "refresh" );
882+
assert.ok( dp.find( "div.on-update-datepicker-test" ).length === 1, "On update datepicker - custom element after refresh" );
883+
inp.datepicker( "hide" ).datepicker( "destroy" );
879884
} );
880885

881886
QUnit.test( "beforeShowDay - tooltips with quotes", function( assert ) {

0 commit comments

Comments
 (0)