[akriukas]

I don't get what these "done()" mean. But otherwise, the logic looks good.

[webrobots]

done() is specific to WR system. Essentially we do not know when a step is finished because it may do waiting from DOM element or Ajax requests. So a developer must place done() where he/she is sure that a step has completed.