Code review comment for lp:~elopio/click-update-manager/fix1234379-autopilot_tests

Revision history for this message
Corey Goldberg (coreygoldberg) wrote :

looks very good.
one small nitpick:

197 + self.app = self.launch_test_application(
198 + '/usr/lib/' + arch + '/qt5/bin/qmlscene',
199 + "-I" + _get_module_include_path(),

... should probably use os.path.join() instead of appending strings so it matches the rest of code style.

« Back to merge proposal