Skip to content

Conversation

@jonhopper-dataengineers

What changes are proposed in this pull request?

Modified the UserAgent.env() so that it doesn't throw an exception if the PATH variable isn't available.
relates to issue #550

env is not always available environments which implement sandboxes such as snowflake

How is this tested?

Added additional unit test to validate new behaviour

Describe any tests you have done; especially if test tests are not part of
the unit tests (e.g. local tests).

ALWAYS ANSWER THIS QUESTION: Answer with "N/A" if tests are not applicable
to your PR (e.g. if the PR only modifies comments). Do not be afraid of
answering "Not tested" if the PR has not been tested. Being clear about what
has been done and not done provides important context to the reviewers.

@github-actions
Copy link

If integration tests don't run automatically, an authorized user can run them manually by following the instructions below:

Trigger:
go/deco-tests-run/sdk-java

Inputs:

  • PR number: 633
  • Commit SHA: 45b9f783eb5773427546206ac659794aab90cfeb

Checks will be approved automatically on success.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant