test: fix tests

Makefile

@@ -13,7 +13,7 @@ test-lint: ## Run code linting tests
 	pylint --errors-only --enable=unused-import,unused-argument --ignore=templates --ignore=docs/_ext ${SRC_DIRS}
 
 test-types: ## Run type checks.
-	mypy --exclude=templates --ignore-missing-imports --implicit-reexport ${SRC_DIRS}
+	mypy --exclude=templates --ignore-missing-imports --implicit-reexport --strict ${SRC_DIRS}
 
 format: ## Format code automatically
 	black $(BLACK_OPTS)

tutorpicasso/commands/enable_themes.py

@@ -3,6 +3,7 @@
 
 import click
 from tutor import config as tutor_config
+from tutor.types import Config
 
 
 @click.command(name="enable-themes", help="Enable picasso themes")
@@ -18,7 +19,7 @@ def enable_themes() -> None:
         .decode("utf-8")
         .strip()
     )
-    config = tutor_config.load(tutor_root)
+    config: Config = tutor_config.load(tutor_root)
 
     if config.get("PICASSO_THEMES"):
         for theme in config["PICASSO_THEMES"]:

tutorpicasso/commands/run_extra_commands.py

@@ -2,10 +2,11 @@
 import subprocess
 
 # Was necessary to use this for compatibility with Python 3.8
-from typing import List
+from typing import Any, List
 
 import click
 from tutor import config as tutor_config
+from tutor.types import Config, ConfigValue
 
 COMMAND_CHAINING_OPERATORS = ["&&", "&", "||", "|", ";"]
 
@@ -20,15 +21,15 @@ def run_extra_commands() -> None:
         .decode("utf-8")
         .strip()
     )
-    config = tutor_config.load(tutor_root)
-    picasso_extra_commands = config.get("PICASSO_EXTRA_COMMANDS", None)
+    config: Config = tutor_config.load(tutor_root)
+    picasso_extra_commands: ConfigValue = config.get("PICASSO_EXTRA_COMMANDS", None)
     if picasso_extra_commands is not None:
         validate_commands(picasso_extra_commands)
         for command in picasso_extra_commands:
             run_command(command)
 
 
-def validate_commands(commands: List[str]) -> None:
+def validate_commands(commands: Any) -> None:
     """
     Takes all the extra commands sent through config.yml and verifies that
     all the commands are correct before executing them