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

Fix for Bug 92089 #47

Closed
wants to merge 1 commit into from
Closed

Fix for Bug 92089 #47

wants to merge 1 commit into from

Conversation

aarondcohen
Copy link
Contributor

  • fixed issue where a trait generated on the fly could not be resolved
  • revert _apply_all_roles to use _load_user_class
  • revert unit tests to not check for files
  • added test for bug regression

Bur ticket can be found at: https://rt.cpan.org/Ticket/Display.html?id=92089

- fixed issue where a trait generated on the fly could not be resolved
- revert _apply_all_roles to use _load_user_class
- revert unit tests to not check for files
- added test for bug regression
@karenetheridge
Copy link
Member

This is (mostly -- see #moose discussion) great, except it uses Test::Exception, which is not in our requirements list. Would you mind changing lives_ok ... to is(exception { ... }, undef,...) (using Test::Fatal)?

@karenetheridge
Copy link
Member

This was merged as #135 and will be in 2.1806. thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants