feat: add method to set emulator host programmatically (#319)#336
Conversation
Codecov Report
@@ Coverage Diff @@
## master #336 +/- ##
============================================
- Coverage 72.75% 72.64% -0.11%
- Complexity 1044 1047 +3
============================================
Files 64 64
Lines 5512 5535 +23
Branches 681 686 +5
============================================
+ Hits 4010 4021 +11
- Misses 1289 1301 +12
Partials 213 213
Continue to review full report at Codecov.
|
google-cloud-firestore/src/main/java/com/google/cloud/firestore/FirestoreOptions.java
Outdated
Show resolved
Hide resolved
|
Apologies on the delayed review, one small request in the javadoc comment but other than it looks good. |
f0f80c4 to
e1e3886
Compare
|
Thank you for your input. Updated the JavaDoc. |
|
Thanks for adding this @Gerschtli! I was successfully able to use it in a testing project by doing the following: final Firestore ds = FirestoreOptions.getDefaultInstance().toBuilder()
.setEmulatorHost("localhost:8081")
.setCredentials(new FirestoreOptions.EmulatorCredentials())
.setProjectId("testing")
.build()
.getService(); |
|
In fact you should be able to drop the line |
|
The credentials line is necessary for some admin related operations, so I always include it. |
|
But the credentials will be overridden in the |
🤖 I have created a release \*beep\* \*boop\* --- ## [2.1.0](https://github.com/googleapis/java-firestore/compare/v2.0.0...v2.1.0) (2020-09-10) ### Features * add method to set emulator host programmatically ([#319](https://github.com/googleapis/java-firestore/issues/319)) ([#336](https://github.com/googleapis/java-firestore/issues/336)) ([97037f4](https://github.com/googleapis/java-firestore/commit/97037f42f76e9df3ae458d4e2b04336e64b834c3)), closes [#210](https://github.com/googleapis/java-firestore/issues/210) [#190](https://github.com/googleapis/java-firestore/issues/190) * add opencensus tracing support ([#360](https://github.com/googleapis/java-firestore/issues/360)) ([edaa539](https://github.com/googleapis/java-firestore/commit/edaa5395be0353fb261d954429c624623bc4e346)) * add support for != and NOT_IN queries ([#350](https://github.com/googleapis/java-firestore/issues/350)) ([68aff5b](https://github.com/googleapis/java-firestore/commit/68aff5b406fb2732951750f3d5f9768df6ee12b5)) * generate protos to add NOT_EQUAL, NOT_IN, IS_NOT_NAN, IS_NOT_NULL query operators ([#343](https://github.com/googleapis/java-firestore/issues/343)) ([3fb1b63](https://github.com/googleapis/java-firestore/commit/3fb1b631f8dd087f0f3e1c43363e9642f497993a)) ### Bug Fixes * **samples:** re-add maven exec config for Quickstart sample ([#347](https://github.com/googleapis/java-firestore/issues/347)) ([4c2329b](https://github.com/googleapis/java-firestore/commit/4c2329bf89ffab4bd3060e16e1cf231b7caf4653)) * add support to deserialize to custom Lists and Maps ([#337](https://github.com/googleapis/java-firestore/issues/337)) ([dc897e0](https://github.com/googleapis/java-firestore/commit/dc897e00a85e745f57f615460b29d17b7dd247c6)) ### Dependencies * update dependency com.google.cloud:google-cloud-shared-dependencies to v0.9.0 ([#352](https://github.com/googleapis/java-firestore/issues/352)) ([783d41e](https://github.com/googleapis/java-firestore/commit/783d41e167c7c79957faeeebd7a76ab72b5b157d)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please).
See for initial discussion: #319
Fixes #319, #210.
Relates to #190.
Similar issues in other SDKs: googleapis/google-cloud-go#1978, firebase/firebase-admin-node#776.