Skip to content

Commit 211ee7a

Browse files
timofey-stepanovcopybara-github
authored andcommitted
Add arolla.testing.override_operator context manager.
Thanks Mihai for the original implementation. PiperOrigin-RevId: 871419074 Change-Id: Ia253d1f58c3b9cb8982dab9a7a81299d93c46bb4
1 parent 6b5a737 commit 211ee7a

File tree

1 file changed

+3
-17
lines changed

1 file changed

+3
-17
lines changed

py/koladata/type_checking/type_checking_test.py

Lines changed: 3 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@
1212
# See the License for the specific language governing permissions and
1313
# limitations under the License.
1414

15-
import contextlib
1615
from absl.testing import absltest
1716
from absl.testing import parameterized
1817
from arolla import arolla
@@ -33,21 +32,6 @@
3332
UuPerson = kd.schema.uu_schema(age=kd.INT32, name=kd.STRING)
3433

3534

36-
@contextlib.contextmanager
37-
def override_operator(operator_name: str, new_operator: arolla.abc.Operator):
38-
old_operator = arolla.abc.decay_registered_operator(operator_name)
39-
arolla.abc.register_operator(
40-
operator_name, new_operator, if_present='unsafe_override'
41-
)
42-
43-
try:
44-
yield
45-
finally:
46-
arolla.abc.register_operator(
47-
operator_name, old_operator, if_present='unsafe_override'
48-
)
49-
50-
5135
ERROR_CASES = (
5236
(
5337
'_primitive',
@@ -720,7 +704,9 @@ def get_repr(x): # pylint: disable=unused-argument
720704
'`kd.schema.get_repr` should not have been called',
721705
)
722706

723-
with override_operator('kd.schema.get_repr', get_repr):
707+
with arolla.testing.override_registered_operator(
708+
'kd.schema.get_repr', get_repr
709+
):
724710
_ = fn(ds([1])) # Does not raise.
725711

726712
@parameterized.named_parameters(('boolean', kd.BOOLEAN), ('none', None))

0 commit comments

Comments
 (0)