-
Notifications
You must be signed in to change notification settings - Fork 13.3k
Add test for issue #16994. #41632
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
Add test for issue #16994. #41632
Conversation
@bors r+ rollup |
📌 Commit efe8872 has been approved by |
⌛ Testing commit efe8872 with merge 7a0814c... |
💔 Test failed - status-travis |
efe8872
to
9bc53c8
Compare
r? @arielb1, I forgot the |
@bors r+ rollup |
📌 Commit 9bc53c8 has been approved by |
⌛ Testing commit 9bc53c8 with merge 510d89c... |
💔 Test failed - status-travis |
9bc53c8
to
3b00323
Compare
I should really stop trying to guess at the correct fix. This time tests passed locally. I'd wait for Travis though. |
@bors r+ rollup |
📌 Commit 3b00323 has been approved by |
☀️ Test successful - status-appveyor, status-travis |
Fixes #16994.
Please check that this is the correct way to write this test.
r? @arielb1 (author of test case)