Skip to content

Commit 36433fa

Browse files
perryqhclaude
andauthored
Fix CD smoke test by exercising Rust extension instead of for_file (#166)
The x86_64-linux smoke test in CD called CodeOwnership.for_file, which requires config/code_ownership.yml in the CWD. This repo intentionally does not ship that config (removed in e9f29ac), so the step failed with "Can't open config file" on the first real dispatch after #157. Replace the call with CodeOwnership.version, which exercises the cross-compiled Rust native extension (loads the .so, calls into Rust) without depending on a project config — the right check for a platform-gem smoke test. Prior smoke test (pre-#157) called a non-existent method with `|| true`, so it silently no-op'd. #157 switched to a real method and dropped the suppression, exposing the latent missing-config problem once CD next ran on workflow_dispatch. Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 40e252a commit 36433fa

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

.github/workflows/cd.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -72,8 +72,8 @@
7272
# Test that it works
7373
ruby -e "require 'code_ownership'; puts 'Version: ' + CodeOwnership::VERSION"
7474
75-
# Run a simple functionality test
76-
ruby -e "require 'code_ownership'; puts CodeOwnership.for_file('lib/code_ownership.rb').inspect"
75+
# Run a simple functionality test that exercises the Rust native extension
76+
ruby -e "require 'code_ownership'; puts CodeOwnership.version"
7777
7878
echo "✅ Successfully tested ${{ matrix.ruby-platform }} gem"
7979

0 commit comments

Comments
 (0)