diff options
author | Peter Andersson <[email protected]> | 2012-03-18 01:56:40 +0100 |
---|---|---|
committer | Peter Andersson <[email protected]> | 2012-03-19 14:16:04 +0100 |
commit | 86acf3b0515628e645f1641c3b0542071a87d05d (patch) | |
tree | 43697fcc4f7bfd5c00fe3e50912ffe90a68dfc97 /lib/common_test/src/ct_hooks.erl | |
parent | 2409000567436f33fd912e9a809cc9dda3e9172f (diff) | |
download | otp-86acf3b0515628e645f1641c3b0542071a87d05d.tar.gz otp-86acf3b0515628e645f1641c3b0542071a87d05d.tar.bz2 otp-86acf3b0515628e645f1641c3b0542071a87d05d.zip |
Fix bug in handling ct_hooks start arguments
Diffstat (limited to 'lib/common_test/src/ct_hooks.erl')
-rw-r--r-- | lib/common_test/src/ct_hooks.erl | 19 |
1 files changed, 2 insertions, 17 deletions
diff --git a/lib/common_test/src/ct_hooks.erl b/lib/common_test/src/ct_hooks.erl index 8f452ef5e3..2a23cd992b 100644 --- a/lib/common_test/src/ct_hooks.erl +++ b/lib/common_test/src/ct_hooks.erl @@ -213,25 +213,10 @@ call([{HookId, Fun} | Rest], Config, Meta, Hooks) -> try Hook = lists:keyfind(HookId, #ct_hook_config.id, Hooks), {NewConf, NewHook} = Fun(Hook, Config, Meta), - - %%! --- Fri Mar 16 17:27:25 2012 --- peppe was here! - %%!!?? - NewConf1 = - if is_tuple(NewConf) -> - if element(1, NewConf) == skip; - element(1, NewConf) == fail -> - Config; - true -> - NewConf - end; - true -> - NewConf - end, - - NewCalls = get_new_hooks(NewConf1, Fun), + NewCalls = get_new_hooks(NewConf, Fun), NewHooks = lists:keyreplace(HookId, #ct_hook_config.id, Hooks, NewHook), call(resort(NewCalls ++ Rest,NewHooks,Meta), %% Resort if call_init changed prio - remove(?config_name, NewConf1), Meta, + remove(?config_name, NewConf), Meta, terminate_if_scope_ends(HookId, Meta, NewHooks)) catch throw:{error_in_cth_call,Reason} -> call(Rest, {fail, Reason}, Meta, |