340: Changed default for Param.required to True

ipalib/frontend.py

@@ -118,10 +118,20 @@ def parse_param_spec(spec):
 
 class Param(plugable.ReadOnly):
     __nones = (None, '', tuple(), [])
+    __default = dict(
+        type=ipa_types.Unicode(),
+        doc='',
+        required=True,
+        multivalue=False,
+        default=None,
+        default_from=None,
+        rules=tuple(),
+        normalize=None
+    )
 
     def __init__(self, name, type_,
             doc='',
-            required=False,
+            required=True,
             multivalue=False,
             default=None,
             default_from=None,

ipalib/tests/test_frontend.py

@@ -143,7 +143,7 @@ class test_Param(ClassChecker):
         assert read_only(o, 'name') is name
         assert read_only(o, 'type') is type_
         assert read_only(o, 'doc') == ''
-        assert read_only(o, 'required') is False
+        assert read_only(o, 'required') is True
         assert read_only(o, 'multivalue') is False
         assert read_only(o, 'default') is None
         assert read_only(o, 'default_from') is None