Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FAIL: ckan.tests.lib.test_resource_search.TestSearch.test_13_pagination #818

Merged
merged 1 commit into from Jul 22, 2013

Conversation

tobes
Copy link
Contributor

@tobes tobes commented Jul 1, 2013

this test seems to have a race condition and often fails under travis example https://travis-ci.org/okfn/ckan/builds/6605215

ckan.tests.lib.test_resource_search.TestSearch.test_13_pagination

======================================================================
FAIL: ckan.tests.lib.test_resource_search.TestSearch.test_13_pagination
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/travis/virtualenv/python2.7/local/lib/python2.7/site-packages/nose/case.py", line 197, in runTest
    self.test(*self.arg)
  File "/home/travis/build/okfn/ckan/ckan/tests/lib/test_resource_search.py", line 154, in test_13_pagination
    assert resources == all_resources[:2], '%r, %r' % (resources, all_resources)
AssertionError: [u'2537c680-fadb-4710-96dc-735f5711d56e', u'90d6b31f-ed98-46c4-949c-0982997dde25'], [u'90d6b31f-ed98-46c4-949c-0982997dde25', u'2537c680-fadb-4710-96dc-735f5711d56e', u'c164728f-4640-413f-a1ed-db1e6cfd73a9', u'51788d8c-d6ee-4e65-b058-604ebfe53e73', u'eec7bbc8-8424-453f-bed2-a90c587aea46', u'7a3e45d1-fde7-476d-8917-21b8dac5f701']
----------------------------------------------------------------------

@tobes
Copy link
Contributor Author

tobes commented May 29, 2013

I think this is related to the hash as the returned results seem correct but the order is wrong this makes me think that the hash is never set.

@ghost ghost assigned joetsoi Jul 2, 2013
@joetsoi joetsoi merged commit 887e50f into master Jul 22, 2013
@tobes tobes deleted the 818-attempt-to-fix-recuring-test-fail branch July 22, 2013 14:49
amercader pushed a commit that referenced this pull request Aug 19, 2014
amercader pushed a commit that referenced this pull request Aug 19, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants