change test flags names

Author: Aethor

README.md

@@ -68,7 +68,7 @@ Alternatively, the project Makefile has a test target:
 
 > make test
 
-Expensive tests are disabled by default. These can be run by setting the environment variable `RENARD_TEST_ALL` to `1`.
+Expensive tests are disabled by default. These can be run by setting the environment variable `RENARD_TEST_SLOW` to `1`.
 
 
 

docs/contributing.rst

@@ -7,7 +7,7 @@ issue if you encounter a problem or want to discuss a specific
 feature. If you want to contribute a patch:
 
 1. Check that your code matches our code quality guidelines and that
-   all existing tests are passing with ``RENARD_TEST_ALL=1``.
+   all existing tests are passing with ``RENARD_TEST_SLOW=1``.
 2. Create a Github pull request with your patch, explaining the
    rationale behind it and giving a high level overview of your
    code. Mention the relevant issue if applicable.
@@ -36,9 +36,7 @@ the ``tests`` directory. We use ``pytest`` to test code, and also use
 ``hypothesis`` when applicable. If you open a patch, make sure that
 all tests are passing. In particular, do not rely on the CI, as it
 does not run time costly tests! Check for yourself locally, using
-``RENARD_TEST_ALL=1 python -m pytest tests``. Note that there are
+``RENARD_TEST_SLOW=1 python -m pytest tests``. Note that there are
 specific tests and environment variable for optional dependencies such
-as *stanza* (``RENARD_TEST_STANZA_OPTDEP``). These must be explicitely
-set to ``1`` if you want to test optional dependencies, as
-``RENARD_TEST_ALL=1`` does not enable test on these optional
-dependencies.
+as *stanza* (``RENARD_TEST_OPTDEP_STANZA``). These must be explicitely
+set to ``1`` if you want to test optional dependencies.

docs/pipeline.rst

@@ -200,7 +200,7 @@ to a unique character.
 Relation Extraction
 -------------------
 
-- :class:`.T5RelationExtractor` extracts relations between characters
+- :class:`.GenerativeRelationExtractor` extracts relations between characters
   using a finetuned T5 model.
 
 

tests/test_corefs.py

@@ -4,7 +4,7 @@
 from renard.pipeline.corefs import BertCoreferenceResolver
 
 
-@pytest.mark.skipif(os.getenv("RENARD_TEST_ALL") != "1", reason="performance")
+@pytest.mark.skipif(os.getenv("RENARD_TEST_SLOW") != "1", reason="performance")
 def test_bert_coreference_resolver_runs():
     pipeline = Pipeline([BertCoreferenceResolver()], progress_report=None)
     tokens = "Princess Liana felt sad , because Zarth Arn was gone . The princess went to sleep .".split(

tests/test_ner.py

@@ -18,7 +18,7 @@
 
 
 @pytest.mark.skipif(
-    os.getenv("RENARD_TEST_SEQEVAL_OPTDEP") != "1",
+    os.getenv("RENARD_TEST_OPTDEP_SEQEVAL") != "1",
     reason="not testing seqeval based functions",
 )
 @given(lists(sampled_from(("B-PER", "I-PER", "O")), min_size=1))
@@ -34,7 +34,7 @@ def test_has_correct_number_of_entities(tokens: List[str]):
     assert len(entities) == len(tokens)
 
 
-@pytest.mark.skipif(os.getenv("RENARD_TEST_ALL") != "1", reason="performance")
+@pytest.mark.skipif(os.getenv("RENARD_TEST_SLOW") != "1", reason="performance")
 @pytest.mark.parametrize(
     "retriever_class", [NERSamenounContextRetriever, NERBM25ContextRetriever]
 )
@@ -53,7 +53,7 @@ def test_retrieves_context(retriever_class: Type[NERContextRetriever]):
     assert len(ctx_dataset._context_mask) == len(sentences)
 
 
-@pytest.mark.skipif(os.getenv("RENARD_TEST_ALL") != "1", reason="performance")
+@pytest.mark.skipif(os.getenv("RENARD_TEST_SLOW") != "1", reason="performance")
 def test_neural_retrieves_context():
     context_retriever = NERNeuralContextRetriever(
         NEREnsembleContextRetriever(

tests/test_pipeline.py

@@ -53,14 +53,14 @@ def production(self) -> Set[str]:
     assert not pipeline.check_valid()[0]
 
 
-@pytest.mark.skipif(os.getenv("RENARD_TEST_ALL") != "1", reason="performance")
+@pytest.mark.skipif(os.getenv("RENARD_TEST_SLOW") != "1", reason="performance")
 def test_co_occurrence_pipeline_runs():
     text = load_novel_chapters("pride_and_prejudice")[0]
     pipeline = co_occurrence_pipeline(warn=False, progress_report=None)
     pipeline(text)
 
 
-@pytest.mark.skipif(os.getenv("RENARD_TEST_ALL") != "1", reason="performance")
+@pytest.mark.skipif(os.getenv("RENARD_TEST_SLOW") != "1", reason="performance")
 def test_conversational_pipeline_runs():
     text = load_novel_chapters("pride_and_prejudice")[0]
     # # if the text is too long, speaker attribution takes too much time
@@ -70,7 +70,7 @@ def test_conversational_pipeline_runs():
     pipeline(text)
 
 
-@pytest.mark.skipif(os.getenv("RENARD_TEST_ALL") != "1", reason="performance")
+@pytest.mark.skipif(os.getenv("RENARD_TEST_SLOW") != "1", reason="performance")
 def test_relational_pipeline_runs():
     text = load_novel_chapters("pride_and_prejudice")[0]
     pipeline = relational_pipeline(warn=False, progress_report=None)

tests/test_stanza.py

@@ -7,7 +7,7 @@
 
 
 @pytest.mark.skipif(
-    os.getenv("RENARD_TEST_STANZA_OPTDEP") != "1",
+    os.getenv("RENARD_TEST_OPTDEP_STANZA") != "1",
     reason="not testing stanza based modules",
 )
 def test_stanza_pipeline_runs():