Relabel plugin to test to avoid confusion

The original code considered plugins, but not blacklists.
Since the code now correctly considers both types of tests,
the function and variables should be renamed to avoid confusion.

bandit/core/config.py

@@ -151,11 +151,11 @@ def convert_names_to_ids(self):
             # sometimes explicity 'None', for example when the list if given in
             # yaml but not populated with any values.
             include = {
-                (extman.get_plugin_id(i) or i)
+                (extman.get_test_id(i) or i)
                 for i in (profile.get("include") or [])
             }
             exclude = {
-                (extman.get_plugin_id(i) or i)
+                (extman.get_test_id(i) or i)
                 for i in (profile.get("exclude") or [])
             }
             updated_profiles[name] = {"include": include, "exclude": exclude}

bandit/core/extension_loader.py

@@ -53,11 +53,11 @@ def test_has_id(plugin):
         self.plugins_by_id = {p.plugin._test_id: p for p in self.plugins}
         self.plugins_by_name = {p.name: p for p in self.plugins}
 
-    def get_plugin_id(self, plugin_name):
-        if plugin_name in self.plugins_by_name:
-            return self.plugins_by_name[plugin_name].plugin._test_id
-        if plugin_name in self.blacklist_by_name:
-            return self.blacklist_by_name[plugin_name]["id"]
+    def get_test_id(self, test_name):
+        if test_name in self.plugins_by_name:
+            return self.plugins_by_name[test_name].plugin._test_id
+        if test_name in self.blacklist_by_name:
+            return self.blacklist_by_name[test_name]["id"]
         return None
 
     def load_blacklists(self, blacklist_namespace):

bandit/core/manager.py

@@ -460,17 +460,17 @@ def _find_candidate_matches(unmatched_issues, results_list):
 
 
 def _find_test_id_from_nosec_string(extman, match):
-    plugin_id = extman.check_id(match)
-    if plugin_id:
+    test_id = extman.check_id(match)
+    if test_id:
         return match
-    # Finding by short_id didn't work, let's check the plugin name
-    plugin_id = extman.get_plugin_id(match)
-    if not plugin_id:
+    # Finding by short_id didn't work, let's check the test name
+    test_id = extman.get_test_id(match)
+    if not test_id:
         # Name and short id didn't work:
         LOG.warning(
             "Test in comment: %s is not a test name or id, ignoring", match
         )
-    return plugin_id  # We want to return None or the string here regardless
+    return test_id  # We want to return None or the string here regardless
 
 
 def _parse_nosec_comment(comment):