<div dir="ltr"><div>Hi folks!</div><div><br></div><div>Some tests in murano code use incorrect order of arguments in assertEqual.</div><div>The correct order expected by the testtools is</div><div><br></div><div>        def assertEqual(self, expected, observed, message=''):</div><div>            """Assert that 'expected' is equal to 'observed'.</div><div><br></div><div>            :param expected: The expected value.</div><div>            :param observed: The observed value.</div><div>            :param message: An optional message to include in the error.</div><div>            """</div><div><br></div><div>Error message has the following format:</div><div><br></div><div>raise mismatch_error</div><div>    testtools.matchers._impl.MismatchError: !=:</div><div>    reference = <expected value></div><div>    actual    = <observed value></div><div><br></div><div>Use of arguments in incorrect order could make debug output very confusing.</div><div>Let's fix it to make debugging easier.</div><div><br></div><div>Best regards,</div><div>Tetiana Lashchova</div></div>