Commit 229eb63
committed
Fix tests broken by host metadata resolution during init
The tryResolveHostMetadata() call during config.resolve() (added in PR 4)
makes an HTTP call to /.well-known/databricks-config. This broke tests
that use FixtureServer (consumed fixtures out of order) or mock HTTP
clients (unexpected call count). Fixes:
- Catch Throwable (not just Exception) for mock assertion errors
- Add well-known fixture to ExternalBrowser tests using FixtureServer
- Filter metadata calls from captured requests in idempotency test
Co-authored-by: Isaac1 parent e5f0a76 commit 229eb63
File tree
2 files changed
+25
-8
lines changed- databricks-sdk-java/src/test/java/com/databricks/sdk
- core/oauth
- service/gentesting/unittests
2 files changed
+25
-8
lines changedLines changed: 14 additions & 4 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
34 | 34 | | |
35 | 35 | | |
36 | 36 | | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
37 | 43 | | |
38 | | - | |
| 44 | + | |
39 | 45 | | |
40 | 46 | | |
41 | 47 | | |
| |||
81 | 87 | | |
82 | 88 | | |
83 | 89 | | |
| 90 | + | |
| 91 | + | |
| 92 | + | |
| 93 | + | |
| 94 | + | |
| 95 | + | |
84 | 96 | | |
85 | | - | |
| 97 | + | |
86 | 98 | | |
87 | 99 | | |
88 | 100 | | |
| |||
126 | 138 | | |
127 | 139 | | |
128 | 140 | | |
129 | | - | |
130 | 141 | | |
131 | | - | |
132 | 142 | | |
133 | 143 | | |
134 | 144 | | |
| |||
Lines changed: 11 additions & 4 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
0 commit comments