Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[create_framework] phpunit test error #5487

Closed
emillosanti opened this issue Jul 5, 2015 · 2 comments
Closed

[create_framework] phpunit test error #5487

emillosanti opened this issue Jul 5, 2015 · 2 comments
Labels
hasPR A Pull Request has already been submitted for this issue.

Comments

@emillosanti
Copy link
Contributor

error when running phpunit test:

Call to a member function fromRequest() on a non-object in .../src/Simplex/Framework.php on line 29

resolved by adding

protected function getFrameworkForException($exception)
{
    //...

    $matcher
        ->expects($this->once())
        ->method('getContext')
        ->will($this->returnValue($this->getMock('Symfony\Component\Routing\RequestContext')))
    ;

based on old issue :
https://github.com/fabpot/Create-Your-Framework/issues/29

http://webcache.googleusercontent.com/search?q=cache:yufENRDJeo8J:https://github.com/fabpot/Create-Your-Framework/issues/29+&cd=2&hl=en&ct=clnk
emillosanti added a commit to emillosanti/symfony-docs that referenced this issue Jul 5, 2015
@emillosanti
resolve symfony#5487
5c1b5e0 
[create_framework] phpunit test error symfony#5487

Call to a member function fromRequest() on a non-object in .../src/Simplex/Framework.php on line 29
@emillosanti emillosanti mentioned this issue Jul 5, 2015
Merged
emillosanti added a commit to emillosanti/symfony-docs that referenced this issue Jul 5, 2015
@emillosanti
Merge pull request #1 from emillosanti/emillosanti-patch-1
4a5c538 
resolve symfony#5487
@xabbuh xabbuh added the hasPR A Pull Request has already been submitted for this issue. label Jul 7, 2015
@xabbuh
Copy link
Member

xabbuh commented Jul 7, 2015

for the records: #5488 fixes this issue

@xabbuh xabbuh closed this as completed Jul 7, 2015
@xabbuh
Copy link
Member

xabbuh commented Jul 7, 2015

sorry, wrong button

@xabbuh xabbuh reopened this Jul 7, 2015
xabbuh pushed a commit that referenced this issue Jul 7, 2015
@emillosanti @xabbuh
fix #5487
68c7364
xabbuh added a commit that referenced this issue Jul 7, 2015
@xabbuh
bug #5488 fix #5487 (emillosanti)
5624ed7 
This PR was submitted for the 2.7 branch but it was merged into the 2.3 branch instead (closes #5488).

Discussion
----------

fix #5487

Commits
-------

68c7364 fix #5487
@xabbuh xabbuh closed this as completed Jul 7, 2015
weaverryan added a commit that referenced this issue Jul 7, 2015
@weaverryan
Merge branch '2.3' into 2.6
92e20cf 
* 2.3:
  Reordered the configuration blocks of the first example
  Fixed some errors and made some simplifications
  Added the "payload" option back (was removed by mistake)
  Fixed the order of the examples
  Simplified the first example and added more examples
  [Validator] Updated documentation of URL validator
  Fixed typos
  Removed duplication and moved a caution message
  [Console] Added a cookbook entry on invoking other commands
  [#5367] Making the titles more searchable
  fix #5487
  Changed Authorization and Authentication to use subsections
  Split Security into Authentication & Authorization
weaverryan added a commit that referenced this issue Jul 7, 2015
@weaverryan
Merge branch '2.6' into 2.7
6daf19f 
* 2.6:
  Reordered the configuration blocks of the first example
  Fixed some errors and made some simplifications
  Added the "payload" option back (was removed by mistake)
  Fixed the order of the examples
  Simplified the first example and added more examples
  [Validator] Updated documentation of URL validator
  Fixed typos
  Removed duplication and moved a caution message
  [Console] Added a cookbook entry on invoking other commands
  [#5367] Making the titles more searchable
  fix #5487
  Changed Authorization and Authentication to use subsections
  Split Security into Authentication & Authorization
weaverryan added a commit that referenced this issue Jul 7, 2015
@weaverryan
Merge branch '2.7' into 2.8
2f65bf2 
* 2.7:
  [#5426] Porting the checkDNS option to 2.7 only
  Reordered the configuration blocks of the first example
  Fixed some errors and made some simplifications
  Added the "payload" option back (was removed by mistake)
  Fixed the order of the examples
  Simplified the first example and added more examples
  [Validator] Updated documentation of URL validator
  Fixed typos
  Removed duplication and moved a caution message
  [Console] Added a cookbook entry on invoking other commands
  [#5367] Making the titles more searchable
  fix #5487
  [FrameworkBundle] Update serializer configuration reference
  Changed Authorization and Authentication to use subsections
  Split Security into Authentication & Authorization
weaverryan added a commit that referenced this issue Jul 7, 2015
@weaverryan
Merge branch '2.7'
6cc3445 
* 2.7:
  [#5426] Porting the checkDNS option to 2.7 only
  Reordered the configuration blocks of the first example
  Fixed some errors and made some simplifications
  Added the "payload" option back (was removed by mistake)
  Fixed the order of the examples
  Simplified the first example and added more examples
  [Validator] Updated documentation of URL validator
  Fixed typos
  Removed duplication and moved a caution message
  [Console] Added a cookbook entry on invoking other commands
  [#5367] Making the titles more searchable
  fix #5487
  [FrameworkBundle] Update serializer configuration reference
  Changed Authorization and Authentication to use subsections
  Split Security into Authentication & Authorization
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
hasPR A Pull Request has already been submitted for this issue.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@xabbuh @emillosanti