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

Fixes #22065 - Fix missing permissions table in tests #5146

Merged
merged 1 commit into from
Jan 3, 2018

Conversation

xprazak2
Copy link
Contributor

@xprazak2 xprazak2 commented Jan 3, 2018

When loading plugins in test for the first time, the permissions table does not yet exist, which was causing failures. We do not use permissions registered from plugins in tests, but we create those we need with fixtures or factories.

@theforeman-bot
Copy link
Member

Issues: #22065

Copy link
Member

@iNecas iNecas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested and works well

Copy link
Member

@tbrisker tbrisker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @xprazak2 !

@tbrisker tbrisker merged commit 1d92402 into theforeman:develop Jan 3, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants