-
Notifications
You must be signed in to change notification settings - Fork 55
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.
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
controller test broken: eval error: undefined method `keys' for nil:NilClass #199
Comments
I was able to reproduce with a very simple app and may have found the culprit: |
mcmire
pushed a commit
that referenced
this issue
Nov 13, 2023
I believe this fixes #199 and, I suspect, fixes #163. See #199 (comment) for more background. You can reproduce errors like those raised in the aforementioned issues by checking out 26e1f6c and running `bundle exec rspec spec/unit/active_support/object_inspection_spec.rb`. The solution I opted for here was to create a hash-like inspector for `ActiveSupport::OrderedOptions` that takes precedence over the default `CustomObject` inspector, which in turn [takes precedence over](https://github.com/mcmire/super_diff/blob/fb6718a2b60bc8135424295cc069a4b984983f77/lib/super_diff/object_inspection/inspection_tree_builders/defaults.rb#L5-L7) the `Hash` inspector. The `OrderedOptions` inspection tree is basically a copy-paste of the one for `HashWithIndifferentAccess` – let me know if I should try to DRY those up.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
We have a pretty straight forward controller test that breaks execution
The exception bubble up from
ruby/gems/3.2.0/gems/super_diff-0.10.0/lib/super_diff/object_inspection/inspection_tree.rb:72
in `insert_hash_inspection_of'.The text was updated successfully, but these errors were encountered: