Merge pull request #27 from seddonym/add-failing-functional-tests

Add failing functional tests

tests/assets/testpackage/.brokencontract.ini

@@ -0,0 +1,11 @@
+[importlinter]
+root_package_name = testpackage
+
+
+[importlinter:contract:one]
+name=Expected broken contract
+; This should fail, as high.blue.one -> utils -> high.green.
+type=independence
+modules=
+    testpackage.high.blue
+    testpackage.high.green

tests/assets/testpackage/.malformedcontract.ini

@@ -0,0 +1,9 @@
+[importlinter]
+root_package_name = testpackage
+
+
+[importlinter:contract:one]
+name=Expected malformed contract
+; This should fail validation, as modules expects multiple arguments.
+type=independence
+modules=testpackage.high.blue

tests/functional/test_lint_imports.py

@@ -1,5 +1,7 @@
 import os
 
+import pytest
+
 from importlinter import cli
 
 
@@ -11,9 +13,21 @@
 )
 
 
-def test_lint_imports():
+@pytest.mark.parametrize(
+    'config_filename, expected_result',
+    (
+        (None, cli.EXIT_STATUS_SUCCESS),
+        ('.brokencontract.ini', cli.EXIT_STATUS_ERROR),
+        ('.malformedcontract.ini', cli.EXIT_STATUS_ERROR),
+    )
+)
+def test_lint_imports(config_filename, expected_result):
 
     os.chdir(testpackage_directory)
-    result = cli.lint_imports()
 
-    assert 0 == result
+    if config_filename:
+        result = cli.lint_imports(config_filename=config_filename)
+    else:
+        result = cli.lint_imports()
+
+    assert expected_result == result