Toggle graphs in the training board olena#825
Conversation
tharvik
left a comment
There was a problem hiding this comment.
thanks for the work! a few comments on how to improve it, please also fix the tests/lint found by the CI
|
|
||
| expect(wrapper.findAll("#mapHeader li")).toHaveLength(3); | ||
| const listItems = wrapper.findAll("#mapHeader li"); | ||
| expect(listItems.length === 0 || listItems.length === 3).toBe(true); |
There was a problem hiding this comment.
Only one length should be expected (is the advanced information open or closed by default?).
Can you break this down into two assertions: one when the length is expected to be 0 and another when the length should be 3? You can use cypress' .click() to toggle the advanced information.
There was a problem hiding this comment.
This test is designed to validate the Training Logs section, which has been removed. Therefore, the test should only pass when the length is 0. However, does it make sense to keep this test at all, considering there is essentially nothing left for it to verify?
There was a problem hiding this comment.
Indeed you can remove it altogether!
|
Hey @iamsherlocked1891, could you wait until Dec 5th to merge this PR? There a DISCO demo happening tomorrow so it would be safer to wait until it's past :) |
3ec8015 to
b807d92
Compare
Client-side
Additions & Modifications:
Enhanced DropdownCard Component:
Refinements in the Training Information Section:
Removed Training Logs from Training Information: