Skip to content

Conversation

pmakani
Copy link
Contributor

@pmakani pmakani commented Jun 23, 2020

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

  • Make sure to open an issue as a bug/issue before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
  • Ensure the tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)
@pmakani pmakani requested a review from stephaniewang526 June 23, 2020 11:27
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jun 23, 2020
@codecov
Copy link

codecov bot commented Jun 23, 2020

Codecov Report

Merging #478 into master will decrease coverage by 0.14%.
The diff coverage is n/a.

Impacted file tree graph

@@ Coverage Diff @@ ## master #478 +/- ## ============================================ - Coverage 81.49% 81.35% -0.15%  Complexity 1226 1226 ============================================ Files 77 77 Lines 6220 6220 Branches 691 685 -6 ============================================ - Hits 5069 5060 -9  - Misses 792 797 +5  - Partials 359 363 +4 
Impacted Files Coverage Δ Complexity Δ
...n/java/com/google/cloud/bigquery/BigQueryImpl.java 82.54% <0.00%> (-1.05%) 74.00% <0.00%> (ø%)
...ogle/cloud/bigquery/WriteChannelConfiguration.java 86.36% <0.00%> (-0.46%) 46.00% <0.00%> (ø%)
...om/google/cloud/bigquery/LoadJobConfiguration.java 92.83% <0.00%> (-0.38%) 56.00% <0.00%> (ø%)
...m/google/cloud/bigquery/QueryJobConfiguration.java 89.80% <0.00%> (-0.28%) 66.00% <0.00%> (ø%)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 82341cb...315cbb5. Read the comment docs.

@Test
public void testAuthUserFlow() {
// TODO(stephaniewang526): Replace client_secret.json
/*File credentialsPath = new File(""path/to/your/client_secret.json");
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we cannot upload any files that contains secrets -- have to use GCS
String sourceUri = "gs://cloud-devrel-kokoro-resources/java-docs-samples/service-acct.json"; so we have to figure out how to generate File from gcs path.

@stephaniewang526 stephaniewang526 added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jul 6, 2020
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jul 6, 2020
selectedScopes)
.setDataStoreFactory(new FileDataStoreFactory(DATA_STORE_DIR))
.setAccessType("offline")
.setApprovalPrompt("force")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

change to "auto" so user does not need to manually confirm

@stephaniewang526 stephaniewang526 added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jul 6, 2020
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jul 6, 2020
@pmakani pmakani added kokoro:force-run Add this label to force Kokoro to re-run the tests. and removed kokoro:force-run Add this label to force Kokoro to re-run the tests. labels Jul 7, 2020
@stephaniewang526 stephaniewang526 added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jul 7, 2020
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jul 7, 2020
@stephaniewang526
Copy link
Contributor

since we cannot manually approve approval prompt, we should try to do mock testing instead here.

@pmakani pmakani requested a review from a team July 17, 2020 06:10
@stephaniewang526 stephaniewang526 merged commit 65dffac into googleapis:master Jul 17, 2020
@pmakani pmakani deleted the auth-user-flow branch July 20, 2020 11:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

4 participants