Updates for tests

Makefile

@@ -94,11 +94,11 @@ prepare-tests-windows: prepare-wget-windows prepare-tests-files
 	choco install graphviz
 
 test: clean
-	# OMP_NUM_THREADS can improve overral performance using one thread by process (on tensorflow), avoiding overload
+	# OMP_NUM_THREADS can improve overrall performance using one thread by process (on tensorflow), avoiding overload
 	OMP_NUM_THREADS=1 poetry run pytest tests -n $(JOBS) --cov rasa
 
 test-windows: clean
-	# OMP_NUM_THREADS can improve overral performance using one thread by process (on tensorflow), avoiding overload
+	# OMP_NUM_THREADS can improve overrall performance using one thread by process (on tensorflow), avoiding overload
 	OMP_NUM_THREADS=1 poetry run pytest tests -n $(JOBS) -m "not unix" --cov rasa
 
 doctest: clean

changelog/5784.misc.rst

@@ -1 +1 @@
-Windows testing support.
+Added testing of Rasa Open Source installations on Windows. 

tests/core/test_tracker_stores.py

@@ -44,7 +44,7 @@
 from rasa.core.trackers import DialogueStateTracker
 from rasa.utils.endpoints import EndpointConfig, read_endpoint_config
 from tests.core.conftest import DEFAULT_ENDPOINTS_FILE, MockedMongoTrackerStore
-from tests.utilities import write_file_config
+import tests.utilities
 
 domain = Domain.load("data/test_domains/default.yml")
 
@@ -357,7 +357,7 @@ def test_db_url_with_query_from_endpoint_config():
         another: query
     """
 
-    filename = write_file_config(endpoint_config)
+    filename = tests.utilities.write_file_config(endpoint_config)
     store_config = read_endpoint_config(filename, "tracker_store")
     url = SQLTrackerStore.get_db_url(**store_config.kwargs)
 

tests/nlu/conftest.py

@@ -2,11 +2,11 @@
 import ruamel.yaml as yaml
 
 import pytest
+import tests.utilities
 
 from rasa.nlu.config import RasaNLUModelConfig
 from rasa.nlu.components import ComponentBuilder
 from rasa.utils.tensorflow.constants import EPOCHS, RANDOM_SEED
-from tests.utilities import write_file_config
 
 DEFAULT_DATA_PATH = "data/examples/rasa/demo-rasa.json"
 
@@ -41,7 +41,7 @@ def blank_config() -> RasaNLUModelConfig:
 
 @pytest.fixture(scope="session")
 def config_path() -> Text:
-    return write_file_config(
+    return tests.utilities.write_file_config(
         yaml.safe_dump(
             {
                 "language": "en",

tests/nlu/test_config.py

@@ -4,6 +4,7 @@
 from typing import Text, List
 
 import pytest
+import tests.utilities
 
 import rasa.utils.io as io_utils
 from rasa.nlu.config import RasaNLUModelConfig
@@ -13,12 +14,11 @@
 from rasa.nlu.constants import TRAINABLE_EXTRACTORS
 from rasa.nlu.registry import registered_pipeline_templates
 from rasa.nlu.model import Trainer
-from tests.utilities import write_file_config
 
 
 def test_blank_config(blank_config):
     file_config = {}
-    filename = write_file_config(yaml.safe_dump(file_config))
+    filename = tests.utilities.write_file_config(yaml.safe_dump(file_config))
     final_config = config.load(filename)
 
     assert final_config.as_dict() == blank_config.as_dict()
@@ -27,15 +27,15 @@ def test_blank_config(blank_config):
 def test_invalid_config_json():
     file_config = """pipeline: [pretrained_embeddings_spacy"""  # invalid yaml
 
-    filename = write_file_config(file_config)
+    filename = tests.utilities.write_file_config(file_config)
 
     with pytest.raises(config.InvalidConfigError):
         config.load(filename)
 
 
 def test_invalid_pipeline_template():
     args = {"pipeline": "my_made_up_name"}
-    filename = write_file_config(yaml.safe_dump(args))
+    filename = tests.utilities.write_file_config(yaml.safe_dump(args))
 
     with pytest.raises(config.InvalidConfigError) as execinfo:
         config.load(filename)
@@ -97,7 +97,7 @@ def test_missing_property(pipeline_config):
 @pytest.mark.unix
 def test_pipeline_registry_lookup(pipeline_template: Text):
     args = {"pipeline": pipeline_template}
-    filename = write_file_config(yaml.safe_dump(args))
+    filename = tests.utilities.write_file_config(yaml.safe_dump(args))
 
     final_config = config.load(filename)
     components = [c for c in final_config.pipeline]

tests/utilities.py

@@ -19,7 +19,7 @@ def write_temp_file(contents, suffix, mode="w+") -> Text:
     return filename
 
 
-def write_file_config(file_config) -> Text:
+def write_file_config(file_config: Text) -> Text:
     return write_temp_file(file_config, "_tmp_config_file.yml")