Bug #21811
open
developer-run-tests-services-workbench2: #889 ![](https://ci.arvados.org/buildStatus/icon?job=developer-run-tests-services-workbench2&build=889)
21811-side-favorites-test @ arvados|7223714878b7ff520c58261d12e0b3e9f5aa5c8d
- ✅ All agreed upon points are implemented / addressed.
- ✅ Anything not implemented (discovered or discussed during work) has a follow-up story.
- ✅ Code is tested and passing, both automated and manual, what manual testing was done is described
no manual testing was done
- n/a Documentation has been updated.
- ✅ Behaves appropriately at the intended scale (describe intended scale).
- n/a Considered backwards and forwards compatibility issues between client and server.
- ✅ Follows our coding standards and GUI style guidelines.
Notes:
- The new test runs twice because that is what would have caught the bug resolved in #21712. Also fixed a failing banner test that had previously been pushed to main.
- Status changed from New to In Progress
- Target version set to Development 2024-06-19 sprint
- Target version changed from Development 2024-06-19 sprint to Development 2024-07-03 sprint
- Target version changed from Development 2024-07-03 sprint to Development 2024-07-24 sprint
Sorry for not looking into this earlier. In the future feel free to remind me of pending reviews at standup regardless of the review task setting.
I think the assertions could be more thorough, for example asserting that "myFavoriteProjectN" doesn't exist before expanding "My favorites" and also after collapsing it, and the same with public favorites. That way, we make sure all the behaviors are well tested.
Apart from that, the rest LGTM, thanks!
Also available in: Atom
PDF