Bug 68442 - [Qt] QTouchWebView missing signal tests for loadProgress
Summary: [Qt] QTouchWebView missing signal tests for loadProgress
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit2 (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-09-20 08:31 PDT by Gopal Raghavan
Modified: 2011-09-20 15:15 PDT (History)
3 users (show)

See Also:


Attachments
TouchWebView loadProgress signal test (2.65 KB, patch)
2011-09-20 08:37 PDT, Gopal Raghavan
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Gopal Raghavan 2011-09-20 08:31:19 PDT
Need signal tests for QTouchWebView loadProgress
Comment 1 Gopal Raghavan 2011-09-20 08:37:47 PDT
Created attachment 108006 [details]
TouchWebView loadProgress signal test

Adding signal tests for TouchWebView.
This is very similar to DesktopWebView (68280).
Here is the test results:

********* Start testing of qmltests *********
Config: Using QTest library 5.0.0, Qt 5.0.0
PASS   : DesktopWebViewLoadProgress::initTestCase()
QWARN  : DesktopWebViewLoadProgress::test_loadProgress() QSocketNotifier: socket notifiers cannot be enabled from another thread
QSocketNotifier: socket notifiers cannot be enabled from another thread
PASS   : DesktopWebViewLoadProgress::test_loadProgress()
PASS   : DesktopWebViewLoadProgress::cleanupTestCase()
PASS   : DesktopWebViewLoadProgressSignal::initTestCase()
QWARN  : DesktopWebViewLoadProgressSignal::test_loadProgressSignal() QSocketNotifier: socket notifiers cannot be enabled from another thread
QSocketNotifier: socket notifiers cannot be enabled from another thread
PASS   : DesktopWebViewLoadProgressSignal::test_loadProgressSignal()
PASS   : DesktopWebViewLoadProgressSignal::cleanupTestCase()
QWARN  : qmltests::UnknownTestFunc() QSocketNotifier: socket notifiers cannot be enabled from another thread
QSocketNotifier: socket notifiers cannot be enabled from another thread
PASS   : DesktopWebViewNavigationPolicyForUrl::initTestCase()
QWARN  : DesktopWebViewNavigationPolicyForUrl::test_ignorePolicy() QObject::startTimer: timers cannot be started from another thread
PASS   : DesktopWebViewNavigationPolicyForUrl::test_ignorePolicy()
PASS   : DesktopWebViewNavigationPolicyForUrl::test_usePolicy()
PASS   : DesktopWebViewNavigationPolicyForUrl::cleanupTestCase()
PASS   : DesktopWebViewProperties::initTestCase()
QWARN  : DesktopWebViewProperties::test_title() QSocketNotifier: socket notifiers cannot be enabled from another thread
QSocketNotifier: socket notifiers cannot be enabled from another thread
PASS   : DesktopWebViewProperties::test_title()
PASS   : DesktopWebViewProperties::test_url()
PASS   : DesktopWebViewProperties::cleanupTestCase()
PASS   : TouchWebViewLoad::initTestCase()
QWARN  : TouchWebViewLoad::test_load() QSocketNotifier: socket notifiers cannot be enabled from another thread
QSocketNotifier: socket notifiers cannot be enabled from another thread
PASS   : TouchWebViewLoad::test_load()
PASS   : TouchWebViewLoad::cleanupTestCase()
PASS   : TouchWebViewLoad::initTestCase()
QWARN  : TouchWebViewLoad::test_loadNegativeSizeView() QSocketNotifier: socket notifiers cannot be enabled from another thread
QSocketNotifier: socket notifiers cannot be enabled from another thread
PASS   : TouchWebViewLoad::test_loadNegativeSizeView()
PASS   : TouchWebViewLoad::cleanupTestCase()
PASS   : TouchWebViewLoadProgress::initTestCase()
QSocketNotifier: socket notifiers cannot be enabled from another thread
PASS   : TouchWebViewLoadProgress::test_loadProgress()
PASS   : TouchWebViewLoadProgress::cleanupTestCase()
PASS   : TouchWebViewLoadProgressSignal::initTestCase()
QWARN  : TouchWebViewLoadProgressSignal::test_loadProgressSignal() QSocketNotifier: socket notifiers cannot be enabled from another thread
QSocketNotifier: socket notifiers cannot be enabled from another thread
PASS   : TouchWebViewLoadProgressSignal::test_loadProgressSignal()
PASS   : TouchWebViewLoadProgressSignal::cleanupTestCase()
PASS   : TouchWebViewLoad::initTestCase()
QWARN  : TouchWebViewLoad::test_loadZeroSizeView() QSocketNotifier: socket notifiers cannot be enabled from another thread
QSocketNotifier: socket notifiers cannot be enabled from another thread
PASS   : TouchWebViewLoad::test_loadZeroSizeView()
PASS   : TouchWebViewLoad::cleanupTestCase()
PASS   : TouchWebViewProperties::initTestCase()
QWARN  : TouchWebViewProperties::test_title() QSocketNotifier: socket notifiers cannot be enabled from another thread
QSocketNotifier: socket notifiers cannot be enabled from another thread
PASS   : TouchWebViewProperties::test_title()
PASS   : TouchWebViewProperties::test_url()
PASS   : TouchWebViewProperties::cleanupTestCase()
Totals: 33 passed, 0 failed, 0 skipped
********* Finished testing of qmltests *********
Comment 2 Chang Shu 2011-09-20 13:26:39 PDT
I am not sure about the history of the QTouchWebView code. Why the tests were missing at the first place and why it's essential to add them now? Btw, do we run Qt tests on the buildbot?
Comment 3 Gopal Raghavan 2011-09-20 14:04:53 PDT
Good question. I do not know why tests are missing. 
 
I am trying to rectify the situation by adding these api tests for coverage and regression. 

loadProgress Signal was not tested. So I think it is a good idea to add the test. We did similar thing for DesktopWebView. Reviewer also suggested to keep TouchWebView in sync.
Comment 4 Chang Shu 2011-09-20 14:39:34 PDT
Comment on attachment 108006 [details]
TouchWebView loadProgress signal test

Sure. If the TouchWebView code has not been covered by any layout tests or other Qt api tests, it's good to add these qml tests. But we don't want to duplicate the coverage.
Comment 5 WebKit Review Bot 2011-09-20 15:15:42 PDT
Comment on attachment 108006 [details]
TouchWebView loadProgress signal test

Clearing flags on attachment: 108006

Committed r95572: <http://trac.webkit.org/changeset/95572>
Comment 6 WebKit Review Bot 2011-09-20 15:15:46 PDT
All reviewed patches have been landed.  Closing bug.