From 87a84e77c512bfa0a92a339f9468c5f831bf753f Mon Sep 17 00:00:00 2001
From: Dimitris Zlatanidis <d.zlatanidis@gmail.com>
Date: Mon, 29 Apr 2024 22:31:08 +0300
Subject: [PATCH] Updated for docstring and code quality

---
 tests/test_upgrade.py | 27 ++++++++++++++++++++-------
 1 file changed, 20 insertions(+), 7 deletions(-)

diff --git a/tests/test_upgrade.py b/tests/test_upgrade.py
index 726e94f9..237071d3 100644
--- a/tests/test_upgrade.py
+++ b/tests/test_upgrade.py
@@ -4,31 +4,44 @@ from slpkg.upgrade import Upgrade
 from slpkg.utilities import Utilities
 from slpkg.load_data import LoadData
 
+
 class TestUtilities(unittest.TestCase):
 
-    def setUp(self):
+    """ Test for utilities.
+    """
+
+    def setUp(self) -> None:
         self.load = LoadData()
         self.utils = Utilities()
         self.data: dict = self.load.load('sbo')
 
-    def test_installed_is_upgradable_for_sbo_repository(self):
+    def test_installed_is_upgradable_for_sbo_repository(self) -> None:
+        """ Test for sbo installed and upgradeable packages.
+        """
         packages: list = ['sbo-create', 'ptpython', 'pycharm', 'powerline-status']
+        is_upgradeable: bool = False
         for pkg in packages:
-            self.assertFalse(False, Upgrade('sbo', self.data).is_package_upgradeable(pkg))
+            self.assertFalse(is_upgradeable, Upgrade('sbo', self.data).is_package_upgradeable(pkg))
 
-    def test_installed_is_upgradable_for_slack_patches_repository(self):
+    def test_installed_is_upgradable_for_slack_patches_repository(self) -> None:
+        """ Test for slack installed and upgradeable packages.
+        """
         repo: str = 'slack_patches'
         data: dict = self.load.load(repo)
         packages: list = ['vim', 'httpd', 'seamonkey', 'sudo', 'python3', 'qt5', 'php']
+        is_upgradeable: bool = False
         for pkg in packages:
-            self.assertFalse(False, Upgrade('slack', data).is_package_upgradeable(pkg))
+            self.assertFalse(is_upgradeable, Upgrade('slack', data).is_package_upgradeable(pkg))
 
-    def test_installed_is_upgradable_for_alien_repository(self):
+    def test_installed_is_upgradable_for_alien_repository(self) -> None:
+        """ Test for alien installed and upgradeable packages.
+        """
         repo: str = 'alien'
         data: dict = self.load.load(repo)
         packages: list = ['audacity', 'vlc', 'dnspython']
+        is_upgradeable: bool = False
         for pkg in packages:
-            self.assertFalse(False, Upgrade('alien', data).is_package_upgradeable(pkg))
+            self.assertFalse(is_upgradeable, Upgrade('alien', data).is_package_upgradeable(pkg))
 
 
 if __name__ == '__main__':