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.

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

Fix of "Internal classes cannot be mocked without invoking their constructor in PHP 5.4.29/5.5.13" unit tests failure on Travis CI #618

Closed
wants to merge 2 commits into from

Conversation

orlangur
Copy link
Contributor

After the fix builds are green: https://travis-ci.org/orlangur/magento2

…tructor in PHP 5.4.29" unit tests failure on Travis CI
…tructor in PHP 5.4.29" unit tests failure on Travis CI

- fix issues appeared due to incomplete fix:
-- Argument 1 passed to Magento\Backend\Model\Menu::__construct() must be an instance of Magento\Framework\Logger, none given
-- DOMException: DOMElement::__construct() expects at least 1 parameter, 0 given
@verklov
Copy link
Contributor

verklov commented Jul 24, 2014

@orlangur, thank you for your contribution! We have deployed the code in version dev87. We are closing this pull request.

@verklov verklov closed this Jul 24, 2014
@verklov verklov self-assigned this Aug 5, 2014
magento-team pushed a commit that referenced this pull request Oct 21, 2015
magento-team pushed a commit that referenced this pull request May 14, 2016
mmansoor-magento pushed a commit that referenced this pull request Nov 25, 2016
MAGETWO-61132: Integration test Magento\Directory\Model\ObserverTest fails due to 3rd party service outage
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants