Here is a patch that makes possible to run the tests for HEAD (6.x)

CommentFileSizeAuthor
#1 context_more_test.patch2.45 KBaron novak
context_test_6x.patch1.42 KBaron novak

Comments

aron novak’s picture

aron novak
Primary language Hungarian
Location Hungary, Budapest
commented
StatusFileSize
new context_more_test.patch2.45 KB

I wrote additional assertation at the test.
It does not pass, but it should. When I manually do the same, it works as expected.

jmiccolis’s picture

jmiccolis commented
Assigned: Unassigned » jmiccolis

Thanks for the patch Aron, I'm going to update this for the newer version of simpletest, and will post back when it's commited.

jmiccolis’s picture

jmiccolis commented
Status: Needs review » Needs work

I've updated the tests, but I've had to simplify things a bit and it's note quite clear the best way to test certain things. I'd really like to determine a way to test the setting of context based on a context definition and also response to a context being set. However because of the way the module works it looks like we're going to need to break these into a number of different tests.

For now we're just testing saving a context definition to the database.

pasqualle’s picture

pasqualle
Location 🇭🇺 Budapest
commented
Version: 5.x-1.x-dev » 6.x-1.x-dev
yhahn’s picture

yhahn commented
Status: Needs work » Closed (fixed)

Closing. We have working tests in the the 3.x branch.