Yoav Kleinberger has posted comments on this change.
Change subject: tests: introducing whitebox tests
......................................................................
Patch Set 2:
(1 comment)
http://gerrit.ovirt.org/#/c/29745/2//COMMIT_MSG
Commit Message:
Line 18: because they are aware of the inside implementation (as opposed to
Line 19: blackbox tests), but they check that things "really work" (e.g.
Line 20: libraries get called, sockets are opened, etc.) , and not just
Line 21: the internal logic of some small section of code (which would
Line 22: characterize a unit test).
you can put README within test/functional with set of variable to
enable/di
why would I put a non-functional test in a dir called
'functional', and then use a README to compensate for this confusion
Line 23:
Line 24: Change-Id: I4e20d17c3ebee1203bb5a721ce44d5867570ce8e
--
To view, visit
http://gerrit.ovirt.org/29745
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I4e20d17c3ebee1203bb5a721ce44d5867570ce8e
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yoav Kleinberger <ykleinbe(a)redhat.com>
Gerrit-Reviewer: Adam Litke <alitke(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com>
Gerrit-Reviewer: Yoav Kleinberger <ykleinbe(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes