unnecessary arg check

Author: vassilyl

msrc-appconfig/msrc/appconfig/__init__.py

@@ -140,8 +140,6 @@ def gather_config(
 
     Returns the collected configuration."""
 
-    if not inspect.isclass(config_type):
-        raise TypeError("First argument must be a class.")
     if env_var_prefix is None and _main_script.name:
         env_var_prefix = _main_script.name.upper() + '_'
     try:

msrc-appconfig/tests/gather_test.py

@@ -142,6 +142,14 @@ def test_arg_overrides_env(tmp_path, mocker):
 # ==========
 
 
+def test_gather_invalid_schema():
+    "First parameter must be a class."
+    with pytest.raises(TypeError):
+        msrc.appconfig.read_all.gather(WithDefaults())  # type: ignore
+    with pytest.raises(ValueError):
+        msrc.appconfig.read_all.gather(dict)
+
+
 def test_gather_invalid_defaults():
     "Data in `override_defailts` must be parseable."
     with pytest.raises(ValueError):