Wenchao Xia has posted comments on this change.

Change subject: introducing test package to vdsm
......................................................................


Patch Set 4: (7 inline comments)

I am trying to adjust the test package as following:
tests
  |testrrunner.py
  |run_test.sh
  |validate.py
  --ut
  |  |-*.py
  |
  --vt
     |--rest
         |--*.py
     |--xmlrpc
         |--*.py

  Because vt case should be run after installation at target machine, so I want 
to class the test cases into different directories, and user can choose cases 
to run.

....................................................
File tests/testrunner.py
Line 156: def search_files(path, suffix):
function removed

Line 186:         print "running test suits %s..." % case_unit["name"]
Done

Line 220:     parser.add_argument('--case', help='''specify a case or a 
directory
using nose searching capability now, but still keeps case option for easying 
the coding.

Line 225:     inputArgs = sys.argv[1:]
argparser got error if arg[0]is given

Line 226:     args = parser.parse_args(inputArgs)
I did not find a good way to pop the args that have been parsed, so I am still 
combining args nose got from the argparser result.

Line 240:         print "hacking vdsm modles..."
Done

Line 244: def genCaseList(testConfig):
Done

--
To view, visit http://gerrit.ovirt.org/3923
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I193950b73a9bc1583433c3814f4c7d028f01ecd0
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Wenchao Xia <[email protected]>
Gerrit-Reviewer: Ayal Baron <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Mark Wu <[email protected]>
Gerrit-Reviewer: Saggi Mizrahi <[email protected]>
Gerrit-Reviewer: Shu Ming <[email protected]>
Gerrit-Reviewer: Wenchao Xia <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to