git.iko.soy/unity8
on xenial help sign in
Re-enable disabled tests
commit79dab12d22e41982efc49e93a604edfea4bb0c31 authorDalton Durst <me@daltondurst.com> dateFri 20 Jul 2018, 12:17 (8 years ago) parent720eb14
2 files changed+18−18
compare with xenial
unifiedsplit
plugins/Wizard/System.h+1−1 tests/plugins/Wizard/tst_system.cpp+17−17
plugins/Wizard/System.h+1−1
@@ -45,7 +45,7 @@ public:
4545 * For example, if ::CURRENT_VERSION is 4, version 4 has not run, but the
4646 * first-run wizard has run, this method will return 4
4747 */
48− signed int versionToShow() const;
48+ int versionToShow() const;
4949
5050 /**
5151 * The update version for the Wizard in this version of Unity8. Bump this
tests/plugins/Wizard/tst_system.cpp+17−17
@@ -118,7 +118,7 @@ void SystemTest::testUpdate()
118118
119119 System system;
120120 QVERIFY(system.wizardEnabled());
121− QVERIFY(system.versionToShow() > 0);
121+ QVERIFY(system.versionToShow() == system.CURRENT_VERSION);
122122
123123 system.setWizardEnabled(false);
124124 QVERIFY(!system.wizardEnabled());
@@ -128,28 +128,28 @@ void SystemTest::testUpdate()
128128
129129void SystemTest::testNoticeChanges()
130130{
131− // enable();
131+ enable();
132132
133− // System system;
134− // QSignalSpy spy(&system, SIGNAL(wizardEnabledChanged()));
133+ System system;
134+ QSignalSpy spy(&system, SIGNAL(wizardEnabledChanged()));
135135
136− // // System only guarantees its signals work correcty when using its own set
137− // // methods (i.e. it won't necessarily notice if we modify the file behind
138− // // the scenes). This is because watching all parent directories of the
139− // // wizard-has-run file with QFileSystemWatcher is a nightmare and waste of
140− // // resources for the corner case it is. So we'll just test the set method.
136+ // System only guarantees its signals work correcty when using its own set
137+ // methods (i.e. it won't necessarily notice if we modify the file behind
138+ // the scenes). This is because watching all parent directories of the
139+ // wizard-has-run file with QFileSystemWatcher is a nightmare and waste of
140+ // resources for the corner case it is. So we'll just test the set method.
141141
142− // system.setWizardEnabled(false);
143− // QTRY_COMPARE(spy.count(), 1);
142+ system.setWizardEnabled(false);
143+ QTRY_COMPARE(spy.count(), 1);
144144
145− // system.setWizardEnabled(true);
146− // QTRY_COMPARE(spy.count(), 2);
145+ system.setWizardEnabled(true);
146+ QTRY_COMPARE(spy.count(), 2);
147147
148− // system.setWizardEnabled(false);
149− // QTRY_COMPARE(spy.count(), 3);
148+ system.setWizardEnabled(false);
149+ QTRY_COMPARE(spy.count(), 3);
150150
151− // system.setWizardEnabled(true);
152− // QTRY_COMPARE(spy.count(), 4);
151+ system.setWizardEnabled(true);
152+ QTRY_COMPARE(spy.count(), 4);
153153}
154154
155155QTEST_MAIN(SystemTest)