-
Notifications
You must be signed in to change notification settings - Fork 129
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
OpenCloud driver #266
Comments
OK, I'm converting the tests to use Guzzle's MockPlugin to simulate the responses from the API. That way at least the php-opencloud touch points get tested. |
My code is available over here. I'd do a PR right away, but I'm beginning to think this would be better in a separate repository (as maybe would the other drivers?). There are a couple of reasons:
@henrikbjorn - what are your thoughts? I'll do a PR against this repo if you want. Or if you think a separate repo is a good idea, create one and I'll do a PR against that. Cheers! |
Hey @kynx I created https://github.com/bernardphp/OpenCloud for you. |
Awesome! PR is at bernardphp/OpenCloud#1 |
Moving to bernardphp/drivers#12 |
I've written a driver for php-opencloud which I'd like to send a PR for. But I've got a couple of questions...
pushMessage()
to a non-existent queue it'll get created for you. But it doesn't look like SQS or IronMQ work that way (though I'm not at all familiar with the underlying API). What's right?peekQueue()
I think it probably should. Is that right?The tests I've got require a Rackspace account. I don't like mocking what I don't know, but I guess I could skip them if an account isn't available and add some mocks. What's the best approach here?Thanks!
The text was updated successfully, but these errors were encountered: