Fix for issue #3945, and fix for PUT with request content #4366

Closed
wants to merge 3 commits into
from

Projects

None yet

2 participants

@superdweebie

Small fixes to make the AbstractControllerTest more useful.

@weierophinney
Zend Framework member

@superdweebie Please take a look at the tests.

  • I tried merging locally, and immediately ran across an error where the request class was not an HTTP request, which made a call to getMethod() invalid.
  • After modifying the code to test the request type, I get 4 new failures.
@superdweebie

I'll be fixing it in a few hours. My mistake.

@superdweebie

Green :)

@weierophinney weierophinney added a commit that closed this pull request May 2, 2013
@weierophinney weierophinney Merge branch 'hotfix/4366'
Close #4366
Fixes #3945
2159035
@weierophinney weierophinney added a commit that referenced this pull request May 2, 2013
@weierophinney weierophinney Merge branch 'hotfix/4366' into develop
Forward port #4366
45951c4
@weierophinney weierophinney was assigned May 2, 2013
@ghost Unknown pushed a commit that referenced this pull request Jul 14, 2013
@weierophinney weierophinney Merge branch 'hotfix/4366'
Close #4366
Fixes #3945
6719408
@ghost Unknown pushed a commit that referenced this pull request Jul 14, 2013
@weierophinney weierophinney Merge branch 'hotfix/4366' into develop
Forward port #4366
7785b61
@weierophinney weierophinney added a commit to zendframework/zend-test that referenced this pull request May 15, 2015
@weierophinney weierophinney Merge pull request zendframework/zendframework#4366 from superdweebie…
…/AbstractControllerTest-fix

Fix for issue zendframework/zendframework#3945, and fix for PUT with request content
c4e2470
@weierophinney weierophinney added a commit to zendframework/zend-test that referenced this pull request May 15, 2015
@weierophinney weierophinney Merge branch 'hotfix/4366' 41f8677
@weierophinney weierophinney added a commit to zendframework/zend-test that referenced this pull request May 15, 2015
@weierophinney weierophinney Merge branch 'hotfix/4366' into develop 26a7c2f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment