You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by GitBox <gi...@apache.org> on 2022/08/25 08:28:59 UTC
[GitHub] [camel-quarkus] jamesnetherton opened a new issue, #4047: Use WireMock for Dropbox integration tests
jamesnetherton opened a new issue, #4047:
URL: https://github.com/apache/camel-quarkus/issues/4047
Not sure if it is possible or not but it'd be worth investigating to see if we have the tests run on the CI build without requiring real credentials.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-quarkus] ppalaga commented on issue #4047: Use WireMock for Dropbox integration tests
Posted by "ppalaga (via GitHub)" <gi...@apache.org>.
ppalaga commented on issue #4047:
URL: https://github.com/apache/camel-quarkus/issues/4047#issuecomment-1402019721
Thanks for your patient investigation, @djcoleman !
We should perhaps try our luck asking the Dropbox client team for a proper base URI support? https://github.com/dropbox/dropbox-sdk-java/issues
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-quarkus] djcoleman commented on issue #4047: Use WireMock for Dropbox integration tests
Posted by "djcoleman (via GitHub)" <gi...@apache.org>.
djcoleman commented on issue #4047:
URL: https://github.com/apache/camel-quarkus/issues/4047#issuecomment-1402205472
I found a [question](https://www.dropboxforum.com/t5/Discuss-Dropbox-Developer-API/Unable-to-customize-port-and-protocol-in-dropbox-SDK/td-p/356875) from 2019 on dropboxforum.com requesting WireMock support. The responder acknowledged the problem and promised to submit a feature request, although he didn't make any commitments to implementing it.
I don't see any tickets in the dropbox-sdk-java issues list already though, so I can submit a request myself and see if anything happens.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-quarkus] djcoleman commented on issue #4047: Use WireMock for Dropbox integration tests
Posted by "djcoleman (via GitHub)" <gi...@apache.org>.
djcoleman commented on issue #4047:
URL: https://github.com/apache/camel-quarkus/issues/4047#issuecomment-1401772018
At first I thought this was going to be a fairly quick task based on the Geocoder tests, but I found myself heading down a warren of rabbit holes with this.
In order to test the Dropbox extension with Wiremock, the wiremock server URL must be passed to the dropbox client within the Camel component.
I first tried creating an instance of `DbxClientV2` (the client supplied by the Dropbox SDK), but it had limited support for passing in a new host - I had to access a private constructor via reflection to be able to pass in the Wiremock URL as well as the credentials.
The next issue was setting the client in the Camel component. I had to create a sub-class of the `DropboxComponent` and register that in place of the original. The sub-class overrode the `createEndpoint` method that first called the parent implementation and then accessed the `DropboxConfiguration` object, to which I could pass the `DbxClientV2` object.
However, this still did not work as the `DbxClientV2` object doesn't handle port numbers - it only expects hostnames, so URL creation fails.
I don't know if the Wiremock proxy support can be used to create an alias that can be used instead of `localhost:{randomPort}`, but I've spent more time than I'd intended on this already.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Use WireMock for Dropbox integration tests [camel-quarkus]
Posted by "jamesnetherton (via GitHub)" <gi...@apache.org>.
jamesnetherton closed issue #4047: Use WireMock for Dropbox integration tests
URL: https://github.com/apache/camel-quarkus/issues/4047
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-quarkus] ppalaga commented on issue #4047: Use WireMock for Dropbox integration tests
Posted by "ppalaga (via GitHub)" <gi...@apache.org>.
ppalaga commented on issue #4047:
URL: https://github.com/apache/camel-quarkus/issues/4047#issuecomment-1408181990
@djcoleman could you please paste the Dropbox issue URL here?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-quarkus] djcoleman commented on issue #4047: Use WireMock for Dropbox integration tests
Posted by "djcoleman (via GitHub)" <gi...@apache.org>.
djcoleman commented on issue #4047:
URL: https://github.com/apache/camel-quarkus/issues/4047#issuecomment-1408235302
Here it is: https://github.com/dropbox/dropbox-sdk-java/issues/487
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org