Commit fed6158
fix(test): allow write access to config file in Docker tests
Connectors with config migration (e.g., OAuth token refresh) need write
access to the config file during tests. This change:
1. Updates temp config file permissions from 0o444 (read-only) to 0o666
(read-write) in scenario.py
2. Adds explicit :rw flag to Docker volume mounts in docker_base.py
This fixes PermissionError when testing connectors that use
_migrate_and_transform_config to persist config changes.
Co-Authored-By: aldo.gonzalez@airbyte.io <aldo.gonzalez@airbyte.io>1 parent d85cb1c commit fed6158
2 files changed
Lines changed: 6 additions & 5 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
143 | 143 | | |
144 | 144 | | |
145 | 145 | | |
146 | | - | |
| 146 | + | |
| 147 | + | |
147 | 148 | | |
148 | | - | |
| 149 | + | |
149 | 150 | | |
150 | 151 | | |
151 | 152 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
243 | 243 | | |
244 | 244 | | |
245 | 245 | | |
246 | | - | |
| 246 | + | |
247 | 247 | | |
248 | 248 | | |
249 | 249 | | |
| |||
336 | 336 | | |
337 | 337 | | |
338 | 338 | | |
339 | | - | |
| 339 | + | |
340 | 340 | | |
341 | 341 | | |
342 | 342 | | |
| |||
393 | 393 | | |
394 | 394 | | |
395 | 395 | | |
396 | | - | |
| 396 | + | |
397 | 397 | | |
398 | 398 | | |
399 | 399 | | |
| |||
0 commit comments