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

Commit

Permalink
Remove unneeded dependencies
Browse files Browse the repository at this point in the history
Removes each of:

- zend-filter
- zend-form
- zend-log
- zend-view

as each component was used only to test specific instantiation patterns, each of
which could be mimiced via generic test assets or PHP built-in classes.
  • Loading branch information
weierophinney committed Feb 23, 2016
1 parent 8e9fdcd commit 8216bd1
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 15 deletions.
9 changes: 1 addition & 8 deletions composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -19,22 +19,15 @@
"zendframework/zend-stdlib": "^2.7 || ^3.0"
},
"require-dev": {
"zendframework/zend-filter": "^2.6",
"zendframework/zend-form": "^2.7",
"zendframework/zend-log": "^2.7",
"zendframework/zend-view": "^2.6",
"fabpot/php-cs-fixer": "1.7.*",
"phpunit/PHPUnit": "~4.0"
},
"suggest": {
"zendframework/zend-servicemanager": "Zend\\ServiceManager component"
},
"minimum-stability": "dev",
"prefer-stable": true,
"extra": {
"branch-alias": {
"dev-master": "2.5-dev",
"dev-develop": "3.0-dev"
"dev-develop": "2.6-dev"
}
},
"autoload-dev": {
Expand Down
11 changes: 4 additions & 7 deletions test/DiCompatibilityTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@
use BadMethodCallException;
use Exception;
use PHPUnit_Framework_Error;
use SplStack;
use Zend\Di\Di;

class DiCompatibilityTest extends \PHPUnit_Framework_TestCase
Expand Down Expand Up @@ -41,13 +42,9 @@ public function testDiSimple($class)
public function providesSimpleClasses()
{
return [
['Zend\Di\Di'],
['Zend\EventManager\EventManager'],
['Zend\Filter\ToNull'],
['Zend\Form\Form'],
['Zend\Log\Logger'],
['Zend\Stdlib\SplStack'],
['Zend\View\Model\ViewModel'],
[Di::class],
[SplStack::class],
[TestAsset\BasicClass::class],
];
}

Expand Down

0 comments on commit 8216bd1

Please sign in to comment.