Last modified: 2014-10-07 08:39:19 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T70094, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 68094 - Create browser smoke test for a typical item
Create browser smoke test for a typical item
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
WikidataRepo (Other open bugs)
unspecified
All All
: High normal (vote)
: ---
Assigned To: Wikidata bugs
u=dev c=QA p=8 s=2014-09-23
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-07-16 09:30 UTC by tobias.gritschacher
Modified: 2014-10-07 08:39 UTC (History)
3 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description tobias.gritschacher 2014-07-16 09:30:48 UTC
Create a test that loads a typical (existing) items and compares it to the expected output.
Comment 1 Jeroen De Dauw 2014-09-23 23:05:40 UTC
We already have many tests tauching items in some way. So what is the point of adding a smoke test? Adding a smoke test when there are more detailed tests already makes little sense. Perhaps the term "smoke test" is used incorrectly here?
Comment 2 Lydia Pintscher 2014-09-24 09:27:46 UTC
It does make sense. We have had too many cases where our specific tests didn't catch issues that would have been caught with this.
Comment 3 tobias.gritschacher 2014-09-24 13:23:03 UTC
(In reply to Jeroen De Dauw from comment #1)
> We already have many tests tauching items in some way. So what is the point
> of adding a smoke test? Adding a smoke test when there are more detailed
> tests already makes little sense. Perhaps the term "smoke test" is used
> incorrectly here?

It happened several times recently that things broke and were only recognized after it was deployed to test.wikidata.org or in worst cases after we went live. All these issues could have been avoided by just looking at a showcase item on beta because breakage was visible there. Just nobody looked there manually.
The purpose of such a test (call it however you want) is to automate this and get instant reports if something that obvious breaks on beta.
Hope that explained the task a bit more.
Comment 4 tobias.gritschacher 2014-09-24 13:26:45 UTC
(In reply to tobias.gritschacher from comment #3)
> (In reply to Jeroen De Dauw from comment #1)
> > We already have many tests tauching items in some way. So what is the point
> > of adding a smoke test? Adding a smoke test when there are more detailed
> > tests already makes little sense. Perhaps the term "smoke test" is used
> > incorrectly here?
> 
> It happened several times recently that things broke and were only
> recognized after it was deployed to test.wikidata.org or in worst cases
> after we went live. All these issues could have been avoided by just looking
> at a showcase item on beta because breakage was visible there. Just nobody
> looked there manually.
> The purpose of such a test (call it however you want) is to automate this
> and get instant reports if something that obvious breaks on beta.
> Hope that explained the task a bit more.

Addition: the detailed tests we have would not catch breakage that is related to existing items. All the tests we currently have create new items. There is no test that checks an existing item with hundreds of statements, references and qualifiers. This task will close this lack.
Comment 5 tobias.gritschacher 2014-09-24 13:27:08 UTC
https://github.com/wmde/WikidataBrowserTests/pull/11
Comment 6 Jeroen De Dauw 2014-09-24 20:54:45 UTC
> Hope that explained the task a bit more.

It did.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links