Modified the code structure based on suggestions

Author: Raj-Parekh24

Diff for: pytensor/graph/basic.py

@@ -605,13 +605,12 @@ def convert_string_keys_to_variables(input_to_values):
                     if not matching_vars:
                         raise Exception(f"{key} not found in graph")
                     elif len(matching_vars) > 1:
-                            raise Exception(
-                                f"Found multiple variables with name {key}"
-                            )
+                        raise Exception(f"Found multiple variables with name {key}")
                     new_input_to_values[matching_vars[0]] = value
                 else:
                     new_input_to_values[key] = value
             return new_input_to_values
+
         inputs_to_values = convert_string_keys_to_variables(inputs_to_values)
 
         if not hasattr(self, "_fn_cache"):

Diff for: tests/graph/test_basic.py

@@ -306,14 +306,14 @@ def test_eval_with_strings(self):
         assert self.w.eval({"x": 1.0, self.y: 2.0}) == 6.0
         assert self.w.eval({self.z: 3}) == 6.0
 
-    def test_eval_errors_having_mulitple_variables_same_name(self):
+    def test_eval_with_strings_multiple_matches(self):
         e = scalars("e")
         t = e + 1
         t.name = "e"
         with pytest.raises(Exception, match="Found multiple variables with name e"):
             t.eval({"e": 1})
 
-    def test_eval_errors_with_no_name_exists(self):
+    def test_eval_with_strings_no_match(self):
         e = scalars("e")
         t = e + 1
         t.name = "p"