Merge lp://qastaging/~coreygoldberg/selenium-simple-test/unified-testcase-runner into lp://qastaging/selenium-simple-test

47 + """scan
Scan should start with upper-case.

Instead of
53 + V = ast.NodeVisitor()
I would write node_visitor = ast.NodeVisitor(). At least it should be start with lower-case,

200 +class SSTBrowserlessTestCase(runtests.SSTScriptTestCase):
Why don't you use the SSTBrowserlessTestCase defined in src/sst/tests/__init__.py?
To use this one for the SSTTestCase test is also weird, because it inherits from SSTScriptTestCase, so in the worst case you are testing the wrong thing. In the best case, it's confusing.

And the most important thing is that we are missing tests for get_case. That's not the fault of this branch, but as we are modifying its behavior, the right thing to do would be to add them.
What if we write a tmp file without classes, and assert that get_case returns a SSTScriptTestCase. And then write something with classes inside, and assert that it returns a SSTTestCase? I think that would be enough.

Also from IRC, why:

50 + found_classes = []
51 + def visit_class_def(node):
52 + found_classes.append(True)

instead of:
found_classes = False
def visit_class_def(node):
   found_classes = True

I changed the test cases and what was being tested, fixed recommendations, and re-pushed.

Thanks Corey.

few final fixes.

There are additional revisions which have not been approved in review. Please seek review and approval of these new revisions.