Adds space_guid, organization_guid as parameter to the list events - #890
Adds space_guid, organization_guid as parameter to the list events#890arghya88 wants to merge 1 commit into
Conversation
|
Hey arghya88! Thanks for submitting this pull request! I'm here to inform the recipients of the pull request that you and the commit authors have already signed the CLA. |
|
Thanks @arghya88 - note the pluralisation of parameters, and the error in the integration test (though I understand that's hard to check without a test environment!) |
|
@twoseat I looked at the integration test...what error you got and why you had to do this change if you can explain ?..I have a cf but I am not sure how/if I can run only this test.Tried running the whole set of integration tests but some other test related to buildpack keeps failing.. |
|
@twoseat Noted on this...Can you explain what failed and why the reordering the tests were necessary? I just tried running this test and it fails with below error on my cf instance: Found unexpected property stack in payload for org.cloudfoundry.client.v2.buildpacks.BuildpackEntity$Json Any idea? |
|
Hi @arghya88! The The reordering of tests isn't strictly necessary, but we try to keep everything in alphabetical order (there are exceptions, but not many!). Fortunately hitting alt-cmd-L inside a class in Intellij generally takes care of such things. The error you're seeing typically means that CF has returned a field that we don't know about. A quick check of the docs suggests that |
|
Got it...doing cf curl v2/buildpacks gives me a json which has stacks in it..api version is 2.111.0 |
closes #889