Skip to content
Permalink
Browse files

[FIX] note: display date picker in activity menu

The date picker should be displayed when clicking on the fa-calendar
when creating a note from activity systray.

The test was missing the assertion checking that the date picker was displayed,
but fixing this issue breaks the test. The datepicker looks to be
asynchronously initialized, and the blur event raises an error because of
that. Since it is difficult to test external libs behaviour, this
step is simply removed from the test.

Task: 1942850
  • Loading branch information...
Xavier-Do committed Feb 19, 2019
1 parent 976b65c commit 5aa3a679ec2a594438ff1653a5e250d4b6455cd0
@@ -118,7 +118,7 @@ ActivityMenu.include({
_onNoteDateTimeSetClick: function (ev) {
ev.preventDefault();
ev.stopPropagation();
this.noteDateTimeWidget.$input.click();
this.noteDateTimeWidget.$input.select();
},
/**
* Saving note (quick create) and updating activity preview
@@ -108,7 +108,6 @@ QUnit.test('note activity menu widget: create note from activity menu', function
// creating quick note with date
activityMenu.$('.o_note_show').click();
activityMenu.$('input.o_note_input').val("New Note");
activityMenu.$('.o_note_set_datetime').click();
activityMenu.$(".o_note_save").click();
assert.strictEqual(activityMenu.$('.o_notification_counter').text(), '2',
"should increment activity notification counter after creating a second note");

0 comments on commit 5aa3a67

Please sign in to comment.
You can’t perform that action at this time.