-
Notifications
You must be signed in to change notification settings - Fork 306
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
Update MRProgressOverlayView.m #89
base: master
Are you sure you want to change the base?
Conversation
Was trying to run KiF UI automation tests on a project with MRProgress installed as a CocoaPod and was running into a strange exception. This seems to resolve the issue.
Interesting! If this would be a public API, I would say, that has to be done, clearly, as it would make the interface more solid, but as this method is private, I have doubts why it is needed at all. This change seems to be appropriate anyway. Can you still post the stacktrace? I wouldn't mind if you redact entries from your namespace, if necessary. |
Sure. I think what happened is multiple modeView are being created and added to that dialog view. I'm not quite sure why it works when running the app normally but not when running through the test bundler.
|
Was trying to run KiF UI automation tests on a project with MRProgress installed as a CocoaPod and was running into a strange exception. This seems to resolve the issue.