Closed (fixed)
Project:
Nodequeue
Version:
7.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
20 May 2009 at 15:09 UTC
Updated:
8 Sep 2015 at 17:04 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
ezra-g commentedThanks!
Marking as needs review.
Comment #2
wulff commentedAdding tests for users with the 'administer nodequeue' permissions.
Next on the list: Tests for the API functions.
Comment #3
wulff commentedAdded some more test cases.
Comment #4
wulff commentedAdded some more test cases. Current coverage is 70%.
Comment #5
ezra-g commentedPlease accept my delayed and large thanks.
When I ran these tests, almost all pass with only a few exceptions, and then I encounter a 500 error. I need to fix my testing environment so I can finish making sure that the tests pass and then this will be committed.
Comment #6
wulff commentedFinally hit 90% coverage. This set of test cases will result in a few fails until we fix a few minor issues.
Comment #7
ezra-g commentedThe 500 errors were happening in part because of undefined functions called in the tests ($document->setMultiValue($key, $sqid) in nodequeue_apachesolr_update_index ) and nodequeue_ajax_add being removed from Nodequeue.module when we implemented #234004: Drag and drop node ordering. Of course, that move broke many of the tests here ;). With the nodequeue_ajax_add tests removed (and $document->setMultiValue temporarily commented out) , I'm showing 623 passes, 32 fails, and 43 exceptions.
The test results are attached as a png, which seems kind of wrong, but allows us to see the results without running the tests again.
@wulff: I'll unassign but of course feel free to join in here again ;).
Comment #8
Steven Merrill commentedHere's one with all tests passing as of today, other than ApacheSolr, which I moved into a new group so that I could not run it with the rest. Mostly, there needed to be some small changes made to stay in step with error messages. In addition, I removed the up/down testing, since the table view is now draggable.
Comment #9
Steven Merrill commentedReview, please!
Comment #10
Steven Merrill commentedWhoops. _nodequeue_ajax_add() is no more.
Here's a new patch. It also has some tests and a mock module in place for a reroll of #606150: Hook for queue deletion .
Comment #11
Steven Merrill commentedSome of these still fail when the site is in a subfolder. Further patch coming.
Comment #12
Steven Merrill commentedOkay - here we go. Support for running in a subdirectory or at the site root, and a couple last fixes around how the test nodequeues were being populated.
Comment #13
ezra-g commentedHooray! All tests pass on the latest 6.x-2.x branch. Committed, and marking as needs porting for 7.x-2.x :).
http://drupal.org/cvs?commit=463068
Huzzaaaah!
Comment #14
kenorb commentedSimpletest under 6.x is returning Fatal error:
#1777622: Simpletest: Fatal error: Call to undefined method stdClass::setMultiValue() in nodequeue.module on line 422
Comment #15
penyaskitoI'm getting fatal errors when running tests with 5.3 because of use of passing references. E.g.: &$form_state is not longer needed.
Comment #16
kenorb commentedComment #17
fizk commentedI've updated the tests to work for 7.x-2.x, but some are still failing.
Comment #18
fizk commentedWe have working tests for 7.x-2.x. Congrats!
Comment #19
fizk commentedFor some reason two tests fail on Drupal.org test bots: #2557343: Two tests fail for nodequeue module.
Does anyone know why it breaks only for the test bots?