803755 - [socorro-crashstats] test_that_search_for_valid_signature is failing

Status: VERIFIED FIXED

(Socorro :: Webapp, task)

Description

[Stephen Donner [:stephend] Not actively reading bugmail]

test_that_search_for_valid_signature [1] is failing:

self = <tests.test_search.TestSearchForIdOrSignature instance at 0x1775238>
mozwebqa = <pytest_mozwebqa.pytest_mozwebqa.TestSetup instance at 0x17873f0>

@pytest.mark.nondestructive
def test_that_search_for_valid_signature(self, mozwebqa):
"""
This is a test for
https://bugzilla.mozilla.org/show_bug.cgi?id=609070
"""
csp = CrashStatsHomePage(mozwebqa)
report_list = csp.click_last_product_top_crashers_link()
signature = report_list.first_signature_title

result = csp.header.search_for_crash(signature)
> Assert.true(result.are_results_found)

tests/test_search.py:45:
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _

self = <class unittestzero.Assert at 0x14a10f0>, first = False, msg = None

@classmethod
def true(self, first, msg=None):
"""
Asserts that what is given is equal to True

:Args:
- First object to be tested
- Message that will be printed if it fails
"""

> assert first is True, msg
E assert False is True

.env/lib/python2.6/site-packages/unittestzero.py:77: AssertionError

[1] https://github.com/mozilla/Socorro-Tests/blob/master/tests/test_search.py#L34

Comment 1

[[DEACTIVATED] Adrian Gaudebert]

I can't reproduce this locally. Stephen, do you have steps to reproduce this? Here is what I am doing: 

1. Go to home page
2. Click topcrasher for the last version (ie Firefox release)
3. Copy the first signature, paste in the upper form and start a search
4. Verify that there are results on the /query page.

Doing that, I indeed get results. Did I miss something in the test?

Comment 2

[Stephen Donner [:stephend] Not actively reading bugmail]

Pretty sure this is just a locator issue, which brings up the whole "do we really want DOM for DOM" compatibility between the current and Django-rewrite sites.

Comment 3

[Robert Helmer [:rhelmer]]

I have selenium running locally (against my local crash-stats instance), so I can take a look.

(In reply to Stephen Donner [:stephend] from comment #2)
> Pretty sure this is just a locator issue, which brings up the whole "do we
> really want DOM for DOM" compatibility between the current and
> Django-rewrite sites.

I think we do want compatibility unless there's a good reason not to. We should at least understand the reason we're breaking it ;)

Comment 4

[Robert Helmer [:rhelmer]]

The problem is that the table id has changed from signatureList to signatures-list.

I went ahead and changed it (and the tests+js that refer to it) back:

r? https://github.com/mozilla/socorro-crashstats/pull/209

Comment 5

[[github robot]]

Commits pushed to master at https://github.com/mozilla/socorro-crashstats

https://github.com/mozilla/socorro-crashstats/commit/82661f654e2e139bdc97fc4cda3deda0f8cd230d
bug 803755 - fix failing selenium test test_that_search_for_valid_signature by changing signatures-list back to signatureList

https://github.com/mozilla/socorro-crashstats/commit/9637f6c6efef82c9a2eabfa2a62ac15d524d5c43
Merge pull request #209 from rhelmer/bug803755-search-test-failing

bug 803755 - fix failing selenium test test_that_search_for_valid_signat...

Comment 6

[Stephen Donner [:stephend] Not actively reading bugmail]

Verified FIXED:

http://qa-selenium-stage.mv.mozilla.com:8080/job/socorro.dev.redesign/272/HTML_Report/?

Comment 7

[Stephen Donner [:stephend] Not actively reading bugmail]

Attachment: Post-fix screenshot