[fix] File path shouldn't be allowed to be empty #166

Fixes #166

Author: codesankalp

netjsonconfig/schema.py

@@ -939,6 +939,7 @@
                         "type": "string",
                         "description": "filesystem path",
                         "propertyOrder": 1,
+                        "minLength": 2,
                     },
                     "mode": {
                         "type": "string",

tests/openvpn/test_parser.py

@@ -3,7 +3,7 @@
 from copy import deepcopy
 
 from netjsonconfig import OpenVpn
-from netjsonconfig.exceptions import ParseError
+from netjsonconfig.exceptions import ParseError, ValidationError
 
 
 class TestParser(unittest.TestCase):
@@ -159,3 +159,10 @@ def test_parse_tar_file(self):
         OpenVpn(native=open('/tmp/test.tar.gz'))
         os.remove('/tmp/test.tar.gz')
         self.assertDictEqual(o.config, self._multiple_vpn)
+
+    def test_file_path_min_length(self):
+        conf = deepcopy(self._multiple_vpn)
+        conf.update({"files": [{"path": ".", "mode": "0644", "contents": "testing!"}]})
+        with self.assertRaises(ValidationError) as err:
+            OpenVpn(conf).generate()
+        self.assertEqual("'.' is too short", err.exception.message)