remove nltk references from tests

Author: AlejandroEsquivel

tests/unit_tests/classes/test_rc.py

@@ -4,11 +4,6 @@
 
 class TestRC:
     def test_load(self, mocker):
-        # TODO: Re-enable this once we move nltk.download calls to individual validator repos.  # noqa
-        # Right now, it fires during our import chain, causing this to blow up
-        mocker.patch("nltk.data.find")
-        mocker.patch("nltk.download")
-
         expanduser_mock = mocker.patch("guardrails.classes.rc.expanduser")
         expanduser_mock.return_value = "/Home"
 

tests/unit_tests/cli/test_configure.py

@@ -61,11 +61,6 @@ def test_configure(mocker, runner, expected_token, enable_metrics, clear_token):
 
 
 def test_save_configuration_file(mocker):
-    # TODO: Re-enable this once we move nltk.download calls to individual validator repos.  # noqa
-    # Right now, it fires during our import chain, causing this to blow up
-    mocker.patch("nltk.data.find")
-    mocker.patch("nltk.download")
-
     expanduser_mock = mocker.patch("guardrails.cli.configure.expanduser")
     expanduser_mock.return_value = "/Home"
 

tests/unit_tests/cli/test_validate.py

@@ -2,9 +2,6 @@
 
 
 def test_validate(mocker):
-    mocker.patch("nltk.data.find")
-    mocker.patch("nltk.download")
-
     mock_validate_llm_output = mocker.patch(
         "guardrails.cli.validate.validate_llm_output"
     )