Fixes issue #200 - Githubissues

excid3 / madmin

A robust Admin Interface for Ruby on Rails apps

https://github.com/excid3/madmin

MIT License

603 stars 66 forks source link

Fixes issue #200 #201

Closed clayton closed 7 months ago

clayton commented 7 months ago

Fixes #200 by explicitly scoping some references to ActiveRecord

Notes:

The enum test is just additive, but wasn't part of the actual problem.
I installed the activerecord-session_store gem and the migration to replicate the problem
New test is fairly basic and just tries to exercise the functionality of infer_type

I'm a bit concerned with some of the changes in the schema (e.g. removal of certain precision values) and not having any explicit tests for the actual problem.

excid3 commented 7 months ago

We already have an enum example, so I'm going to close this in favor of a6a3acfb03fefba46dab699f1809c1e32646f472 instead.