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

Commit f6b89cb

Browse files
weierophinneyweierophinney
committed
Stabilize versions, and selectively omit tests
- Updates all dependencies to stable versions; if the dependency has forwards-compat changes for v3 components, pins to those. - Selectively skip zend-db, zend-session tests when testing against servicemanager v3.
1 parent 08ddcf2 commit f6b89cb

File tree

6 files changed

+106
-37
lines changed

6 files changed

+106
-37
lines changed

‎.travis.yml‎

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -24,21 +24,21 @@ matrix:
2424
- EXECUTE_HOSTNAME_CHECK=true
2525
- php: 5.5
2626
env:
27-
- SERVICE_MANAGER_V2=true
27+
- SERVICE_MANAGER_VERSION="^2.7.5"
2828
- php: 5.6
2929
env:
3030
- EXECUTE_TEST_COVERALLS=true
3131
- php: 5.6
3232
env:
33-
- SERVICE_MANAGER_V2=true
33+
- SERVICE_MANAGER_VERSION="^2.7.5"
3434
- php: 7
3535
- php: 7
3636
env:
37-
- SERVICE_MANAGER_V2=true
37+
- SERVICE_MANAGER_VERSION="^2.7.5"
3838
- php: hhvm
3939
- php: hhvm
4040
env:
41-
- SERVICE_MANAGER_V2=true
41+
- SERVICE_MANAGER_VERSION="^2.7.5"
4242
allow_failures:
4343
- php: hhvm
4444

@@ -50,8 +50,9 @@ before_install:
5050
- if [[ $EXECUTE_TEST_COVERALLS != 'true' ]]; then phpenv config-rm xdebug.ini || return 0 ; fi
5151
- composer self-update
5252
- if [[ $EXECUTE_TEST_COVERALLS == 'true' ]]; then composer require --dev --no-update satooshi/php-coveralls ; fi
53-
- if [[ $SERVICE_MANAGER_V2 == 'true' ]]; then composer require --dev --no-update "zendframework/zend-servicemanager:^2.7.2" ; fi
54-
- if [[ $SERVICE_MANAGER_V2 != 'true' ]]; then composer require --dev --no-update "zendframework/zend-servicemanager:^3.0" ; fi
53+
- if [[ $SERVICE_MANAGER_VERSION != '' ]]; then composer require --dev --no-update "zendframework/zend-servicemanager:$SERVICE_MANAGER_VERSION" ; fi
54+
- if [[ $SERVICE_MANAGER_VERSION == '' ]]; then composer require --dev --no-update "zendframework/zend-servicemanager:^3.0.3" ; fi
55+
- if [[ $SERVICE_MANAGER_VERSION == '' ]]; then composer remove --dev --no-update zendframework/zend-db zendframework/zend-session ; fi
5556

5657
install:
5758
- travis_retry composer install --no-interaction --ignore-platform-reqs

‎composer.json‎

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -18,18 +18,18 @@
1818
"container-interop/container-interop": "^1.1"
1919
},
2020
"require-dev": {
21-
"zendframework/zend-cache": "dev-develop as 2.6.0",
21+
"zendframework/zend-cache": "^2.6.1",
2222
"zendframework/zend-config": "^2.6",
23-
"zendframework/zend-db": "dev-develop as 2.7.0",
23+
"zendframework/zend-db": "^2.5",
2424
"zendframework/zend-filter": "^2.6",
25-
"zendframework/zend-http": "^2.5",
26-
"zendframework/zend-i18n": "dev-develop as 2.6.0",
25+
"zendframework/zend-http": "^2.5.4",
26+
"zendframework/zend-i18n": "^2.6",
2727
"zendframework/zend-math": "^2.6",
2828
"zendframework/zend-servicemanager": "^2.7.5 || ^3.0.3",
29-
"zendframework/zend-session": "dev-develop as 2.6.0",
29+
"zendframework/zend-session": "^2.5",
3030
"zendframework/zend-uri": "^2.5",
3131
"fabpot/php-cs-fixer": "1.7.*",
32-
"phpunit/PHPUnit": "~4.0"
32+
"phpunit/PHPUnit": "^4.0"
3333
},
3434
"suggest": {
3535
"zendframework/zend-db": "Zend\\Db component",

‎test/CsrfTest.php‎

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,13 @@ class CsrfTest extends \PHPUnit_Framework_TestCase
2828

2929
public function setUp()
3030
{
31+
if (! class_exists(Container::class)) {
32+
$this->markTestSkipped(
33+
'Skipping zend-session-related tests until the component is updated '
34+
. ' to zend-servicemanager/zend-eventmanager v3'
35+
);
36+
}
37+
3138
// Setup session handling
3239
$_SESSION = [];
3340
$sessionConfig = new StandardConfig([
@@ -42,6 +49,10 @@ public function setUp()
4249

4350
public function tearDown()
4451
{
52+
if (! class_exists(Container::class)) {
53+
return;
54+
}
55+
4556
$_SESSION = [];
4657
Container::setDefaultManager(null);
4758
}

‎test/Db/AbstractDbTest.php‎

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,13 @@ class AbstractDbTest extends \PHPUnit_Framework_TestCase
2424

2525
public function setUp()
2626
{
27+
if (! class_exists('Zend\Db\Adapter\Adapter')) {
28+
$this->markTestSkipped(
29+
'Skipping zend-db-related tests until that component is updated '
30+
. 'to zend-servicemanager/zend-eventmanager v3'
31+
);
32+
}
33+
2734
$this->validator = new ConcreteDbValidator([
2835
'table' => 'table',
2936
'field' => 'field',

‎test/Db/NoRecordExistsTest.php‎

Lines changed: 39 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,16 @@
1818
*/
1919
class NoRecordExistsTest extends \PHPUnit_Framework_TestCase
2020
{
21+
public function setUp()
22+
{
23+
if (! class_exists('Zend\Db\Adapter\Adapter')) {
24+
$this->markTestSkipped(
25+
'Skipping zend-db-related tests until that component is updated '
26+
. 'to zend-servicemanager/zend-eventmanager v3'
27+
);
28+
}
29+
}
30+
2131
/**
2232
* Return a Mock object for a Db result with rows
2333
*
@@ -121,8 +131,12 @@ public function testBasicFindsNoRecord()
121131
*/
122132
public function testExcludeWithArray()
123133
{
124-
$validator = new NoRecordExists('users', 'field1', ['field' => 'id', 'value' => 1],
125-
$this->getMockHasResult());
134+
$validator = new NoRecordExists(
135+
'users',
136+
'field1',
137+
['field' => 'id', 'value' => 1],
138+
$this->getMockHasResult()
139+
);
126140
$this->assertFalse($validator->isValid('value3'));
127141
}
128142

@@ -134,8 +148,12 @@ public function testExcludeWithArray()
134148
*/
135149
public function testExcludeWithArrayNoRecord()
136150
{
137-
$validator = new NoRecordExists('users', 'field1', ['field' => 'id', 'value' => 1],
138-
$this->getMockNoResult());
151+
$validator = new NoRecordExists(
152+
'users',
153+
'field1',
154+
['field' => 'id', 'value' => 1],
155+
$this->getMockNoResult()
156+
);
139157
$this->assertTrue($validator->isValid('nosuchvalue'));
140158
}
141159

@@ -172,8 +190,10 @@ public function testExcludeWithStringNoRecord()
172190
public function testThrowsExceptionWithNoAdapter()
173191
{
174192
$validator = new NoRecordExists('users', 'field1', 'id != 1');
175-
$this->setExpectedException('Zend\Validator\Exception\RuntimeException',
176-
'No database adapter present');
193+
$this->setExpectedException(
194+
'Zend\Validator\Exception\RuntimeException',
195+
'No database adapter present'
196+
);
177197
$validator->isValid('nosuchvalue');
178198
}
179199

@@ -184,8 +204,10 @@ public function testThrowsExceptionWithNoAdapter()
184204
*/
185205
public function testWithSchema()
186206
{
187-
$validator = new NoRecordExists(['table' => 'users', 'schema' => 'my'],
188-
'field1', null, $this->getMockHasResult());
207+
$validator = new NoRecordExists([
208+
'table' => 'users',
209+
'schema' => 'my'
210+
], 'field1', null, $this->getMockHasResult());
189211
$this->assertFalse($validator->isValid('value1'));
190212
}
191213

@@ -196,15 +218,20 @@ public function testWithSchema()
196218
*/
197219
public function testWithSchemaNoResult()
198220
{
199-
$validator = new NoRecordExists(['table' => 'users', 'schema' => 'my'],
200-
'field1', null, $this->getMockNoResult());
221+
$validator = new NoRecordExists([
222+
'table' => 'users',
223+
'schema' => 'my'
224+
], 'field1', null, $this->getMockNoResult());
201225
$this->assertTrue($validator->isValid('value1'));
202226
}
203227

204228
public function testEqualsMessageTemplates()
205229
{
206230
$validator = new NoRecordExists('users', 'field1');
207-
$this->assertAttributeEquals($validator->getOption('messageTemplates'),
208-
'messageTemplates', $validator);
231+
$this->assertAttributeEquals(
232+
$validator->getOption('messageTemplates'),
233+
'messageTemplates',
234+
$validator
235+
);
209236
}
210237
}

‎test/Db/RecordExistsTest.php‎

Lines changed: 36 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,16 @@
2121
*/
2222
class RecordExistsTest extends \PHPUnit_Framework_TestCase
2323
{
24+
public function setUp()
25+
{
26+
if (! class_exists(Adapter::class)) {
27+
$this->markTestSkipped(
28+
'Skipping zend-db-related tests until that component is updated '
29+
. 'to zend-servicemanager/zend-eventmanager v3'
30+
);
31+
}
32+
}
33+
2434
/**
2535
* Return a Mock object for a Db result with rows
2636
*
@@ -102,9 +112,11 @@ protected function getMockNoResult()
102112
*/
103113
public function testBasicFindsRecord()
104114
{
105-
$validator = new RecordExists(['table' => 'users',
106-
'field' => 'field1',
107-
'adapter' => $this->getMockHasResult()]);
115+
$validator = new RecordExists([
116+
'table' => 'users',
117+
'field' => 'field1',
118+
'adapter' => $this->getMockHasResult()
119+
]);
108120
$this->assertTrue($validator->isValid('value1'));
109121
}
110122

@@ -197,8 +209,10 @@ public function testExcludeConstructor()
197209
public function testThrowsExceptionWithNoAdapter()
198210
{
199211
$validator = new RecordExists('users', 'field1', 'id != 1');
200-
$this->setExpectedException('Zend\Validator\Exception\RuntimeException',
201-
'No database adapter present');
212+
$this->setExpectedException(
213+
'Zend\Validator\Exception\RuntimeException',
214+
'No database adapter present'
215+
);
202216
$validator->isValid('nosuchvalue');
203217
}
204218

@@ -209,8 +223,10 @@ public function testThrowsExceptionWithNoAdapter()
209223
*/
210224
public function testWithSchema()
211225
{
212-
$validator = new RecordExists(['table' => 'users', 'schema' => 'my'],
213-
'field1', null, $this->getMockHasResult());
226+
$validator = new RecordExists([
227+
'table' => 'users',
228+
'schema' => 'my'
229+
], 'field1', null, $this->getMockHasResult());
214230
$this->assertTrue($validator->isValid('value1'));
215231
}
216232

@@ -221,8 +237,10 @@ public function testWithSchema()
221237
*/
222238
public function testWithSchemaNoResult()
223239
{
224-
$validator = new RecordExists(['table' => 'users', 'schema' => 'my'],
225-
'field1', null, $this->getMockNoResult());
240+
$validator = new RecordExists([
241+
'table' => 'users',
242+
'schema' => 'my'
243+
], 'field1', null, $this->getMockNoResult());
226244
$this->assertFalse($validator->isValid('value1'));
227245
}
228246

@@ -232,8 +250,10 @@ public function testWithSchemaNoResult()
232250
*/
233251
public function testSelectAcknowledgesTableAndSchema()
234252
{
235-
$validator = new RecordExists(['table' => 'users', 'schema' => 'my'],
236-
'field1', null, $this->getMockHasResult());
253+
$validator = new RecordExists([
254+
'table' => 'users',
255+
'schema' => 'my'
256+
], 'field1', null, $this->getMockHasResult());
237257
$table = $validator->getSelect()->getRawState('table');
238258
$this->assertInstanceOf('Zend\Db\Sql\TableIdentifier', $table);
239259
$this->assertEquals(['users', 'my'], $table->getTableAndSchema());
@@ -242,8 +262,11 @@ public function testSelectAcknowledgesTableAndSchema()
242262
public function testEqualsMessageTemplates()
243263
{
244264
$validator = new RecordExists('users', 'field1');
245-
$this->assertAttributeEquals($validator->getOption('messageTemplates'),
246-
'messageTemplates', $validator);
265+
$this->assertAttributeEquals(
266+
$validator->getOption('messageTemplates'),
267+
'messageTemplates',
268+
$validator
269+
);
247270
}
248271

249272
/**

0 commit comments

Comments
 (0)