Closed
Bug 703013
Opened 13 years ago
Closed 13 years ago
Talos should provide a sensible API for running tests
Categories
(Testing :: Talos, defect)
Tracking
(Not tracked)
RESOLVED
DUPLICATE
of bug 704654
People
(Reporter: wlach, Unassigned)
References
Details
(Whiteboard: [mozbase])
Right now eideticker shells out to talos like this:
It would be great if talos exposed a nice API for running tests and automatically generating an in-memory configuration, for use-cases like this. There's nothing stopping me from importing talos directly now, but it would look pretty weird.
I expect working on this bug would involve the following:
1. Providing a clean API to generate a configuration in-memory.
2. Providing an API to run a configuration.
Fixing this bug in an elegant way might involve moving the logic to rewrite remote manifest files into the part of talos which runs a test. It doesn't really make sense as it is, but a user-facing API definitely shouldn't have such surprising behaviour IMO.
Updated•13 years ago
|
Whiteboard: [mozbase]
Reporter | ||
Comment 1•13 years ago
|
||
(In reply to William Lachance (:wlach) from comment #0)
> Right now eideticker shells out to talos like this:
I meant to include a link there, but didn't. Here's one:
https://github.com/mozilla/eideticker/blob/master/bin/runtest.py#L118
Comment 2•13 years ago
|
||
I'm interested in working on this bug. But I'm a bit confused about where in the above link eideticker is using the talos api. Also do you guys have any ideas on other places in talos I should look at that might help for this bug? Thanks!
Reporter | ||
Comment 3•13 years ago
|
||
Hey Fahim, this bug was originally filed when it looked like we wanted to use Talos with Eideticker. This is not currently the case.
Nonetheless, it looks like things will be much more reasonable to use Talos programmatically when :jhammel's unified configuration work lands. Here's an example of what it will enable:
conf = PerfConfigurator()({'config': 'i', 'want': 1}); run_tests.run_tests(conf.config)
I'd say the best thing is probably to build on Jeff's work here. I'm going to mark this bug as a duplicate.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•