Skip to content

Synchronize auth permutation tests with Go SDK#165

Merged
pietern merged 2 commits intomainfrom
auth-permutation-test-sync
Jun 13, 2023
Merged

Synchronize auth permutation tests with Go SDK#165
pietern merged 2 commits intomainfrom
auth-permutation-test-sync

Conversation

@pietern
Copy link
Copy Markdown
Contributor

@pietern pietern commented Jun 13, 2023

Changes

Incorporate auth permutations tests from databricks/databricks-sdk-go#363 and databricks/databricks-sdk-go#496.

Tests

  • make test run locally
  • make fmt applied
  • relevant integration tests applied

@pietern pietern requested a review from nfx June 13, 2023 16:17
@pietern
Copy link
Copy Markdown
Contributor Author

pietern commented Jun 13, 2023

Note: I had to manually change some assertions from https://x to x after regenerating to make it work.

@pietern pietern merged commit ca18ffe into main Jun 13, 2023
@pietern pietern deleted the auth-permutation-test-sync branch June 13, 2023 16:32
nfx added a commit that referenced this pull request Jun 15, 2023
* Regenerate from OpenAPI spec ([#176](#176)).
* Added improved notebook-native authentication ([#152](#152)).
* Added methods to provide extra user agent and upstream user agent to SDK config ([#163](#163)).
* Added more missing `Optional` type hints ([#171](#171)).
* Add more missing optional fields ([#177](#177)).
* Correctly serialize external entities ([#178](#178)).
* Correctly serialize external enum values in paths ([#179](#179)).
* Mark non-required fields as `Optional` ([#170](#170)).
* Synchronize auth permutation tests with Go SDK ([#165](#165)).
@nfx nfx mentioned this pull request Jun 15, 2023
1 task
nfx added a commit that referenced this pull request Jun 15, 2023
* Regenerate from OpenAPI spec
([#176](#176)).
* Added improved notebook-native authentication
([#152](#152)).
* Added methods to provide extra user agent and upstream user agent to
SDK config
([#163](#163)).
* Added more missing `Optional` type hints
([#171](#171)).
* Added more missing optional fields
([#177](#177)).
* Correctly serialize external entities
([#178](#178)).
* Correctly serialize external enum values in paths
([#179](#179)).
* Mark non-required fields as `Optional`
([#170](#170)).
* Synchronize auth permutation tests with Go SDK
([#165](#165)).

## Tests

- [x] relevant integration tests applied

---------

Signed-off-by: Serge Smertin <[email protected]>
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.

2 participants