Skip to content

fix: add require 'ostruct' for Ruby 3.2+ compatibility#296

Merged
damacus merged 1 commit into
mainfrom
fix/ark-rspec-ostruct
Mar 16, 2026
Merged

fix: add require 'ostruct' for Ruby 3.2+ compatibility#296
damacus merged 1 commit into
mainfrom
fix/ark-rspec-ostruct

Conversation

@damacus
Copy link
Copy Markdown
Member

@damacus damacus commented Mar 16, 2026

Ruby 3.2+ no longer auto-loads OpenStruct. The spec_helper uses OpenStruct for test doubles but was missing the explicit require, causing all 19 library specs to fail.

Ruby 3.2+ no longer auto-loads OpenStruct. The spec_helper uses

OpenStruct for test doubles but was missing the explicit require.
@damacus damacus requested a review from a team as a code owner March 16, 2026 09:14
@github-actions
Copy link
Copy Markdown

Slowest examples

Top 10 slowest examples (0.58 seconds, 46.6% of total time)
Example Description Time in seconds
spec/recipes/default_spec.rb:10 ark::default does not install the gcc-c++ package 0.09213
spec/recipes/default_spec.rb:6 ark::default installs core packages 0.064
spec/recipes/default_spec.rb:14 ark::default is expected not to install seven_zip_tool "ark" 0.06397
spec/recipes/default_spec.rb:23 ark::default sets default attributes prefix root 0.06021
spec/recipes/default_spec.rb:19 ark::default sets default attributes apache mirror 0.05971
spec/recipes/default_spec.rb:27 ark::default sets default attributes prefix bin 0.05764
spec/recipes/default_spec.rb:31 ark::default sets default attributes prefix home 0.05703
spec/recipes/centos/default_spec.rb:8 ark::default installs core packages 0.04627
spec/resources/default_spec.rb:190 ark dump dumps 0.0453
spec/resources/default_spec.rb:316 ark install_with_make installs with make 0.03874

@damacus damacus merged commit 65d27bf into main Mar 16, 2026
13 of 39 checks passed
@damacus damacus deleted the fix/ark-rspec-ostruct branch March 16, 2026 10:33
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.

1 participant