Add is_notice flag to the --classify option, issue #3822

Signed-off-by: Aayush Kumar <[email protected]>

Author: aayushkdev

src/summarycode/classify.py

@@ -91,6 +91,9 @@ def get_relative_path(root_path, path):
     'readme',
 )
 
+NOTICE_STARTS_ENDS = (
+    "notice",
+)
 
 def check_resource_name_start_and_end(resource, STARTS_ENDS):
     """
@@ -111,6 +114,7 @@ def set_classification_flags(resource,
     _LEGAL=LEGAL_STARTS_ENDS,
     _MANIF=MANIFEST_ENDS,
     _README=README_STARTS_ENDS,
+    _NOTICE=NOTICE_STARTS_ENDS,
 ):
     """
     Set classification flags on the `resource` Resource
@@ -119,8 +123,9 @@ def set_classification_flags(resource,
 
     resource.is_legal = is_legal = check_resource_name_start_and_end(resource, _LEGAL)
     resource.is_readme = is_readme = check_resource_name_start_and_end(resource, _README)
+    resource.is_notice = is_notice = check_resource_name_start_and_end(resource, _NOTICE)
     # FIXME: this will never be picked up as this is NOT available in a pre-scan plugin
     has_package_data = bool(getattr(resource, 'package_data', False))
     resource.is_manifest = is_manifest = path.endswith(_MANIF) or has_package_data
-    resource.is_key_file = (resource.is_top_level and (is_readme or is_legal or is_manifest))
+    resource.is_key_file = (resource.is_top_level and (is_readme or is_legal or is_manifest or is_notice))
     return resource

src/summarycode/classify_plugin.py

@@ -75,7 +75,10 @@ class FileClassifier(PostScanPlugin):
 
         ('is_key_file',
          Boolean(help='True if this file is "top-level" file and either a '
-                      'legal, readme or manifest file.')),
+                      'legal, notice, readme or manifest file.')),
+                      
+        ('is_notice',
+        Boolean(help='True if this file is likely a notice file')),
 
 #         ('is_doc',
 #          Boolean(help='True if this file is likely a documentation file.')),

tests/summarycode/data/classify/cli.expected.json

@@ -24,6 +24,7 @@
       "is_readme": false,
       "is_top_level": true,
       "is_key_file": false,
+      "is_notice": false,
       "files_count": 8,
       "dirs_count": 1,
       "size_count": 0,
@@ -53,6 +54,7 @@
       "is_readme": true,
       "is_top_level": true,
       "is_key_file": true,
+      "is_notice": false,
       "files_count": 0,
       "dirs_count": 0,
       "size_count": 0,
@@ -82,6 +84,7 @@
       "is_readme": true,
       "is_top_level": true,
       "is_key_file": true,
+      "is_notice": false,
       "files_count": 0,
       "dirs_count": 0,
       "size_count": 0,
@@ -111,6 +114,7 @@
       "is_readme": false,
       "is_top_level": true,
       "is_key_file": true,
+      "is_notice": false,
       "files_count": 0,
       "dirs_count": 0,
       "size_count": 0,
@@ -140,6 +144,7 @@
       "is_readme": false,
       "is_top_level": true,
       "is_key_file": true,
+      "is_notice": false,
       "files_count": 0,
       "dirs_count": 0,
       "size_count": 0,
@@ -169,6 +174,7 @@
       "is_readme": false,
       "is_top_level": true,
       "is_key_file": false,
+      "is_notice": false,
       "files_count": 2,
       "dirs_count": 0,
       "size_count": 0,
@@ -198,6 +204,7 @@
       "is_readme": false,
       "is_top_level": false,
       "is_key_file": false,
+      "is_notice": false,
       "files_count": 0,
       "dirs_count": 0,
       "size_count": 0,
@@ -227,6 +234,7 @@
       "is_readme": false,
       "is_top_level": false,
       "is_key_file": false,
+      "is_notice": false,
       "files_count": 0,
       "dirs_count": 0,
       "size_count": 0,
@@ -256,6 +264,7 @@
       "is_readme": false,
       "is_top_level": true,
       "is_key_file": false,
+      "is_notice": false,
       "files_count": 0,
       "dirs_count": 0,
       "size_count": 0,
@@ -285,6 +294,7 @@
       "is_readme": false,
       "is_top_level": true,
       "is_key_file": false,
+      "is_notice": false,
       "files_count": 0,
       "dirs_count": 0,
       "size_count": 0,