Skip to content

Commit 51e22de

Browse files
author
Bernhard Schmitt
committed
Adjust test suite
1 parent 8af42ea commit 51e22de

4 files changed

Lines changed: 12 additions & 57 deletions

File tree

.github/workflows/qa.yml

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -53,9 +53,7 @@ jobs:
5353
5454
- name: Run phpstan
5555
run: |
56-
bin/phpstan analyse \
57-
--level 8 \
58-
Tests/Unit Classes
56+
composer lint
5957
6058
unit-tests:
6159
runs-on: ubuntu-20.04
@@ -87,7 +85,4 @@ jobs:
8785
8886
- name: Run unit tests
8987
run: |
90-
bin/phpunit -c phpunit.xml \
91-
--enforce-time-limit \
92-
--coverage-text \
93-
Tests
88+
composer test

Tests/Unit/Fusion/AbstractComponentPresentationObjectTest.php

Lines changed: 0 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -23,22 +23,6 @@
2323
*/
2424
final class AbstractComponentPresentationObjectTest extends UnitTestCase
2525
{
26-
/**
27-
* @test
28-
* @small
29-
* @return void
30-
*/
31-
public function enforcesStructuralPropertyAccessToCircumventFaultToleranceInEel(): void
32-
{
33-
$this->expectException(\BadMethodCallException::class);
34-
35-
$presentationObject = new class extends AbstractComponentPresentationObject {
36-
};
37-
38-
// @phpstan-ignore-next-line
39-
$presentationObject->getFoo();
40-
}
41-
4226
/**
4327
* @return array<mixed>
4428
*/

Tests/Unit/Fusion/PresentationObjectComponentImplementationTest.php

Lines changed: 2 additions & 34 deletions
Original file line numberDiff line numberDiff line change
@@ -47,40 +47,11 @@ public function tearDownPresentationObjectComponentImplementation(): void
4747
}
4848

4949
/**
50-
* @test
50+
* @todo adjust to Neos 8
5151
* @throws \ReflectionException
5252
* @return void
5353
*/
54-
public function prepareProperlyMergesPropsToStubbedPresentationObjectInPreviewMode(): void
55-
{
56-
$runtime = $this->prophet->prophesize(Runtime::class);
57-
58-
$subject = new PresentationObjectComponentImplementation(
59-
$runtime->reveal(),
60-
'test',
61-
'My.Package:Component'
62-
);
63-
$subject['foo'] = 'bar';
64-
65-
$runtime
66-
->getCurrentContext()
67-
->willReturn([]);
68-
$runtime
69-
->evaluate('test/foo', $subject)
70-
->willReturn('bar');
71-
72-
$context = $this->getPrepare()->invokeArgs($subject, [[]]);
73-
74-
$this->assertSame(['foo' => 'bar'], $context['props']);
75-
$this->assertSame(['foo' => 'bar'], $context[PresentationObjectComponentImplementation::OBJECT_NAME]);
76-
}
77-
78-
/**
79-
* @test
80-
* @throws \ReflectionException
81-
* @return void
82-
*/
83-
public function prepareWritesPresentationObjectToContextWhenNotInPreviewMode(): void
54+
public function prepareWritesPresentationObjectToContext(): void
8455
{
8556
$runtime = $this->prophet->prophesize(Runtime::class);
8657
$presentationObject = $this->prophet->prophesize(ComponentPresentationObjectInterface::class);
@@ -91,9 +62,6 @@ public function prepareWritesPresentationObjectToContextWhenNotInPreviewMode():
9162
'My.Package:Component'
9263
);
9364

94-
$runtime
95-
->getCurrentContext()
96-
->willReturn([]);
9765
$runtime
9866
->evaluate('test/' . PresentationObjectComponentImplementation::OBJECT_NAME, $subject)
9967
->willReturn($presentationObject);

composer.json

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -22,8 +22,16 @@
2222
"mikey179/vfsstream": "^1.6",
2323
"squizlabs/php_codesniffer": "^3.6",
2424
"jangregor/phpstan-prophecy": "^1.0",
25+
"phpspec/prophecy": "^1.15",
2526
"spatie/phpunit-snapshot-assertions": "^4.2"
2627
},
28+
"suggest": {
29+
"sitegeist/monocle-presentationobjects": "For Monocle integration"
30+
},
31+
"scripts": {
32+
"lint": "bin/phpunit -c phpunit.xml --enforce-time-limit --coverage-html Build/Reports/coverage Tests",
33+
"test": "bin/phpunit -c phpunit.xml --enforce-time-limit --coverage-html Build/Reports/coverage Tests"
34+
},
2735
"config": {
2836
"vendor-dir": "Packages/Libraries",
2937
"bin-dir": "bin",

0 commit comments

Comments
 (0)