[FFE] autopilot-qt lacks testing
Bug #1157697 reported by
Michael Zanetti
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Autopilot Qt Support |
Fix Released
|
Critical
|
Michael Zanetti | ||
autopilot-qt (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned | ||
Raring |
Won't Fix
|
Undecided
|
Unassigned |
Bug Description
There should be unit tests and integration tests. This led to unwanted breakage already and needs to be tackled asap.
Related branches
lp:~mzanetti/autopilot-qt/add-tests
Rejected
for merging
into
lp:autopilot-qt/0.8
- PS Jenkins bot: Approve (continuous-integration)
- Mathieu Trudel-Lapierre: Pending requested
- Autopilot Hackers: Pending requested
-
Diff: 1062 lines (+773/-129)23 files modifiedautopilot-qt.pro (+1/-1)
debian/control (+12/-0)
debian/libautopilot-qt-autopilot.install (+3/-0)
debian/rules (+2/-2)
driver/introspection.cpp (+21/-4)
test/main.cpp (+0/-17)
test/test.pro (+0/-11)
test/test_dbus_introspection.cpp (+0/-50)
test/test_dbus_introspection.h (+0/-44)
tests/autopilot/autopilot.pro (+2/-0)
tests/autopilot/libautopilot_qt/__init__.py (+8/-0)
tests/autopilot/libautopilot_qt/emulators/__init__.py (+6/-0)
tests/autopilot/libautopilot_qt/emulators/main_window_qt4.py (+57/-0)
tests/autopilot/libautopilot_qt/emulators/main_window_qt5.py (+57/-0)
tests/autopilot/libautopilot_qt/tests/__init__.py (+66/-0)
tests/autopilot/libautopilot_qt/tests/test_main.py (+103/-0)
tests/autopilot/testapp/qt4.qml (+55/-0)
tests/autopilot/testapp/qt5.qml (+55/-0)
tests/autopilot/testapp/testapp.cpp (+52/-0)
tests/autopilot/testapp/testapp.pro (+22/-0)
tests/tests.pro (+2/-0)
tests/unittests/tst_introspection.cpp (+223/-0)
tests/unittests/unittests.pro (+26/-0)
- Mathieu Trudel-Lapierre (community): Approve
- PS Jenkins bot: Approve (continuous-integration)
- Martin Mrazik (community): Approve
-
Diff: 1062 lines (+773/-129)23 files modifiedautopilot-qt.pro (+1/-1)
debian/control (+12/-0)
debian/libautopilot-qt-autopilot.install (+3/-0)
debian/rules (+2/-2)
driver/introspection.cpp (+21/-4)
test/main.cpp (+0/-17)
test/test.pro (+0/-11)
test/test_dbus_introspection.cpp (+0/-50)
test/test_dbus_introspection.h (+0/-44)
tests/autopilot/autopilot.pro (+2/-0)
tests/autopilot/libautopilot_qt/__init__.py (+8/-0)
tests/autopilot/libautopilot_qt/emulators/__init__.py (+6/-0)
tests/autopilot/libautopilot_qt/emulators/main_window_qt4.py (+57/-0)
tests/autopilot/libautopilot_qt/emulators/main_window_qt5.py (+57/-0)
tests/autopilot/libautopilot_qt/tests/__init__.py (+66/-0)
tests/autopilot/libautopilot_qt/tests/test_main.py (+103/-0)
tests/autopilot/testapp/qt4.qml (+55/-0)
tests/autopilot/testapp/qt5.qml (+55/-0)
tests/autopilot/testapp/testapp.cpp (+52/-0)
tests/autopilot/testapp/testapp.pro (+22/-0)
tests/tests.pro (+2/-0)
tests/unittests/tst_introspection.cpp (+223/-0)
tests/unittests/unittests.pro (+26/-0)
Changed in autopilot-qt: | |
status: | New → In Progress |
Changed in autopilot-qt: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
The change involves adding a new package to autopilot-qt; so I think this needs a Feature Freeze exception. Subscribing ubuntu-release.
Since it's a new package that isn't really used by much (though it will be eventually), I don't see much impact in adding the binary package, but let's see what the release team thinks... It's also getting a bit late to add this kind of new stuff, but I think the benefit of having more tests for autopilot-qt itself outweighs the risks.