Add language_version to the manifest loader.

asottile · asottile · commit 4f2f1731ea6a · 2014-06-02T06:50:57.000-07:00
diff --git a/pre_commit/clientlib/validate_manifest.py b/pre_commit/clientlib/validate_manifest.py
@@ -20,6 +20,7 @@ class InvalidManifestError(ValueError):
             'description': {'type': 'string', 'default': ''},
             'entry': {'type': 'string'},
             'language': {'type': 'string'},
+            'language_version': {'type': 'string', 'default': 'default'},
             'expected_return_value': {'type': 'number', 'default': 0},
         },
         'required': ['id', 'name', 'entry', 'language'],
diff --git a/tests/languages/ruby_test.py b/tests/languages/ruby_test.py
@@ -1,8 +1,10 @@
 import os.path
 
 from pre_commit.languages.ruby import _install_rbenv
+from testing.util import skipif_slowtests_false
 
 
+@skipif_slowtests_false
 def test_install_rbenv(cmd_runner):
     _install_rbenv(cmd_runner)
     # Should have created rbenv directory
diff --git a/tests/manifest_test.py b/tests/manifest_test.py
@@ -19,6 +19,7 @@ def test_manifest_contents(manifest):
         'expected_return_value': 0,
         'id': 'bash_hook',
         'language': 'script',
+        'language_version': 'default',
         'name': 'Bash hook',
     }]
 
@@ -30,5 +31,6 @@ def test_hooks(manifest):
         'expected_return_value': 0,
         'id': 'bash_hook',
         'language': 'script',
+        'language_version': 'default',
         'name': 'Bash hook',
     }
