-
-
Notifications
You must be signed in to change notification settings - Fork 4.5k
Closed
Labels
Description
FAIL tests/e2e/modules/calendar_spec.js (77.317 s)
● Calendar module › Events from multiple calendars › should show multiple events with the same title and start time from different calendars
expect(received).resolves.toBe()
Received promise rejected instead of resolved
Rejected to value: [Error: expect(received).toHaveLength(expected)·
Expected length: 22
Received length: 20
Received object: [<tr class="event-wrapper normal event"><td class="symbol align-right "><span class="fas fa-fw fa-calendar-alt" /></td><td class="title bright ">TestEvent</td><td class="time light ">Mar 9th</td></tr>, <tr class="event-wrapper normal event"><td class="symbol align-right "><span class="fas fa-fw fa-calendar-alt" /></td><td class="title bright ">TestEvent</td><td class="time light ">Mar 9th</td></tr>, <tr class="event-wrapper normal event"><td class="symbol align-right "><span class="fas fa-fw fa-calendar-alt" /></td><td class="title bright ">TestEvent</td><td class="time light ">Sep 9th</td></tr>, <tr class="event-wrapper normal event"><td class="symbol align-right "><span class="fas fa-fw fa-calendar-alt" /></td><td class="title bright ">TestEvent</td><td class="time light ">Sep 9th</td></tr>, <tr class="event-wrapper normal event"><td class="symbol align-right "><span class="fas fa-fw fa-calendar-alt" /></td><td class="title bright ">TestEvent</td><td class="time light ">Mar 9th</td></tr
96 |
97 | it("should show multiple events with the same title and start time from different calendars", async () => {
> 98 | await expect(testElementLength(".calendar .event", 22)).resolves.toBe(true);
| ^
99 | });
100 | });
101 |
at expect (node_modules/expect/build/index.js:113:15)
at Object.expect (tests/e2e/modules/calendar_spec.js:98:10)This seems to be a data problem. Running this test 10 days back the test succeeds.
Running the config and checking result in the browser results in 20 calendar entries.
Quickfix would be to change expected 22 to now 20. But I will try to dig deeper and find a real fix.