Last modified: 2014-11-18 22:22:03 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 T75451, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 73451 - Flow QA collapse browsertest should stop testing moderation
Flow QA collapse browsertest should stop testing moderation
Status: PATCH_TO_REVIEW
Product: MediaWiki extensions
Classification: Unclassified
Flow (Other open bugs)
master
All All
: Unprioritized major (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-11-15 02:38 UTC by spage
Modified: 2014-11-18 22:22 UTC (History)
6 users (show)

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


Attachments

Description spage 2014-11-15 02:38:20 UTC
The scenario "For a moderated post, a mode cycle with no user override keeps it hidden" is broken. Its background creates then hides a topic, and then the test asserts
    Given I am on Flow page 
    Then I should not see a moderated message on the first moderated topic

In fact the moment Flow reloads a page (the "Given I am on Flow page"), the hidden topic that this test creates vanishes altogether.  Often there happens to be a locked topic (which remains visible) lower down on the page, and the test finds it and fails at
   features/collapse.feature:65:in `Then I should not see a moderated message on the first moderated topic'

So the test is irrelevant[1] and it intermittently fails.

I rewrote the test in Gerrit change #164720 to "remove moderated notion from collape tests".  But we will remove the collapse switcher feature (Topics and posts > Topics only > Small topics) altogether when we implement a TOC for Flow boards, so it's probably easier to just kill that one part of the test right now.

[1] The actual feature to test is unrelated to collapsing, it is
  Given I hide (or delete or suppress) a topic
  Then ...
  When I reload the page
  Then I should not see hidden, deleted, or suppressed topics
Gerrit change #164655 is a step towards this.
Comment 1 spage 2014-11-15 02:54:44 UTC
See Gerrit change #164720
Comment 2 Chris McMahon 2014-11-18 22:22:03 UTC
this patch removes the test completely. I would like to remove the test completely until the behavior is sane: https://gerrit.wikimedia.org/r/#/c/166727/

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


Navigation
Links