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

Update from 1.4.4 to 1.5.0 multiple issues #188

Closed
Maks3w opened this issue Sep 10, 2015 · 10 comments
Closed

Update from 1.4.4 to 1.5.0 multiple issues #188

Maks3w opened this issue Sep 10, 2015 · 10 comments

Comments

@Maks3w
Copy link
Contributor

Maks3w commented Sep 10, 2015

Does this project follow semantic versioning?

Undefined property: stdClass::$definitions
 C:\Proyectos\SwaggerAssertions\vendor\justinrainbow\json-schema\src\JsonSchema\RefResolver.php:270
 C:\Proyectos\SwaggerAssertions\vendor\justinrainbow\json-schema\src\JsonSchema\RefResolver.php:238
 C:\Proyectos\SwaggerAssertions\vendor\justinrainbow\json-schema\src\JsonSchema\RefResolver.php:123
 C:\Proyectos\SwaggerAssertions\src\SchemaManager.php:201
 C:\Proyectos\SwaggerAssertions\src\SchemaManager.php:65
 C:\Proyectos\SwaggerAssertions\src\PhpUnit\ResponseBodyConstraint.php:86
 C:\Proyectos\SwaggerAssertions\src\PhpUnit\ResponseBodyConstraint.php:52
 C:\Proyectos\SwaggerAssertions\src\PhpUnit\AssertsTrait.php:39
 C:\Proyectos\SwaggerAssertions\tests\PhpUnit\AssertsTraitTest.php:34
The maximum stack depth has been exceeded
 C:\Proyectos\SwaggerAssertions\vendor\justinrainbow\json-schema\src\JsonSchema\RefResolver.php:105
 C:\Proyectos\SwaggerAssertions\src\SchemaManager.php:201
 C:\Proyectos\SwaggerAssertions\src\SchemaManager.php:232
 C:\Proyectos\SwaggerAssertions\src\SchemaManager.php:77
 C:\Proyectos\SwaggerAssertions\src\PhpUnit\ResponseHeadersConstraint.php:87
 C:\Proyectos\SwaggerAssertions\src\PhpUnit\ResponseHeadersConstraint.php:52
 C:\Proyectos\SwaggerAssertions\tests\PhpUnit\ResponseHeadersConstraintTest.php:42

From https://github.com/Maks3w/SwaggerAssertions test suite.

Maks3w added a commit to Maks3w/SwaggerAssertions that referenced this issue Sep 10, 2015
justinrainbow/json-schema: 1.5.0 brakes current implementation

jsonrainbow/json-schema#188
@bighappyface
Copy link
Collaborator

@Maks3w welcome back.

There is discussion (see issues) around the addition of an exception in 1.4.4 for maximum depth, and this is the first I am seeing of anything related to the definitions property. Thanks for the report.

What about semver are you expecting presently?

@Maks3w
Copy link
Contributor Author

Maks3w commented Sep 10, 2015

Maximum depth exception was added on 1.4.4. The issue is upgrading from 1.4.4 to 1.5.0

I expect no backward compatibility issues while upgrading between minor versions.

@Maks3w
Copy link
Contributor Author

Maks3w commented Sep 10, 2015

$definitions it's part of the json schema under test (Swagger v2.0 specification)

@Maks3w
Copy link
Contributor Author

Maks3w commented Feb 11, 2016

This still broken in 1.6.1

@bighappyface
Copy link
Collaborator

See #210

@jojo1981
Copy link

See #245

@bighappyface
Copy link
Collaborator

bighappyface commented Apr 14, 2016

Closed with #245

@Maks3w
Copy link
Contributor Author

Maks3w commented Apr 14, 2016

Can you elaborate why #174 is related to this?

@jojo1981
Copy link

@Maks3w I think @bighappyface did mean #245

@bighappyface
Copy link
Collaborator

@jojo1981 @Maks3w yes, I meant #245

Sorry for the confusion

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

No branches or pull requests

3 participants