-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
TST: wb test failing #8768
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
Comments
Looking at this now. |
The problem is that world bank's search results are not static. it's now returning...
...so the test fails. It's a poorly designed test. FWIW, I updated the test in my last commit, but wasn't the original creator/designer of it. A reasonable check, would be to make sure the contents of the 'name' column contain, say, at least 1 element with 'GDP' in it. I'll can do a PR momentarily. |
maybe just assert that u can at least 1 row back |
Already had this done, before I saw your comment. |
I'm crashing for the night. That's ready to be merged, and is actually an improved test - and in fewer lines! |
closed by #8769 |
xref #8551
cc @jnmclarty
The text was updated successfully, but these errors were encountered: