Skip to content

Conversation

ysiraichi
Copy link
Collaborator

In summary, this PR:

  • Moves tests that checked error message of PyTorch operations into test_ops_error_message.py
  • Introduces expecttest as a dependency in requirements.in file
  • Introduces expecttest to those tests, so as to avoid copy-and-pasting error messages

The introduction of expecttest Python package was, in fact, implicit to our tests because of the following PyTorch testing library import:

from torch.testing._internal.common_device_type import dtypes

@ysiraichi ysiraichi force-pushed the ysiraichi/move-error-message-tests branch from 5922160 to ee84cdd Compare September 5, 2025 21:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant