Alon Bar-Lev 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).
I assume you meant its own directory.
you can put README
within test/functional with set of variable to enable/disable to gain functionality X
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