Issue #183 test suite #188

Merged
merged 3 commits into from Mar 4, 2013

Projects

None yet

2 participants

@e1himself
Contributor

Implement test suite to check issue #183 functionality.
There was quite a big work and I modified exising testing code and making it more common. But now it seems to be working fine.

@e1himself
Owner

I've implemented actual test to reveal this issue #183:

  • Modified route to allow wildcards
  • Modified crud11Test.php to use admin15 generator theme
  • Modified ProjectConfiguration to generate also model filter classes (needded by admin15 theme)
  • Modified schema and added 'sortable' behavior to Artiicle model
  • Added crudBrowserAdmin15 class to test this issue (invalid links for 'Move up'/'Move down' actions)
Owner

Oh, I've noticed that Travis has reported that build is broken. I'll check that.

@willdurand willdurand merged commit 9c8c208 into propelorm:master Mar 4, 2013

1 check passed

default The Travis build passed
Details
@willdurand
Member

thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment