[rfc][icedtea-web][policyeditor] Unit tests fix after last PolicyEditor async IO patch

Omair Majid omajid at redhat.com
Thu Jun 5 19:54:38 UTC 2014


* Andrew Azores <aazores at redhat.com> [2014-06-03 17:59]:
> I don't think I'm really on board with adding that kind of additional
> complexity to it just so that this one method can be tested "better".

Fair enough.

> As above, it already does this. The reason the tests sporadically fail
> without this patch is because the parsing didn't happen (on time).

But that's my exact concern. The more asynchronous the system under
test, the less guarantee you have about what it's doing. You *know*,
that the code under test is not doing some things, but the tests can not
prove that, because it's happening in a way the test can not look into.

Thanks,
Omair

-- 
PGP Key: 66484681 (http://pgp.mit.edu/)
Fingerprint = F072 555B 0A17 3957 4E95  0056 F286 F14F 6648 4681


More information about the distro-pkg-dev mailing list