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

EZP-28527: Fix ContentTypeFormProcessorTest::testAddFieldDefinition test #190

Merged
merged 1 commit into from
Dec 14, 2017

Conversation

webhdx
Copy link
Contributor

@webhdx webhdx commented Dec 14, 2017

JIRA: https://jira.ez.no/browse/EZP-28527

Description

Test were broken by #187, this PR fixes failing test. This PR has to be merged up to the master.

@webhdx webhdx self-assigned this Dec 14, 2017
@webhdx webhdx requested review from lserwatka and andrerom December 14, 2017 10:52
@webhdx webhdx added the Bug label Dec 14, 2017
@webhdx webhdx changed the title EZP-28527: Fixes ContentTypeFormProcessorTest::testAddFieldDefinition test EZP-28527: Fix ContentTypeFormProcessorTest::testAddFieldDefinition test Dec 14, 2017
@andrerom
Copy link
Contributor

andrerom commented Dec 14, 2017

@webhdx seesm to still be some issue on PHP 5.6 on travis:

1) EzSystems\RepositoryForms\Tests\Form\Processor\ContentTypeFormProcessorTest::testAddFieldDefinition

Expectation failed for method name is equal to <string:addFieldDefinition> when invoked 1 time(s)

Parameter 1 for invocation eZ\Publish\API\Repository\ContentTypeService::addFieldDefinition(eZ\Publish\Core\Repository\Values\ContentType\ContentTypeDraft Object (...), eZ\Publish\API\Repository\Values\ContentType\FieldDefinitionCreateStruct Object (...)) does not match expected value.

Failed asserting that two objects are equal.
--- Expected
+++ Actual
@@ @@
 eZ\Publish\API\Repository\Values\ContentType\FieldDefinitionCreateStruct Object (
     'fieldTypeIdentifier' => 'ezstring'
-    'identifier' => 'new_ezstring_3'
+    'identifier' => 'new_ezstring_1'

@webhdx
Copy link
Contributor Author

webhdx commented Dec 14, 2017

@andrerom Is this only on PHP 5.6? It's not possible as this PR is fixing mentioned issue.

PHPUnit run locally for this PR:
image

@andrerom
Copy link
Contributor

andrerom commented Dec 14, 2017

restarting the job to see if travis is doing hick-ups on us, but indeed this is strange as the error is the same as before your fix. Seems also this branch is running on very old version of travis, but we can take that separately later.

So merging this so we can see what is remaining afterwards, will merge up to master.

@andrerom andrerom merged commit 7063efb into ezsystems:1.5 Dec 14, 2017
@webhdx webhdx deleted the broken_unit_tests_on_add_field_def branch February 23, 2018 08:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

3 participants