Index: openacs-4/packages/acs-tcl/tcl/test/html-conversion-procs.tcl =================================================================== RCS file: /usr/local/cvsroot/openacs-4/packages/acs-tcl/tcl/test/html-conversion-procs.tcl,v diff -u -N -r1.16 -r1.17 --- openacs-4/packages/acs-tcl/tcl/test/html-conversion-procs.tcl 18 Jul 2018 18:54:23 -0000 1.16 +++ openacs-4/packages/acs-tcl/tcl/test/html-conversion-procs.tcl 22 Jul 2018 09:58:43 -0000 1.17 @@ -212,7 +212,7 @@ set html {"<&text>"} aa_log "Unquote html=$html" set result [ns_quotehtml $html] - aa_true "Quoute html=$result" [string equal ""<&text>"" $result] + aa_equals "Quoute html=$result" ""<&text>"" $result #unquote html set html $result Index: openacs-4/packages/acs-tcl/tcl/test/navigation-procs.tcl =================================================================== RCS file: /usr/local/cvsroot/openacs-4/packages/acs-tcl/tcl/test/navigation-procs.tcl,v diff -u -N -r1.9 -r1.10 --- openacs-4/packages/acs-tcl/tcl/test/navigation-procs.tcl 4 Apr 2018 18:51:56 -0000 1.9 +++ openacs-4/packages/acs-tcl/tcl/test/navigation-procs.tcl 22 Jul 2018 09:58:43 -0000 1.10 @@ -140,7 +140,7 @@ # Test aa_log "ad_context_bar 1: '$context_bar'\nad_context_bar 2: '$context_barp'" - aa_true "Context_bar = $context_barp" [string equal $context_bar $context_barp] + aa_equals "Context_bar = $context_barp" $context_bar $context_barp #----------------------------------------------------------------------- # Case 2: node_id = testnode_2 (testnode2 is a testnode_1 children) @@ -159,7 +159,7 @@ append context_barp "$leave_node" set context_bar [ad_context_bar -node_id $idr_2 -separator $separator $leave_node] - aa_true "Context_bar = $context_barp" [string equal $context_bar $context_barp] + aa_equals "Context_bar = $context_barp" $context_bar $context_barp #---------------------------------------------------------------------------- # Case 3: from_node = testnode_1 and node_id = testnode_2 @@ -177,7 +177,7 @@ } append context_barp "$leave_node" set context_bar [ad_context_bar -from_node $idr_1 -node_id $idr_2 -separator $separator $leave_node] - aa_true "Context_bar = $context_barp" [string equal $context_bar $context_barp] + aa_equals "Context_bar = $context_barp" $context_bar $context_barp } } Index: openacs-4/packages/acs-tcl/tcl/test/test-membership-rel-procs.tcl =================================================================== RCS file: /usr/local/cvsroot/openacs-4/packages/acs-tcl/tcl/test/test-membership-rel-procs.tcl,v diff -u -N -r1.3 -r1.4 --- openacs-4/packages/acs-tcl/tcl/test/test-membership-rel-procs.tcl 7 Aug 2017 23:48:00 -0000 1.3 +++ openacs-4/packages/acs-tcl/tcl/test/test-membership-rel-procs.tcl 22 Jul 2018 09:58:43 -0000 1.4 @@ -26,8 +26,8 @@ acs_user::get -user_id $user_id -array user #Verifying if the state was changed - aa_true "Changed State to aprroved" \ - [string equal $user(member_state) "approved"] + aa_equals "Changed State to aprroved" \ + $user(member_state) "approved" #Try to change his state to banned @@ -36,8 +36,8 @@ acs_user::get -user_id $user_id -array user #Verifying if the state was changed - aa_true "Changed State to banned" \ - [string equal $user(member_state) "banned"] + aa_equals "Changed State to banned" \ + $user(member_state) "banned" #Try to change his state to rejected @@ -46,8 +46,8 @@ acs_user::get -user_id $user_id -array user #Verifying if the state was changed - aa_true "Changed State to rejected" \ - [string equal $user(member_state) "rejected"] + aa_equals "Changed State to rejected" \ + $user(member_state) "rejected" #Try to change his state to unapproved @@ -56,17 +56,17 @@ acs_user::get -user_id $user_id -array user #Verifying if the state was changed - aa_true "Changed State to unapproved" \ - [string equal $user(member_state) "needs approval"] + aa_equals "Changed State to unapproved" \ + $user(member_state) "needs approval" #Try to change his state to deleted aa_log "We change the state to deleted" membership_rel::delete -rel_id $rel_id acs_user::get -user_id $user_id -array user #Verifying if the state was changed - aa_true "Changed State to deleted" \ - [string equal $user(member_state) "deleted"] + aa_equals "Changed State to deleted" \ + $user(member_state) "deleted" } Index: openacs-4/packages/acs-tcl/tcl/test/test-set-cookie-procs.tcl =================================================================== RCS file: /usr/local/cvsroot/openacs-4/packages/acs-tcl/tcl/test/test-set-cookie-procs.tcl,v diff -u -N -r1.3 -r1.4 --- openacs-4/packages/acs-tcl/tcl/test/test-set-cookie-procs.tcl 7 Aug 2017 23:48:00 -0000 1.3 +++ openacs-4/packages/acs-tcl/tcl/test/test-set-cookie-procs.tcl 22 Jul 2018 09:58:43 -0000 1.4 @@ -34,7 +34,7 @@ #Get the cookie and we try if exist #------------------------------------------------------------------------- set cookie_info_p [ad_get_cookie -include_set_cookies t test_cookie_test_case "" ] - aa_true "Check if the cookie exist" [string equal $cookie_info_p $data] + aa_equals "Check if the cookie exist" $cookie_info_p $data #------------------------------------------------------------------------- #clearing the cookie Index: openacs-4/packages/lors-central/tcl/test/lors-central-procs.tcl =================================================================== RCS file: /usr/local/cvsroot/openacs-4/packages/lors-central/tcl/test/lors-central-procs.tcl,v diff -u -N -r1.4 -r1.5 --- openacs-4/packages/lors-central/tcl/test/lors-central-procs.tcl 15 May 2018 21:41:34 -0000 1.4 +++ openacs-4/packages/lors-central/tcl/test/lors-central-procs.tcl 22 Jul 2018 10:06:08 -0000 1.5 @@ -60,12 +60,12 @@ from ims_cp_manifest_class where community_id = :test_com_id and man_id = :test_man_id }] - if { [string equal $row_count "1"] } { + if { $row_count == 1 } { set success_p 1 } else { set success_p 0 } - aa_true "Creates only one row" [string equal "1" "$success_p"] + aa_equals "Creates only one row" 1 $success_p # Testing where there is already one association lors_central::add_relation -item_id $test_item_id -community_id $test_com_id_2 @@ -74,7 +74,7 @@ from ims_cp_manifest_class where man_id = :test_man_id }] - if { [string equal $row_count "2"] } { + if { $row_count == 2 } { set success_p 1 } else { set success_p 0 @@ -105,7 +105,7 @@ select max(revision_id) from cr_revisions where item_id = :test_item_id }] set version_number_1 [lors_central::get_version_num -revision_id $test_revision_id] - aa_equals "Min revision_id return version number equal to 1" $version_number_1 "1" + aa_equals "Min revision_id return version number equal to 1" $version_number_1 1 set version_number_2 [lors_central::get_version_num -revision_id $test_revision_id_2] aa_equals "Max revision_id return version number equal to revisions count" $version_number_2 $test_count @@ -195,7 +195,7 @@ }] aa_equals "Change to last version_number ($test_item_id_2) on different community" $check_item $test_item_id_2 - lors_central::change_one_lo_version -ver_num "0" -man_id $test_man_id \ + lors_central::change_one_lo_version -ver_num 0 -man_id $test_man_id \ -community_id $test_com_id_2 -ims_item_id $test_item_id_2 set check_item [db_string get_hide_p { @@ -368,7 +368,7 @@ where community_id = :com_id and man_id in ( select revision_id from cr_revisions where item_id = :item_id ) }] - aa_equals "Deleting everything from ims_cp_manifest_class" $test "0" + aa_equals "Deleting everything from ims_cp_manifest_class" $test 0 } } @@ -530,7 +530,7 @@ select sort_order from ims_cp_items where ims_item_id = :ims_item_id} -default 0] - aa_equals "Sort order proc set the right sort" $proc_sort "2" + aa_equals "Sort order proc set the right sort" $proc_sort 2 set proc_sort [lors_central::set_sort_order -ims_item_id $ims_item_id] set sort_p [db_string get_sort { @@ -562,15 +562,15 @@ where item_id = :item_id }] set proc_name [lors_central::check_item_name -parent_id $parent_id -name $name] - if { [string equal $name $proc_name] } { + if { $name eq $proc_name } { set success_p 0 } else { set success_p 1 } aa_true "Name exist, gives a new name" $success_p set name ${name}h set proc_name [lors_central::check_item_name -parent_id $parent_id -name $name] - if { [string equal $name $proc_name] } { + if { $name eq $proc_name } { set success_p 1 } else { set success_p 0 @@ -601,7 +601,7 @@ set proc_result [lors_central::check_privilege -user_id $user_id -item_id $item_id] aa_true "Check privilege proc when user has privilege" $proc_result set proc_result [lors_central::check_privilege -user_id 0 -item_id $item_id] - aa_true "Check privilege proc when user has no privilege" [string equal $proc_result 0] + aa_equals "Check privilege proc when user has no privilege" $proc_result 0 } } @@ -630,7 +630,7 @@ }] set item_id [lors_central::get_item_id -revision_id $rev_id] set proc_result [lors_central::item_editable_info -item_id $item_id] - aa_true "Sending one non editable item" [string equal [lindex $proc_result 3] 0] + aa_equals "Sending one non editable item" [lindex $proc_result 3] 0 } } @@ -660,7 +660,7 @@ }] set item_id [lors_central::get_item_id -revision_id $rev_id] set proc_result [lors_central::item_editable_p -item_id $item_id] - aa_true "Sending one non editable item" [string equal $proc_result 0] + aa_equals "Sending one non editable item" $proc_result 0 } } @@ -676,27 +676,27 @@ select 1 from cr_folders where label = 'LORSM Root Folder'} -default 0] - aa_true "Folders created" $success_p + aa_true "Folders created" $success_p set success_p [db_string check_folder { select 1 from cr_folders where label = 'LORSM Manifest Folder'} -default 0] - aa_true "Folders created" $success_p + aa_true "Folders created" $success_p set success_p [db_string check_folder { select 1 from cr_folders where label = 'LORSM Organizations Folder'} -default 0] - aa_true "Folders created" $success_p + aa_true "Folders created" $success_p set success_p [db_string check_folder { select 1 from cr_folders where label = 'LORSM Items Folder'} -default 0] - aa_true "Folders created" $success_p + aa_true "Folders created" $success_p set success_p [db_string check_folder { select 1 from cr_folders where label = 'LORSM Resources Folder'} -default 0] - aa_true "Folders created" $success_p + aa_true "Folders created" $success_p } } @@ -734,6 +734,6 @@ set success_p [db_string get_check { select 1 from ims_cp_manifests where man_id = :new_man_id } -default 0] - aa_true "New man_id created" $success_p + aa_true "New man_id created" $success_p } } Index: openacs-4/packages/calendar/tcl/test/cal-item-procs.tcl =================================================================== RCS file: /usr/local/cvsroot/openacs-4/packages/calendar/tcl/test/cal-item-procs.tcl,v diff -u -N -r1.6 -r1.7 --- openacs-4/packages/calendar/tcl/test/cal-item-procs.tcl 30 Apr 2018 13:54:38 -0000 1.6 +++ openacs-4/packages/calendar/tcl/test/cal-item-procs.tcl 22 Jul 2018 10:08:01 -0000 1.7 @@ -28,8 +28,8 @@ calendar::item::get \ -cal_item_id $cal_item_id -array cal_item - aa_true "Name is correct" [string equal $ci_name $cal_item(name)] - aa_true "Description is correct" [string equal $ci_description $cal_item(description)] + aa_equals "Name is correct" $ci_name $cal_item(name) + aa_equals "Description is correct" $ci_description $cal_item(description) # edit the time of the event set recurrence_id \ [calendar::item::add_recurrence \