fixed bandit and linting issues.

Signed-off-by: Manav Gupta <[email protected]>

Author: manavgup

tests/integration/test_utils_pep621.py

@@ -1,4 +1,17 @@
 # This file is part of CycloneDX Python
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
 # SPDX-License-Identifier: Apache-2.0
 # Copyright (c) OWASP Foundation. All Rights Reserved.
 
@@ -16,36 +29,36 @@ class TestUtilsPEP621(unittest.TestCase):
 
     def test_license_dict_text_pep621(self) -> None:
         lfac = LicenseFactory()
-        fpath = tempfile.mktemp()
-        project = {
-            'name': 'testpkg',
-            'license': {'text': 'This is the license text.'},
-        }
-        licenses = list(project2licenses(project, lfac, fpath=fpath))
-        self.assertEqual(len(licenses), 1)
-        lic = licenses[0]
-        self.assertIsInstance(lic, DisjunctiveLicense)
-        self.assertIsNone(lic.id)
-        self.assertEqual(lic.text.content, 'This is the license text.')
-        self.assertEqual(lic.acknowledgement, LicenseAcknowledgement.DECLARED)
+        with tempfile.TemporaryDirectory() as tmpdir:
+            fpath = tmpdir  # Use the temp directory as the base for any temp files
+            project = {
+                'name': 'testpkg',
+                'license': {'text': 'This is the license text.'},
+            }
+            licenses = list(project2licenses(project, lfac, fpath=fpath))
+            self.assertEqual(len(licenses), 1)
+            lic = licenses[0]
+            self.assertIsInstance(lic, DisjunctiveLicense)
+            self.assertIsNone(lic.id)
+            self.assertEqual(lic.text.content, 'This is the license text.')
+            self.assertEqual(lic.acknowledgement, LicenseAcknowledgement.DECLARED)
 
     def test_license_dict_file_pep621(self) -> None:
         lfac = LicenseFactory()
-        with tempfile.NamedTemporaryFile('w+', delete=True) as tf:
-            tf.write('File license text')
-            tf.flush()
+        with tempfile.TemporaryDirectory() as tmpdir:
+            file_path = os.path.join(tmpdir, 'license.txt')
+            with open(file_path, 'w') as tf:
+                tf.write('File license text')
             project = {
                 'name': 'testpkg',
-                'license': {'file': os.path.basename(tf.name)},
+                'license': {'file': 'license.txt'},
             }
-            # fpath should be the file path so dirname(fpath) resolves to the correct directory
-            licenses = list(project2licenses(project, lfac, fpath=tf.name))
-
-        self.assertEqual(len(licenses), 1)
-        lic = licenses[0]
-        self.assertIsInstance(lic, DisjunctiveLicense)
-        self.assertIsNotNone(lic.text.content)
-        self.assertEqual(lic.acknowledgement, LicenseAcknowledgement.DECLARED)
+            licenses = list(project2licenses(project, lfac, fpath=file_path))
+            self.assertEqual(len(licenses), 1)
+            lic = licenses[0]
+            self.assertIsInstance(lic, DisjunctiveLicense)
+            self.assertIsNotNone(lic.text.content)
+            self.assertEqual(lic.acknowledgement, LicenseAcknowledgement.DECLARED)
 
     def test_license_non_dict_pep621(self) -> None:
         lfac = LicenseFactory()