On Tue, Nov 12, 2013 at 1:54 PM, Klaus Aehlig <[email protected]> wrote: >> > op = OpTest() >> > - before = op.__getstate__() >> > op.Validate(True) >> > - self.assertNotEqual(op.__getstate__(), before, >> > - msg="Opcode was not modified") >> >> Why is this not true anymore? Can "Validate" modify the opcode, now? > > The removed test verified that the Validate method did change the > opcode. The change carried out was that for missing mandatory fields > the default value was added. However, this now happens already at > constructor time, so that there are no more changes to do for Validate.
Ok, thanks for the clarification. LGTM. Michele -- Google Germany GmbH Dienerstr. 12 80331 München Registergericht und -nummer: Hamburg, HRB 86891 Sitz der Gesellschaft: Hamburg Geschäftsführer: Graham Law, Christine Elizabeth Flores
