Test wasn't actually covering rewrite

Author: ricardoV94

pytensor/tensor/rewriting/math.py

@@ -147,11 +147,11 @@ def local_0_dot_x(fgraph, node):
     x, y = node.inputs
     if (
         get_underlying_scalar_constant_value(
-            x, only_process_constants=True, raise_not_constant=False
+            x, only_process_constants=False, raise_not_constant=False
         )
         == 0
         or get_underlying_scalar_constant_value(
-            y, only_process_constants=True, raise_not_constant=False
+            y, only_process_constants=False, raise_not_constant=False
         )
         == 0
     ):

tests/tensor/rewriting/test_subtensor.py

@@ -1448,11 +1448,14 @@ def test_dot_allocs_0(self):
                         not isinstance(n.op, Dot) for n in f.maker.fgraph.toposort()
                     )
 
-                    # test that we don't remove shape errors
+                    # test that we don't remove shape errors if we exclude shape_unsafe
+                    f_safe = f = function(
+                        [_e1[0], _e2[0]], o, mode=self.mode.excluding("shape_unsafe")
+                    )
                     with pytest.raises((ValueError, AssertionError)):
-                        f(_e1[1], _e2[2])
+                        f_safe(_e1[1], _e2[2])
                     with pytest.raises((ValueError, AssertionError)):
-                        f(_e1[2], _e2[1])
+                        f_safe(_e1[2], _e2[1])
 
 
 def test_local_IncSubtensor_serialize():