minor code changes with some tests commented out

Signed-off-by: Chin Yeung Li <[email protected]>

Author: chinyeungli

src/attributecode/model.py

@@ -1563,49 +1563,58 @@ def pre_process_and_fetch_license_dict(abouts, api_url=None, api_key=None, scanc
         auth_error = Error(ERROR, u"Authorization denied. Invalid '--api_key'. License generation is skipped.")
         if auth_error in errors:
             break
-        if not about.license_file.value:
+        #if not about.license_file.value:
+        # FIXME
+        # Scancode returns license_expressions while ABcTK uses license_expression
+        lic_exp = ''
+        if about.license_expression or about.license_expressions:
             if about.license_expression.value:
-                special_char_in_expression, lic_list = parse_license_expression(about.license_expression.value)
-                if special_char_in_expression:
-                    msg = (about.about_file_path + u": The following character(s) cannot be in the license_expression: " +
-                           str(special_char_in_expression))
-                    errors.append(Error(ERROR, msg))
-                else:
-                    for lic_key in lic_list:
-                        if not lic_key in captured_license:
-                            lic_url = ''
-                            license_name = ''
-                            license_filename = ''
-                            license_text = ''
-                            detail_list = []
-                            if api_key:
-                                license_name, _license_key, license_text, errs = api.get_license_details_from_api(url, api_key, lic_key)
-                                for severity, message in errs: 
-                                    msg = (about.about_file_path + ": " + message)
-                                    errors.append(Error(severity, msg))
-                                license_filename = lic_key + '.LICENSE'
-                                lic_url = lic_urn + lic_key
-                            else:
-                                license_url = url + lic_key + '.json'
-                                license_text_url = url + lic_key + '.LICENSE'
-                                try:
-                                    json_url = urlopen(license_url)
-                                    data = json.loads(json_url.read())
-                                    license_name = data['name']
-                                    license_text = urllib.request.urlopen(license_text_url).read().decode('utf-8')
-                                    license_filename = data['key'] + '.LICENSE'
-                                    lic_url = url + license_filename
-                                except:
-                                    msg = about.about_file_path + u" : Invalid 'license': " + lic_key
-                                    errors.append(Error(ERROR, msg))
-                            captured_license.append(lic_key)
-                            detail_list.append(license_name)
-                            detail_list.append(license_filename)
-                            detail_list.append(license_text)
-                            detail_list.append(lic_url)
-                            key_text_dict[lic_key] = detail_list
-                    if not about.license_key.value:
-                        about.license_key.value = lic_list
+                lic_exp = about.license_expression.value
+            else:
+                lic_exp = about.license_expressions.value
+             
+        if lic_exp:
+            special_char_in_expression, lic_list = parse_license_expression(lic_exp)
+            if special_char_in_expression:
+                msg = (about.about_file_path + u": The following character(s) cannot be in the license_expression: " +
+                       str(special_char_in_expression))
+                errors.append(Error(ERROR, msg))
+            else:
+                for lic_key in lic_list:
+                    if not lic_key in captured_license:
+                        lic_url = ''
+                        license_name = ''
+                        license_filename = ''
+                        license_text = ''
+                        detail_list = []
+                        if api_key:
+                            license_name, _license_key, license_text, errs = api.get_license_details_from_api(url, api_key, lic_key)
+                            for severity, message in errs: 
+                                msg = (about.about_file_path + ": " + message)
+                                errors.append(Error(severity, msg))
+                            license_filename = lic_key + '.LICENSE'
+                            lic_url = lic_urn + lic_key
+                        else:
+                            license_url = url + lic_key + '.json'
+                            license_text_url = url + lic_key + '.LICENSE'
+                            try:
+                                json_url = urlopen(license_url)
+                                data = json.loads(json_url.read())
+                                license_name = data['name']
+                                license_text = urllib.request.urlopen(license_text_url).read().decode('utf-8')
+                                license_filename = data['key'] + '.LICENSE'
+                                lic_url = url + license_filename
+                            except:
+                                msg = about.about_file_path + u" : Invalid 'license': " + lic_key
+                                errors.append(Error(ERROR, msg))
+                        captured_license.append(lic_key)
+                        detail_list.append(license_name)
+                        detail_list.append(license_filename)
+                        detail_list.append(license_text)
+                        detail_list.append(lic_url)
+                        key_text_dict[lic_key] = detail_list
+                if not about.license_key.value:
+                    about.license_key.value = lic_list
     return key_text_dict, errors
 
 

tests/test_attrib.py

@@ -21,7 +21,9 @@
 from testing_utils import get_test_loc
 from testing_utils import get_temp_file
 
+from attributecode import INFO
 from attributecode import attrib
+from attributecode import gen
 from attributecode import model
 
 
@@ -135,6 +137,76 @@ def test_lic_key_name_sync(self):
 
         assert f1 == f2
 
+    """
+    def test_custom_template(self):
+        test_file = get_test_loc('test_attrib/scancode_custom_template/clean-text-0.3.0-mod-lceupi.json')
+        custom_template =  get_test_loc('test_attrib/scancode_custom_template/scancode.template')
+        errors, abouts = gen.load_inventory(test_file, scancode=True)
+        expected_errors = [(40, 'Field about_resource: Unable to verify path: isc_lic.py: No base directory provided'), (30, "Field license_key: ignored duplicated list value: 'isc'"), (30, "Field license_name: ignored duplicated list value: 'ISC License'")]
+        result = [(level, e) for level, e in errors if level > INFO]
+        assert expected_errors == result
+        #assert not errors
+
+        lic_dict = {'isc': {'key': 'isc', 'short_name': 'ISC License', 'name': 'ISC License',
+                            'category': 'Permissive', 'owner': 'ISC - Internet Systems Consortium',
+                            'homepage_url': 'https://www.isc.org/software/license', 'notes': 'Per SPDX.org, this license is OSI certified.',
+                            'spdx_license_key': 'ISC', 'text_urls': ['http://fedoraproject.org/wiki/Licensing:MIT#Old_Style_with_legal_disclaimer_2', 'http://openbsd.wikia.com/wiki/OpenBSD%27s_BSD_license', 'http://opensource.org/licenses/isc-license.txt', 'https://www.isc.org/software/license'],
+                            'osi_url': 'http://opensource.org/licenses/isc-license.txt', 'other_urls': ['http://openbsd.wikia.com/wiki/OpenBSD%27s_BSD_license', 'http://www.isc.org/software/license', 'http://www.opensource.org/licenses/ISC', 'https://opensource.org/licenses/ISC', 'https://www.isc.org/downloads/software-support-policy/isc-license/', 'https://www.isc.org/isc-license-1.0.html'],
+                            'license_text': 'Permission to use, copy, modify, and/or distribute this software for any purpose\nwith or without fee is hereby granted, provided that the above copyright notice\nand this permission notice appear in all copies.\n\nTHE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES WITH\nREGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF MERCHANTABILITY AND\nFITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR ANY SPECIAL, DIRECT,\nINDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES WHATSOEVER RESULTING FROM LOSS\nOF USE, DATA OR PROFITS, WHETHER IN AN ACTION OF CONTRACT, NEGLIGENCE OR OTHER\nTORTIOUS ACTION, ARISING OUT OF OR IN CONNECTION WITH THE USE OR PERFORMANCE OF\nTHIS SOFTWARE.\n'}}
+        is_about_input = False
+        errors, result = attrib.generate_from_file(abouts, is_about_input, lic_dict, min_license_score=0, template_loc=custom_template)
+        expected_errors = []
+        #result = [(level, e) for level, e in errors if level > INFO]
+        #assert expected_errors == result
+        assert not errors
+
+        expected_file = get_test_loc(
+            'test_attrib/scancode_custom_template/expect.html')
+        with open(expected_file) as exp:
+            expected = exp.read()
+
+        # strip the timestamp: the timestamp is wrapped in italic block
+        result = remove_timestamp(result)
+        expected = remove_timestamp(expected)
+        assert expected == result
+
+    def test_generate_with_default_template(self):
+        test_file = get_test_loc('test_attrib/default_template/simple_sample.csv')
+        errors, abouts = util.load_inventory(test_file)
+        assert not errors
+
+        lic_dict = {'bsd-new': 
+                    {'key': 'bsd-new', 'short_name': 'BSD-3-Clause', 'name': 'BSD-3-Clause',
+                     'category': 'Permissive', 'owner': 'Regents of the University of California',
+                     'homepage_url': 'http://www.opensource.org/licenses/BSD-3-Clause',
+                     'notes': 'Per SPDX.org, this license is OSI certified.',
+                     'spdx_license_key': 'BSD-3-Clause', 'osi_license_key': 'BSD-3',
+                     'text_urls': ['http://www.opensource.org/licenses/BSD-3-Clause'],
+                     'osi_url': 'http://www.opensource.org/licenses/BSD-3-Clause',
+                     'other_urls': ['http://framework.zend.com/license/new-bsd', 'https://opensource.org/licenses/BSD-3-Clause'],
+                     'license_text': 'Redistribution and use in source and binary forms, with or without modification,\nare permitted provided that the following conditions are met:\n\nRedistributions of source code must retain the above copyright notice, this list\nof conditions and the following disclaimer.\n\nRedistributions in binary form must reproduce the above copyright notice, this\nlist of conditions and the following disclaimer in the documentation and/or\nother materials provided with the distribution.\n\nNeither the name of the ORGANIZATION nor the names of its contributors may be\nused to endorse or promote products derived from this software without specific\nprior written permission.\n\nTHIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS\n"AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,\nTHE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE\nARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS\nBE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR\nCONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE\nGOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)\nHOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT\nLIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF\nTHE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.'},
+                    'mit': {'key': 'mit', 'short_name': 'MIT License', 'name': 'MIT License',
+                            'category': 'Permissive', 'owner': 'MIT',
+                            'homepage_url': 'http://opensource.org/licenses/mit-license.php',
+                            'notes': 'Per SPDX.org, this license is OSI certified.', 'spdx_license_key': 'MIT',
+                            'text_urls': ['http://opensource.org/licenses/mit-license.php'],
+                            'osi_url': 'http://www.opensource.org/licenses/MIT',
+                            'other_urls': ['https://opensource.com/article/18/3/patent-grant-mit-license', 'https://opensource.com/article/19/4/history-mit-license', 'https://opensource.org/licenses/MIT'],
+                            'license_text': 'Permission is hereby granted, free of charge, to any person obtaining\na copy of this software and associated documentation files (the\n"Software"), to deal in the Software without restriction, including\nwithout limitation the rights to use, copy, modify, merge, publish,\ndistribute, sublicense, and/or sell copies of the Software, and to\npermit persons to whom the Software is furnished to do so, subject to\nthe following conditions:\n\nThe above copyright notice and this permission notice shall be\nincluded in all copies or substantial portions of the Software.\n\nTHE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,\nEXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF\nMERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.\nIN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY\nCLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT,\nTORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE\nSOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.'}
+                    }
+        error, result = attrib.generate_from_file(abouts, lic_dict, min_license_score=0)
+        assert not error
+
+        expected_file = get_test_loc(
+            'test_attrib/default_template/expect.html')
+        with open(expected_file) as exp:
+            expected = exp.read()
+
+        # strip the timestamp: the timestamp is wrapped in italic block
+        result = remove_timestamp(result)
+        expected = remove_timestamp(expected)
+        assert expected == result
+    """
 
 def remove_timestamp(html_text):
     """

tests/testdata/test_attrib/default_template/expect.html

@@ -0,0 +1,151 @@
+
+<!doctype html>
+<html>
+  <head>
+    <style type="text/css">
+      div.additional-license-text-list {display:block}
+      body {font-family: Helvetica, Arial, sans-serif;}
+      pre {white-space: pre-wrap;}
+    </style>
+    <title>Open Source Software Information</title>
+  </head>
+
+  <body>
+    <h1>OPEN SOURCE SOFTWARE INFORMATION</h1>
+    <h2>  </h2>
+    <div>
+      <p>Licenses, acknowledgments and required copyright notices for 
+      open source components:</p>
+    </div>
+    
+        <div class="oss-table-of-contents">
+            
+                <p><a href="#component_0">cryptohash-sha256 v 0.11.100.1</a></p>
+            
+                <p><a href="#component_1">some_component v 0.0.1</a></p>
+            
+        </div>
+
+    <hr/>
+
+
+    
+        <div class="oss-component" id="component_0">
+            <h3 class="component-name">cryptohash-sha256
+                v 0.11.100.1
+            </h3>
+            
+                <p>This component is licensed under 
+                bsd-new and mit
+            
+            
+            
+            
+                
+                    
+                        <p>Full text of
+                            <a class="bsd-new" href="#component-license-bsd-new">
+                            bsd-new
+                            </a>
+                            is available at the end of this document.</p>
+                    
+                 
+                    
+                        <p>Full text of
+                            <a class="mit" href="#component-license-mit">
+                            mit
+                            </a>
+                            is available at the end of this document.</p>
+                    
+                 
+            
+            
+        </div>
+    
+        <div class="oss-component" id="component_1">
+            <h3 class="component-name">some_component
+                v 0.0.1
+            </h3>
+            
+                <p>This component is licensed under 
+                mit
+            
+            
+            
+            
+                
+                    
+                        <p>Full text of
+                            <a class="mit" href="#component-license-mit">
+                            mit
+                            </a>
+                            is available at the end of this document.</p>
+                    
+                 
+            
+            
+        </div>
+    
+
+    <hr/>
+
+    <h3>Licenses Used in This Product</h3>
+
+    
+        <h3 id="component-license-bsd-new">bsd-new</h3>
+        <pre>Redistribution and use in source and binary forms, with or without modification,
+are permitted provided that the following conditions are met:
+
+Redistributions of source code must retain the above copyright notice, this list
+of conditions and the following disclaimer.
+
+Redistributions in binary form must reproduce the above copyright notice, this
+list of conditions and the following disclaimer in the documentation and/or
+other materials provided with the distribution.
+
+Neither the name of the ORGANIZATION nor the names of its contributors may be
+used to endorse or promote products derived from this software without specific
+prior written permission.
+
+THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+&#34;AS IS&#34; AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
+THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
+ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS
+BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
+CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE
+GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
+HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
+LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF
+THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.</pre>
+        
+            <pre>(http://www.opensource.org/licenses/BSD-3-Clause)</pre>
+        
+    
+        <h3 id="component-license-mit">mit</h3>
+        <pre>Permission is hereby granted, free of charge, to any person obtaining
+a copy of this software and associated documentation files (the
+&#34;Software&#34;), to deal in the Software without restriction, including
+without limitation the rights to use, copy, modify, merge, publish,
+distribute, sublicense, and/or sell copies of the Software, and to
+permit persons to whom the Software is furnished to do so, subject to
+the following conditions:
+
+The above copyright notice and this permission notice shall be
+included in all copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED &#34;AS IS&#34;, WITHOUT WARRANTY OF ANY KIND,
+EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
+IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY
+CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT,
+TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE
+SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.</pre>
+        
+            <pre>(http://opensource.org/licenses/mit-license.php)</pre>
+        
+    
+
+    <h3><a id="End">End</a></h3>
+    <i>This file was generated with AttributeCode version: 0.0.1 on: 2020-12-09 09:43:02.595712 (UTC)</i>
+    </body>
+</html>

tests/testdata/test_attrib/default_template/simple_sample.csv

@@ -0,0 +1,3 @@
+name,version,license_expression
+cryptohash-sha256,v 0.11.100.1,bsd-new and mit
+some_component,v 0.0.1,mit