skipped test_all

[ev-br]

5473d84 skipped a test, tests/test_all.py, which started failing after adding an unrelated tests/test_torch.py.
The test itself seems to probe the contents of internal implementation modules, and fails essientially on this:

In [16]: from array_api_compat.common import _aliases

In [17]: all_names = set(n for n in _aliases.__all__ if not n.startswith('_'))

In [18]: dir_names = set(n for n in dir(_aliases) if not n.startswith('_'))

In [19]: dir_names - all_names      # not empty, failure
Out[19]: 
{'NamedTuple',
 'TYPE_CHECKING',
 'annotations',
 'array_namespace',
 'device',
 'inspect',
 'is_cupy_namespace'}

I've to admit I'm not sure what the intention is, this seems to be a Chesterton fence of sorts. Do you happen to remember the context @asmeurer ?

[asmeurer]

Yes, that test ensure that the __all__ variable is defined correctly. We are defining it in a way that makes it difficult to manage in the ways you usually would, since we re-export names from existing packages, so we can't just manually list out every name that should be included. I would suggest keeping this test working. There's more context here #95.

Note that if a name is in a module that shouldn't be exported you can add it to _all_ignore.

[ev-br]

Thanks @asmeurer !

So common._aliases.__all__ was "wrong" for a while, but testing did not pick it up until an unrelated test actually imported the module. gh-286 adds a fix to __all__, plus a couple of small tweaks to make it less likely to miss a problem going forward. Would appreciate if you could take a look-through @asmeurer