Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix OpenStruct test failures #2548

Closed
kaylareopelle opened this issue Apr 11, 2024 · 1 comment · Fixed by #2547
Closed

Fix OpenStruct test failures #2548

kaylareopelle opened this issue Apr 11, 2024 · 1 comment · Fixed by #2547
Assignees
@kaylareopelle
Labels
0 Story Point Estimate technical debt

Comments

@kaylareopelle
Copy link
Contributor

Rake 13.2.0 stopped requiring OpenStruct: ruby/rake#545

In some of our tests, we were not explicitly requiring OpenStruct. Since our tests were run with rake, the need for a require was concealed.

Add require statements for OpenStruct in files that use it without requiring it.
@kaylareopelle kaylareopelle self-assigned this Apr 11, 2024
@workato-integration
Copy link

https://new-relic.atlassian.net/browse/NR-257838

@kaylareopelle kaylareopelle added 0 Story Point Estimate 1 Story Point Estimate technical debt and removed 1 Story Point Estimate labels Apr 11, 2024
@kaylareopelle kaylareopelle moved this from Triage to In Sprint in Ruby Engineering Board Apr 11, 2024
@kaylareopelle kaylareopelle mentioned this issue Apr 11, 2024
Merged
@github-project-automation github-project-automation bot moved this from In Sprint to Code Complete/Done in Ruby Engineering Board Apr 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kaylareopelle kaylareopelle

Labels
0 Story Point Estimate technical debt
Projects
Ruby Engineering Board
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Require OpenStruct where missing in tests newrelic/newrelic-ruby-agent
1 participant
@kaylareopelle