Testing details that have been known to break once in a while - like error messaging and accumulation of items as multiple are checked out, maybe state handling too (clearing state as user goes to another app)
The module's own suite only tests for error messages in ui-checkout, so I guess that's a bit thin (although it's been useful).
However, the cross-platform suite ('test-exercise' in ui-testing) checks out an item to a patron and looks up the status in the users loans afterwards - so we could say we're good with that for now.
Jakub Skoczen February 23, 2018 at 2:29 PM
Is this issue still relevant?
Niels Erik Nielsen August 24, 2017 at 7:31 AM
The branches (for having tests living in their respective ui module repos) have been merged, in ui-checkout and ui-testing.
Additional tests ought to be added: Testing that consecutive check-outs are accumulated to the list of items (something that we've seen breaking). This depends on though, about finding a way to obtain test data independently of other UI modules.
Testing details that have been known to break once in a while - like error messaging and accumulation of items as multiple are checked out, maybe state handling too (clearing state as user goes to another app)