diff options
| author | Eric Eastwood <contact@ericeastwood.com> | 2017-06-19 18:43:08 -0500 |
|---|---|---|
| committer | Eric Eastwood <contact@ericeastwood.com> | 2017-06-19 18:52:19 -0500 |
| commit | 19e12aac0217efae6b474d8601928a217064fb21 (patch) | |
| tree | 8982e2c5cac148330a5bb5eb24e30b7f954eb847 /spec/javascripts/notes_spec.js | |
| parent | 025cbc2ad4dc175634676ff8a6955e043512d8bf (diff) | |
| download | gitlab-ce-19e12aac0217efae6b474d8601928a217064fb21.tar.gz | |
Fix note highlight being added to new notes33916-make-note-highlight-toggle-boolean-explcit
Always use a explicit true/false value to set highlight because jQuery
assumes `null` means toggle from whatever state and doesn't force false.
Regressed in
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/12098
Diffstat (limited to 'spec/javascripts/notes_spec.js')
| -rw-r--r-- | spec/javascripts/notes_spec.js | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/spec/javascripts/notes_spec.js b/spec/javascripts/notes_spec.js index b272f43a166..5ece4ed080b 100644 --- a/spec/javascripts/notes_spec.js +++ b/spec/javascripts/notes_spec.js @@ -176,7 +176,7 @@ import '~/notes'; Notes.updateNoteTargetSelector($note); - expect($note.toggleClass).toHaveBeenCalledWith('target', null); + expect($note.toggleClass).toHaveBeenCalledWith('target', false); }); }); |