Add tests for invalid TOML files

.pre-commit-config.yaml

@@ -29,6 +29,7 @@ repos:
     rev: v3.3.0
     hooks:
       - id: check-toml
+        exclude: 'tests/data/install/invalid_*'
       - id: check-yaml
       - id: end-of-file-fixer
       - id: trailing-whitespace

tests/data/install/invalid_pipfile/Pipfile

@@ -0,0 +1 @@
+completely invalid pipfile

tests/data/install/invalid_poetry_lock/pyproject.toml

@@ -0,0 +1,18 @@
+[tool.poetry]
+name = "test-micropipenv"
+version = "0.1.0"
+description = ""
+authors = ["Fridolin Pokorny <[email protected]>"]
+
+[tool.poetry.dependencies]
+python = "^3.7"
+micropipenv = {version = "0.0.1", optional = true}
+selinon = {version = "1.0.0", extras = ["s3", "postgres"]}
+daiquiri = {git = "https://github.com/jd/daiquiri", rev = "2.1.0"}
+
+[tool.poetry.dev-dependencies]
+hexsticker = "^1.2.0"
+
+[build-system]
+requires = ["poetry>=0.12"]
+build-backend = "poetry.masonry.api"

tests/data/install/invalid_pyproject_toml/pyproject.toml

@@ -0,0 +1 @@
+completely invalid pyproject.toml file

tests/test_micropipenv.py

@@ -452,6 +452,26 @@ def test_install_pipenv_iter_index(venv):
         assert str(venv.get_version("requests")) == "2.22.0"
 
 
+@pytest.mark.parametrize(
+    "directory, filename, method",
+    [
+        ["invalid_pipfile", "Pipfile", "pipenv"],
+        ["invalid_poetry_lock", "poetry.lock", "poetry"],
+        ["invalid_pyproject_toml", "pyproject.toml", "poetry"],
+    ],
+)
+def test_install_invalid_toml_file(venv, directory, filename, method):
+    """Test exception when a Pipfile is not a valid TOML."""
+    venv.install(MICROPIPENV_TEST_TOML_MODULE)
+    cmd = [os.path.join(venv.path, BIN_DIR, "python"), micropipenv.__file__, "install", "--deploy", "--method", method]
+    work_dir = os.path.join(_DATA_DIR, "install", directory)
+    with cwd(work_dir):
+        with pytest.raises(subprocess.CalledProcessError) as exception:
+            subprocess.check_output(cmd, env=get_updated_env(venv), stderr=subprocess.PIPE, universal_newlines=True)
+
+    assert f"FileReadError: Failed to parse {filename}: " in exception.value.stderr
+
+
 def test_parse_requirements2pipfile_lock():
     """Test parsing of requirements.txt into their Pipfile.lock representation."""
     work_dir = os.path.join(_DATA_DIR, "parse", "pip-tools")