Fixed sonarqube code smell

Author: pulk17

tests/api/test_middleware_auth.py

@@ -75,14 +75,14 @@ def test_revoked_token(self):
         self.assertEqual(res.status_code, 401)
 
     def test_expired_token(self):
-        plaintext, token = self.get_token(self.user, expires_in_days=-1)
+        plaintext, _ = self.get_token(self.user, expires_in_days=-1)
         res = self.client.get('/api/v1/system/queue',
                               headers={'Authorization': f'Bearer {plaintext}'})
         self.assertEqual(res.status_code, 401)
 
     def test_valid_token_missing_scope(self):
         # /api/v1/system/queue requires 'system:read'
-        plaintext, token = self.get_token(self.user, scopes=['runs:read'])
+        plaintext, _ = self.get_token(self.user, scopes=['runs:read'])
         res = self.client.get('/api/v1/system/queue',
                               headers={'Authorization': f'Bearer {plaintext}'})
         self.assertEqual(res.status_code, 403)
@@ -91,29 +91,29 @@ def test_valid_token_missing_scope(self):
         self.assertIn('missing_scopes', res.json['details'])
 
     def test_valid_token_with_scope(self):
-        plaintext, token = self.get_token(self.user, scopes=['system:read'])
+        plaintext, _ = self.get_token(self.user, scopes=['system:read'])
         res = self.client.get('/api/v1/system/queue',
                               headers={'Authorization': f'Bearer {plaintext}'})
         self.assertEqual(res.status_code, 200)
 
     def test_role_decorator_missing_role(self):
         # GET /api/v1/auth/tokens requires 'tokens:manage' and roles ['admin', 'contributor', 'tester']
-        plaintext, token = self.get_token(
+        plaintext, _ = self.get_token(
             self.user, scopes=['tokens:manage'])  # role is user
         res = self.client.get('/api/v1/auth/tokens',
                               headers={'Authorization': f'Bearer {plaintext}'})
         self.assertEqual(res.status_code, 403)
         self.assertEqual(res.json['code'], 'forbidden')
 
     def test_role_decorator_with_role(self):
-        plaintext, token = self.get_token(
+        plaintext, _ = self.get_token(
             self.admin, scopes=['tokens:manage'])  # role is admin
         res = self.client.get('/api/v1/auth/tokens',
                               headers={'Authorization': f'Bearer {plaintext}'})
         self.assertEqual(res.status_code, 200)
 
     def test_scope_boundary_write_endpoints_fail_on_read_only_scopes(self):
-        plaintext, token = self.get_token(
+        plaintext, _ = self.get_token(
             self.user, scopes=['runs:read', 'results:read'])
 
         # 1. POST /runs

tests/api/test_services_log_service.py

@@ -68,7 +68,7 @@ def test_read_log_lines_limit_clamped(self, mock_get_path):
         path = self.create_log_file(content)
         mock_get_path.return_value = path
 
-        lines, next_cursor = read_log_lines(1, limit=2000)
+        lines, _ = read_log_lines(1, limit=2000)
         # Should be clamped to 500
         self.assertEqual(len(lines), 500)
 
@@ -79,16 +79,16 @@ def test_read_log_lines_filters(self, mock_get_path):
         mock_get_path.return_value = path
 
         # Filter by level
-        lines, next_cursor = read_log_lines(1, level='error')
+        lines, _ = read_log_lines(1, level='error')
         self.assertEqual(len(lines), 1)
         self.assertEqual(lines[0]['message'], "ERROR build: Failed")
 
         # Filter by source
-        lines, next_cursor = read_log_lines(1, source='build')
+        lines, _ = read_log_lines(1, source='build')
         self.assertEqual(len(lines), 2)
 
         # Filter by contains
-        lines, next_cursor = read_log_lines(1, contains='STARTING')
+        lines, _ = read_log_lines(1, contains='STARTING')
         self.assertEqual(len(lines), 1)
 
     @patch('mod_api.services.log_service.get_log_file_path')