<div dir="ltr">Hi all,<br>I am fixing assertions in unittests now (initially <a href="https://bugs.launchpad.net/heat/+bug/1259023">https://bugs.launchpad.net/heat/+bug/1259023</a>). I've seen that Clint have split this bug into three separate bugs, one per assertion misuse type. As these changes will affect many files (practically every file in the tests), I would like to ask the following:<div>

<br></div><div>- should I file other separate bugs for other assertion misuse types? (right now I also see ignorance of assertIsInstance)</div><div>- should I then commit changes in small patch-sets, may be one-two files at once?</div>

<div>- wouldn't it be better to keep all these changes in one bug and fix all misuses per file basis (with one file per patch-set for example)? It seems to me it would be easier to review in this way.</div><div><br></div>

<div>Best regards,</div><div>Pavlo Shchelokovskyy.</div></div>