Skip to content

Conversation

Simperfit
Copy link
Contributor

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Tests pass? no
Fixed tickets
License MIT
Doc PR

@Simperfit Simperfit force-pushed the feature/add-hydra-test branch 3 times, most recently from 64296e8 to 282a2b2 Compare July 14, 2016 20:27
class ItemNormalizerTest extends \PHPUnit_Framework_TestCase
{
/**
* @expectedException \ReflectionException
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's weird isn't it?

@dunglas
Copy link
Member

dunglas commented Jul 15, 2016

I left some minor comments! Thank you very much for improving our test coverage.

@Simperfit Simperfit force-pushed the feature/add-hydra-test branch from 48f9c1e to 3cf292e Compare July 15, 2016 19:36
@Simperfit
Copy link
Contributor Author

@dunglas comments addressed

@Simperfit Simperfit changed the title fix: add CollectionNormalizerTest fix: add CollectionNormalizerTest && ItemNormalizerTest && ApiDocumentationBuilderTest Jul 15, 2016
@Simperfit Simperfit force-pushed the feature/add-hydra-test branch from fedca3e to 5016948 Compare July 15, 2016 20:45
@Simperfit Simperfit force-pushed the feature/add-hydra-test branch from 5016948 to f45abe6 Compare July 15, 2016 20:46
@Simperfit
Copy link
Contributor Author

@dunglas ApiDocumentationBuilder Test added.

@Simperfit Simperfit force-pushed the feature/add-hydra-test branch from 5a4b4aa to a068efc Compare July 15, 2016 21:10
@dunglas dunglas merged commit 38ff44c into api-platform:master Jul 16, 2016
@dunglas
Copy link
Member

dunglas commented Jul 16, 2016

Thank you @Simperfit

@Simperfit Simperfit deleted the feature/add-hydra-test branch August 13, 2016 06:44
magarzon pushed a commit to magarzon/core that referenced this pull request Feb 12, 2017
…test

fix: add CollectionNormalizerTest && ItemNormalizerTest && ApiDocumentationBuilderTest
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

Successfully merging this pull request may close these issues.

2 participants