Force reload_dirs to be a list (encode#978)

* Add failing test * Force reload_dirs to be a list * Lint
sephioh · Oct 29, 2022 · 25a6f9a · 25a6f9a
1 parent 2eb8644
commit 25a6f9a
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/tests/test_config.py b/tests/test_config.py
@@ -64,6 +64,11 @@ def test_config_should_reload_is_set(app, expected_should_reload):
     assert config_reload.should_reload is expected_should_reload
 
 
+def test_reload_dir_is_set():
+    config = Config(app=asgi_app, reload=True, reload_dirs="reload_me")
+    assert config.reload_dirs == ["reload_me"]
+
+
 def test_wsgi_app():
     config = Config(app=wsgi_app, interface="wsgi", proxy_headers=False)
     config.load()

diff --git a/uvicorn/config.py b/uvicorn/config.py
@@ -207,7 +207,10 @@ def __init__(
         if reload_dirs is None:
             self.reload_dirs = [os.getcwd()]
         else:
-            self.reload_dirs = reload_dirs
+            if isinstance(reload_dirs, str):
+                self.reload_dirs = [reload_dirs]
+            else:
+                self.reload_dirs = reload_dirs
 
         if env_file is not None:
             from dotenv import load_dotenv