Fix test failures and address feedback

Signed-off-by: Ayan Sinha Mahapatra <[email protected]>

Author: AyanSinhaMahapatra

src/licensedcode/data/rules/cclrc_1.RULE

@@ -0,0 +1,10 @@
+---
+license_expression: cclrc
+is_license_notice: yes
+referenced_filenames:
+    - External_License/CCLRC_CDAT_License.txt
+---
+
+*    This software may be distributed under the terms of the
+ *    {{CCLRC Licence}} for CCLRC Software
+ *    <CDATDIR>/External_License/CCLRC_CDAT_License.txt

src/licensedcode/data/rules/cclrc_2.RULE

@@ -0,0 +1,7 @@
+---
+license_expression: cclrc
+is_license_notice: yes
+---
+
+*    This software may be distributed under the terms of the
+ *    {{CCLRC Licence}} for CCLRC Software

src/licensedcode/data/rules/mit_1182.RULE

@@ -1,6 +1,7 @@
 ---
 license_expression: mit
 is_license_reference: yes
+is_required_phrase: yes
 relevance: 100
 ---
 

src/licensedcode/data/rules/mit_334.RULE

@@ -6,5 +6,5 @@ referenced_filenames:
     - LICENSE
 ---
 
-This software is released under the MIT software license. 
+This software is released under the {{MIT software license}}. 
 This license, including disclaimer, is available in the 'LICENSE' file.

src/licensedcode/data/rules/mit_337.RULE

@@ -6,5 +6,5 @@ ignorable_urls:
     - http://www.opensource.org/licenses/mit-license.php
 ---
 
-This is the http://www.opensource.org/licenses/mit-license.php MIT Software License
+This is the http://www.opensource.org/licenses/mit-license.php {{MIT Software License}}
 which is OSI-certified, and GPL-compatible.

src/licensedcode/data/rules/mit_392.RULE

@@ -8,5 +8,5 @@ ignorable_urls:
     - http://www.opensource.org/licenses/mit-license.php
 ---
 
-"Distributed under the MIT software license, see the accompanying file COPYING or 
+"Distributed under the {{MIT software license}}, see the accompanying file COPYING or 
 http://www.opensource.org/licenses/mit-license.php.

src/licensedcode/data/rules/mit_396.RULE

@@ -6,5 +6,5 @@ referenced_filenames:
     - COPYING
 ---
 
-// Distributed under the MIT software license, see the accompanying
+// Distributed under the {{MIT software license}}, see the accompanying
 // file COPYING

src/licensedcode/data/rules/mit_397.RULE

@@ -5,8 +5,8 @@ relevance: 100
 referenced_filenames:
     - COPYING
 ignorable_urls:
-    - http://www.opensource.org/licenses/mit-license.php
+    - https://www.opensource.org/licenses/mit-license.php
 ---
 
-// Distributed under the MIT software license, see the accompanying
-// file COPYING or shttp://www.opensource.org/{{licenses/mit}}-license.php.
+// Distributed under the {{MIT software license}}, see the accompanying
+// file COPYING or https://www.opensource.org/licenses/mit-license.php.

src/licensedcode/models.py

@@ -1832,10 +1832,6 @@ def license_flags(self):
             for license_flag_name in self.license_flag_names
         }
 
-    @property
-    def license_flag_values(self):
-        return self.license_flags.values()
-
     def validate(self, licensing=None, thorough=False):
         """
         Validate this rule using the provided ``licensing`` Licensing and yield
@@ -1844,8 +1840,7 @@ def validate(self, licensing=None, thorough=False):
         is_false_positive = self.is_false_positive
 
         has_license_flags = any(self.license_flag_values)
-        has_no_license_flags = len([l for l in self.license_flag_values if l]) == 0
-        has_many_license_flags = len([l for l in self.license_flag_values if l]) > 1
+        license_flags_count = sum(self.license_flags.values())
 
         license_expression = self.license_expression
 
@@ -1888,11 +1883,8 @@ def validate(self, licensing=None, thorough=False):
 
             if not (0 <= self.relevance <= 100):
                 yield 'Invalid rule relevance. Should be between 0 and 100.'
-            
-            if has_no_license_flags:
-                yield 'Invalid rule no is_license_* flags present.'
 
-            if has_many_license_flags:
+            if license_flags_count:
                 yield 'Invalid rule is_license_* flags. Only one allowed.'
 
             if not has_license_flags:
@@ -2295,6 +2287,8 @@ def dump(self, rules_data_dir, **kwargs):
         rule_file = self.rule_file(rules_data_dir=rules_data_dir)
 
         metadata = self.to_dict()
+        # This can be used to pass objects to dump on the rule file with
+        # other rule metadata, like debugging collection of required phrases
         if kwargs:
             metadata.update(kwargs)
         content = self.text
@@ -2328,7 +2322,7 @@ def load(self, rule_file, with_checks=True):
             raise e
 
         known_attributes = set(attr.fields_dict(self.__class__))
-        # This is an attirbute used to debug marking required phrases, and is not needed
+        # This is an attribute used to debug marking required phrases, and is not needed
         if "sources" in data:
             data.pop("sources")
         data_file_attributes = set(data)

src/licensedcode/required_phrases.py

@@ -52,11 +52,6 @@ def get_required_phrase_spans(text):
     >>> x = get_required_phrase_spans(text)
     >>> assert x == [Span(4, 6)], x
 
-    >>> text = 'This is enclosed in {{double curly braces}}'
-    >>> #       0    1  2        3    4      5     6
-    >>> x = get_required_phrase_spans(text)
-    >>> assert x == ['double', 'curly', 'braces'], x
-
     >>> text = 'This is {{enclosed}} a  {{double curly braces}} or not'
     >>> #       0    1    2          SW   3      4     5        6  7
     >>> x = get_required_phrase_spans(text)
@@ -114,7 +109,7 @@ def get_required_phrase_texts(text):
     >>> text = 'This is enclosed in {{double curly braces}}'
     >>> #       0    1  2        3    4      5     6
     >>> x = get_required_phrase_texts(text=text)
-    >>> assert x == ['double', 'curly', 'braces'], x
+    >>> assert x == ['double curly braces'], x
     """
     return [
         required_phrase.text