Code review comment for lp:~canonical-platform-qa/ubuntu-system-tests/clean-install

Revision history for this message
Sergio Cazzolato (sergio-j-cazzolato) wrote :

I agree with this implementation, tests passing in my device.

I was thinking that based on this implementation we could make a solution to install the dependencies for a subset of tests. For that we should use the try_import module to load and track all the modules needed, so, when we need to run a test class, we load the class and we can track all the imports needed in the try_import helper and retrieve a list of dependencies. We should have a map to know the package to install based on a dependency. We can discuss it on Monday.

Approving. I would like to wait until Santiago takes a look to the change before top approving it.

Good job richard

review: Approve

« Back to merge proposal