Skip to content
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

feat(java_templates): add sample pom files #430

Merged
merged 3 commits into from
Mar 16, 2020

Conversation

chingor13
Copy link
Contributor

Generates initial sample pom.xml files. In an existing file exists, we will not overwrite it (for now).

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Mar 9, 2020
@chingor13 chingor13 requested a review from a team March 9, 2020 18:14
@@ -0,0 +1,62 @@
{% set group_id = metadata['repo']['distribution_name'].split(':')|first -%}
Copy link
Contributor

Choose a reason for hiding this comment

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

If there's a typo in this file such that the {% %} chunks do not parse correctly, or the resulting XML does not parse correctly, will a test fail?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

For a python runtime error, the synth job will crash (autosynth will open an issue). For invalid generated XML, the client library samples test would fail.

We could try to check for both (probably outside this PR) - add an java template test using a fixture directory and for the xml, try to parse the resulting output. For better or worse, we've been treating our array of synth.py files as our test set as a synth failure is not that bad.

Copy link
Contributor

Choose a reason for hiding this comment

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

Ok. Please create a bug and assign it to yourself to add such a test.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done. Created #436

@chingor13 chingor13 added the automerge Merge the pull request once unit tests and other checks pass. label Mar 16, 2020
@chingor13 chingor13 merged commit bcd75a0 into googleapis:master Mar 16, 2020
@chingor13 chingor13 deleted the java-sample-pom branch March 16, 2020 22:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Merge the pull request once unit tests and other checks pass. cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants