Skip to content
This repository has been archived by the owner on Jan 8, 2020. It is now read-only.

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

Closed
wants to merge 3 commits into from
Closed

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

wants to merge 3 commits into from

Conversation

superdweebie
Copy link
Contributor

Small fixes to make the AbstractControllerTest more useful.

@weierophinney
Copy link
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
Copy link
Contributor Author

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

@superdweebie
Copy link
Contributor Author

Green :)

weierophinney added a commit that referenced this pull request May 2, 2013
Fix for issue #3945, and fix for PUT with request content
weierophinney added a commit that referenced this pull request May 2, 2013
@ghost ghost assigned weierophinney May 2, 2013
weierophinney added a commit to zendframework/zend-test that referenced this pull request May 15, 2015
…/AbstractControllerTest-fix

Fix for issue zendframework/zendframework#3945, and fix for PUT with request content
weierophinney added a commit to zendframework/zend-test that referenced this pull request May 15, 2015
weierophinney added a commit to zendframework/zend-test that referenced this pull request May 15, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants