Fix deepcode issues

Signed-off-by: Adam.Dybbroe <a000680@c21856.ad.smhi.se>
adybbroe · Jun 14, 2021 · ea993fa · ea993fa
1 parent 05632ca
commit ea993fa
Showing 1 changed file with 9 additions and 9 deletions.
diff --git a/nwcsafpps_runner/tests/test_level1c_runner.py b/nwcsafpps_runner/tests/test_level1c_runner.py
@@ -174,13 +174,13 @@ def test_check_service_is_supported(self):
             check_service_is_supported('avhrr')
 
         exception_raised = exec_info.value
-        self.assertTrue('Service name avhrr is not yet supported' == str(exception_raised))
+        self.assertEqual('Service name avhrr is not yet supported', str(exception_raised))
 
     def test_check_message_okay_message_ok(self):
 
         input_msg = Message.decode(rawstr=TEST_INPUT_MSG)
         result = check_message_okay(input_msg)
-        self.assertTrue(result is None)
+        self.assertEqual(result, None)
 
     def test_check_message_okay_message_has_no_dataset(self):
         """Test that message is not okay if it is not a dataset."""
@@ -190,7 +190,7 @@ def test_check_message_okay_message_has_no_dataset(self):
             result = check_message_okay(input_msg)
 
         exception_raised = exec_info.value
-        self.assertTrue("Not a dataset, don't do anything..." == str(exception_raised))
+        self.assertEqual("Not a dataset, don't do anything...", str(exception_raised))
 
     def test_check_message_okay_message_has_no_platform_name(self):
         """Test that message is not okay if it does not contain platform_name."""
@@ -200,7 +200,7 @@ def test_check_message_okay_message_has_no_platform_name(self):
             result = check_message_okay(input_msg)
 
         exception_raised = exec_info.value
-        self.assertTrue("Message is lacking crucial fields: platform_name" == str(exception_raised))
+        self.assertEqual("Message is lacking crucial fields: platform_name", str(exception_raised))
 
     def test_check_message_okay_message_has_no_start_time(self):
         """Test that message is not okay if it does not contain start_time."""
@@ -210,7 +210,7 @@ def test_check_message_okay_message_has_no_start_time(self):
             result = check_message_okay(input_msg)
 
         exception_raised = exec_info.value
-        self.assertTrue("Message is lacking crucial fields: start_time" == str(exception_raised))
+        self.assertEqual("Message is lacking crucial fields: start_time", str(exception_raised))
 
     @patch('nwcsafpps_runner.config.load_config_from_file')
     @patch('nwcsafpps_runner.l1c_processing.cpu_count')
@@ -235,8 +235,8 @@ def test_create_l1c_processor_instance(self, cpu_count, config):
         self.assertEqual(l1c_proc.result_home, '/san1/geo_in/lvl1c')
         self.assertEqual(l1c_proc.publish_topic, ['/1c/nc/0deg'])
         self.assertEqual(l1c_proc.subscribe_topics, ['/1b/hrit/0deg'])
-        self.assertTrue(l1c_proc.message_data is None)
-        self.assertTrue(l1c_proc.pool is None)
+        self.assertEqual(l1c_proc.message_data, None)
+        self.assertEqual(l1c_proc.pool, None)
 
     @patch('nwcsafpps_runner.config.load_config_from_file')
     @patch('nwcsafpps_runner.l1c_processing.cpu_count')
@@ -261,8 +261,8 @@ def test_create_l1c_processor_instance_minimal_config(self, cpu_count, config):
         self.assertEqual(l1c_proc.result_home, '/tmp')
         self.assertEqual(l1c_proc.publish_topic, ['/1c/nc/0deg'])
         self.assertEqual(l1c_proc.subscribe_topics, ['/1b/hrit/0deg'])
-        self.assertTrue(l1c_proc.message_data is None)
-        self.assertTrue(l1c_proc.pool is None)
+        self.assertEqual(l1c_proc.message_data, None)
+        self.assertEqual(l1c_proc.pool, None)
 
     @patch('nwcsafpps_runner.config.load_config_from_file')
     @patch('nwcsafpps_runner.l1c_processing.cpu_count')