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.

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

Error: primary::SchemaMigration is not a valid Class name. #186

Closed
ryanb opened this issue Apr 16, 2020 · 5 comments
Closed

Error: primary::SchemaMigration is not a valid Class name. #186

ryanb opened this issue Apr 16, 2020 · 5 comments

Comments

@ryanb
Copy link

ryanb commented Apr 16, 2020

I see this error in the log when running Wallaby 6.1.2:

WALLABY ERROR: `primary::SchemaMigration` is not a valid Class name.

Is it trying to find a model class for every table? Rails manages this table I think.

@tian-im
Copy link
Collaborator

tian-im commented Apr 16, 2020

Yeah, I saw this error as well. It should be a fake class that comes from Rails test, but I'm not sure how it gets into the environment.

The reason it shows up is that Wallaby will go through every ActiveRecord class and associate it with a mode so that Wallaby knows how to handle it when building the Wallaby::Map.

But yeah, I think it's safe to ignore it until Rails team fixes it.

@ryanb
Copy link
Author

ryanb commented Apr 16, 2020

Is there an active issue for this on the Rails repo? The closest thing I could find is this issue: rails/rails#36757

However this doesn't look like something they plan to fix. Can you add an exception in Wallaby to ignore that class name?

@tian-im
Copy link
Collaborator

tian-im commented Apr 19, 2020

Sure thing

@tian-im
Copy link
Collaborator

tian-im commented Apr 19, 2020

@ryanb could you update to wallaby 6.1.3 and see if it is ok for you?

@tian-im tian-im added the ready label Apr 19, 2020
@ryanb
Copy link
Author

ryanb commented Apr 20, 2020

Fixed, thanks!

@ryanb ryanb closed this as completed Apr 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants