This is an alternative to #9523. Also checking that tests pass with this change.

@eric-wieser No, that is a relative path and can't be used with nose.runmodule, which does not import the tests package, hence has no idea what .. means.

You can still import and run numpy.test(...), this only affects running a test file using run_module_suite.

EDIT: It only runs tests against installed numpy, so can be useful in test development. For testing new code, python runtests.py is still the way to go.

Not sure if this fix will work with pytest without some help. Otherwise might be stuck with only using runtests.py, as scipy is at this time.

Yeah, I can make this work for pytest also, just need the --pyargs argument. For example

pytest --pyargs numpy/f2py/tests/test_kind

That should even work with relative imports. When we are all in for pytest, can maybe change it back. Note the f2py /tests was the only test module that used relative imports after being made into a package.

Hmm, I think I will undo the last commit and just use pytest --pyarg instead. That preserves run_module_suite for downstream backwards compatibility, although I think the current change is actually an improvement,

OK, should be ready to go, I undid last commit. I will add a release note in a future PR once I've settled a few more things.

FWIW when I want to run a subset of tests with pytest I usually do something like pytest numpy -k name_of_test. The downside is that it still loads all the tests and then scans them looking for ones matching that string, which can take a few seconds, but the upside is it's that the exact same test loading logic no matter what.

@njsmith The problem is that after adding the __init__ files, you cannot run pytest in the repo because it passes down through numpy. The reason the __init__ files are needed is because some test files have duplicate names.

E   ImportError: Error importing numpy: you should not try to import numpy from
E           its source directory; please exit the numpy source tree, and relaunch

Gotta happen eventually, so putting this in.