# HG changeset patch # User sylvain.thenault@logilab.fr # Date 1238849112 -7200 # Node ID 25cd18899830b64c1e3f4b0627a209580ee1a015 # Parent 1457a545af0358d8a3250e5b43eba883e17c021f stop monkeypatching choose_term in test diff -r 1457a545af03 -r 25cd18899830 devtools/repotest.py --- a/devtools/repotest.py Sat Apr 04 14:44:04 2009 +0200 +++ b/devtools/repotest.py Sat Apr 04 14:45:12 2009 +0200 @@ -258,38 +258,38 @@ class PartPlanInformation(object): def merge_input_maps(*args): pass - def _choose_term(self, sourceterms): - pass +# def _choose_term(self, sourceterms): +# pass _orig_merge_input_maps = PartPlanInformation.merge_input_maps -_orig_choose_term = PartPlanInformation._choose_term +# _orig_choose_term = PartPlanInformation._choose_term def _merge_input_maps(*args): return sorted(_orig_merge_input_maps(*args)) -def _choose_term(self, sourceterms): - # predictable order for test purpose - def get_key(x): - try: - # variable - return x.name - except AttributeError: - try: - # relation - return x.r_type - except AttributeError: - # const - return x.value - varsinorder = sorted(sourceterms, key=get_key) - if len(self._sourcesterms) > 1: - for var in varsinorder: - if not var.scope is self.rqlst: - return var, sourceterms.pop(var) - else: - for var in varsinorder: - if var.scope is self.rqlst: - return var, sourceterms.pop(var) - var = varsinorder[0] - return var, sourceterms.pop(var) +# def _choose_term(self, sourceterms): +# # predictable order for test purpose +# def get_key(x): +# try: +# # variable +# return x.name +# except AttributeError: +# try: +# # relation +# return x.r_type +# except AttributeError: +# # const +# return x.value +# varsinorder = sorted(sourceterms, key=get_key) +# if len(self._sourcesterms) > 1: +# for var in varsinorder: +# if not var.scope is self.rqlst: +# return var, sourceterms.pop(var) +# else: +# for var in varsinorder: +# if var.scope is self.rqlst: +# return var, sourceterms.pop(var) +# var = varsinorder[0] +# return var, sourceterms.pop(var) def do_monkey_patch(): @@ -299,7 +299,7 @@ ExecutionPlan.tablesinorder = None ExecutionPlan.init_temp_table = _init_temp_table PartPlanInformation.merge_input_maps = _merge_input_maps - PartPlanInformation._choose_term = _choose_term + #PartPlanInformation._choose_term = _choose_term def undo_monkey_patch(): RQLRewriter.insert_snippets = _orig_insert_snippets @@ -307,5 +307,5 @@ ExecutionPlan._check_permissions = _orig_check_permissions ExecutionPlan.init_temp_table = _orig_init_temp_table PartPlanInformation.merge_input_maps = _orig_merge_input_maps - PartPlanInformation._choose_term = _orig_choose_term + #PartPlanInformation._choose_term = _orig_choose_term