Code review comment for lp:~agilebg/account-consolidation/7.0-fix-1334645-elbati

Revision history for this message
Leonardo Pistone (lepistone) wrote :

Hi Lorenzo,

I am still non convinced :)

This way the test is not actually testing a useful workflow, but just a "return True" workaround.

I'd be happier if the yaml test actually tested a realistic case. (or at least, clarify in the yaml file what's going on)

Thanks!

review: Needs Fixing

« Back to merge proposal