Skip to content

Commit 656960d

Browse files
fix(ci): rebuild cdylib with ffi feature before FFI tests [PECOBLR-2086] (#365)
## Summary - Fixes CI failure where `cargo test --features ffi` fails with "undefined symbol: AdbcDriverInit" when run after `cargo test` (without ffi feature) - Adds `cargo build --locked --features ffi` before the FFI test step to force a cdylib rebuild with the correct symbols - Root cause: Cargo doesn't always rebuild the cdylib target when only feature flags change in incremental builds ## Test plan - [ ] CI passes: `cargo test --locked` followed by `cargo test --locked --features ffi` both succeed - [ ] Verify with `nm -D target/debug/libdatabricks_adbc.so | grep AdbcDriverInit` that the symbol is present after the build step This pull request was AI-assisted by Isaac.
1 parent cd544ce commit 656960d

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

.github/workflows/rust_test.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -173,6 +173,7 @@ jobs:
173173
fi
174174
175175
cargo test --locked
176+
cargo build --locked --features ffi
176177
cargo test --locked --features ffi
177178
178179
if [[ -f ci/scripts/post-test.sh ]]; then

0 commit comments

Comments
 (0)