Skip to content

Conversation

@chellem
Copy link

@chellem chellem commented May 18, 2018

to include:

  • statistics
  • custom attributes

to include:
- statistics
- custom attributes
@chellem
Copy link
Author

chellem commented May 26, 2018

@m1guelpf Is it possible to check this pull request ?

@m1guelpf
Copy link
Member

@chellem Can you add some tests?

@m1guelpf
Copy link
Member

m1guelpf commented May 27, 2018

1) Gitlab\Tests\Api\ProjectsTest::shouldShowProjectWithStatistics
Expectation failed for method name is equal to <string:get> when invoked 1 time(s)
Parameter 1 for invocation Gitlab\Api\AbstractApi::get('projects/1', Array (...), Array ()) does not match expected value.
Failed asserting that two arrays are equal.
--- Expected
+++ Actual
@@ @@
 Array (
-    'statistics' => true
+    'statistics' => 'true'
 )

@chellem
Copy link
Author

chellem commented May 27, 2018

@m1guelpf solved it. All green now 👍

@m1guelpf m1guelpf merged commit 5a3ef56 into GitLabPHP:master May 27, 2018
@m1guelpf
Copy link
Member

Thank you for your contribution, @chellem!

@chellem
Copy link
Author

chellem commented May 27, 2018

My pleasure 💯

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants