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.12 -r1.13 --- openacs-4/packages/acs-tcl/tcl/test/navigation-procs.tcl 25 Jul 2018 13:42:48 -0000 1.12 +++ openacs-4/packages/acs-tcl/tcl/test/navigation-procs.tcl 10 Aug 2018 09:58:39 -0000 1.13 @@ -1,5 +1,5 @@ ad_library { - + Tests that deal with the context bar creation. @author Juan Pablo Amaya jpamaya@unicauca.edu.co @@ -12,37 +12,37 @@ Procedure for the context_bar_multirow test filter } { aa_run_with_teardown -test_code { - set testnode_1 [list "/navigation_test_node1/" "navigation_test_node1"] - set testnode_2 [list "[lindex $testnode_1 0]navigation_test_node2/" "navigation_test_node2"] + set testnode_1 [list "/navigation_test_node1/" "navigation_test_node1"] + set testnode_2 [list "[lindex $testnode_1 0]navigation_test_node2/" "navigation_test_node2"] - # Create hierarchy from the random created nodes - db_1row query { - select MIN(node_id) as first_node from site_nodes - } - set idp $first_node - set idr_1 [site_node::new -name [lindex $testnode_1 1] -parent_id $idp] - set idr_2 [site_node::new -name [lindex $testnode_2 1] -parent_id $idr_1] - - set from_node $first_node - set node_id $idr_2 - set context "last" - - set page [ad_parse_template -params [list [list from_node $from_node] [list node_id $node_id] [list context $context]] "/packages/acs-tcl/tcl/test/multirow-test"] - - site_node::delete -node_id $idr_2 - site_node::delete -node_id $idr_1 - + # Create hierarchy from the random created nodes + db_1row query { + select MIN(node_id) as first_node from site_nodes + } + set idp $first_node + set idr_1 [site_node::new -name [lindex $testnode_1 1] -parent_id $idp] + set idr_2 [site_node::new -name [lindex $testnode_2 1] -parent_id $idr_1] + + set from_node $first_node + set node_id $idr_2 + set context "last" + + set page [ad_parse_template -params [list [list from_node $from_node] [list node_id $node_id] [list context $context]] "/packages/acs-tcl/tcl/test/multirow-test"] + + site_node::delete -node_id $idr_2 + site_node::delete -node_id $idr_1 + } -teardown_code { site_node::delete -node_id $idr_2 - site_node::delete -node_id $idr_1 - + site_node::delete -node_id $idr_1 + } ns_return 200 text/html $page - + return filter_return } - + aa_register_case \ -cats {api smoke} \ -procs { @@ -64,7 +64,7 @@ } aa_register_case -cats { - api + api smoke } -procs { ad_context_bar @@ -76,26 +76,26 @@ Test if returns a well formed context_bar in HTML format from a site node. } { - + aa_run_with_teardown -rollback -test_code { - # Setup nodes from the context bar, create two random nodes to include - set separator "-" - set random1 [ad_generate_random_string] - set testnode_1 [list "/$random1/" "ACS Automated Testing"] + # Setup nodes from the context bar, create two random nodes to include + set separator "-" + set random1 [ad_generate_random_string] + set testnode_1 [list "/$random1/" "ACS Automated Testing"] - set random2 [ad_generate_random_string] - set testnode_2 [list "[lindex $testnode_1 0]$random2/" "ACS Automated Testing"] + set random2 [ad_generate_random_string] + set testnode_2 [list "[lindex $testnode_1 0]$random2/" "ACS Automated Testing"] - set leave_node "ref_final" - set root_node [list "/" \#acs-kernel.Main_Site\#] - if { [string match "admin/*" [ad_conn extra_url]] } { - set admin_node [list "[ad_conn package_url]admin/" "Administration"] - } else { - set admin_node "" - } - - # Create hierarchy from the random created nodes + set leave_node "ref_final" + set root_node [list "/" \#acs-kernel.Main_Site\#] + if { [string match "admin/*" [ad_conn extra_url]] } { + set admin_node [list "[ad_conn package_url]admin/" "Administration"] + } else { + set admin_node "" + } + + # Create hierarchy from the random created nodes db_1row query { select min(node_id) as first_node from site_nodes } @@ -104,9 +104,9 @@ set idr_2 [site_node::new -name $random2 -parent_id $idr_1] site_node::mount -node_id $idr_1 -object_id [ad_conn package_id] site_node::mount -node_id $idr_2 -object_id [ad_conn package_id] - aa_log "Created two test sites nodes: testnode_1 = [lindex $testnode_1 1],\n\ - testnode_2 = [lindex $testnode_2 1]n\ - testnode_2 is a child of testnode_1" + aa_log "Created two test sites nodes: testnode_1 = [lindex $testnode_1 1],\n\ + testnode_2 = [lindex $testnode_2 1]n\ + testnode_2 is a child of testnode_1" array set node [site_node::get -node_id $idp] array set node1 [site_node::get -node_id $idr_1] @@ -117,29 +117,29 @@ "\nnode1 $idr_1 parent $node1(parent_id) url $node1(url) object_id $node1(object_id)" \ "\nnode2 $idr_2 parent $node2(parent_id) url $node2(url) object_id $node2(object_id)" aa_log $msg - - #----------------------------------------------------------------------- - # Case 1: node_id = testnode_1 - #----------------------------------------------------------------------- - aa_log "Case 1: node_id = testnode_1 <$testnode_1>" - set bar_components [list $root_node $testnode_1 $admin_node] + + #----------------------------------------------------------------------- + # Case 1: node_id = testnode_1 + #----------------------------------------------------------------------- + aa_log "Case 1: node_id = testnode_1 <$testnode_1>" + set bar_components [list $root_node $testnode_1 $admin_node] #aa_log "bar_components $bar_components" set context_barp "" - foreach value $bar_components { + foreach value $bar_components { append context_barp \ [subst {[lindex $value 1] $separator }] - } - append context_barp "$leave_node" - set context_bar [ad_context_bar -node_id $idr_1 -separator $separator $leave_node] + } + append context_barp "$leave_node" + set context_bar [ad_context_bar -node_id $idr_1 -separator $separator $leave_node] - # Test + # Test aa_log "ad_context_bar 1: '$context_bar'\nad_context_bar 2: '$context_barp'" - aa_equals "Context_bar = $context_barp" $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) - #----------------------------------------------------------------------- - aa_log "Case 2: node_id = testnode_2 (testnode2 is a testnode_1 children)" + #----------------------------------------------------------------------- + aa_log "Case 2: node_id = testnode_2 (testnode2 is a testnode_1 children)" set bar_components [list $root_node $testnode_1 $testnode_2 $admin_node] set context_barp "" foreach value $bar_components { @@ -151,25 +151,24 @@ aa_equals "Context_bar = $context_barp" $context_bar $context_barp - #---------------------------------------------------------------------------- + #---------------------------------------------------------------------------- # Case 3: from_node = testnode_1 and node_id = testnode_2 - #---------------------------------------------------------------------------- - aa_log "Case 3: from_node = testnode_1 and node_id = testnode_2" + #---------------------------------------------------------------------------- + aa_log "Case 3: from_node = testnode_1 and node_id = testnode_2" set bar_components [list $testnode_1 $testnode_2 $admin_node] set context_barp "" foreach value $bar_components { append context_barp \ [subst {[lindex $value 1] $separator }] } append context_barp "$leave_node" - set context_bar [ad_context_bar -from_node $idr_1 -node_id $idr_2 -separator $separator $leave_node] - aa_equals "Context_bar = $context_barp" $context_bar $context_barp + set context_bar [ad_context_bar -from_node $idr_1 -node_id $idr_2 -separator $separator $leave_node] + aa_equals "Context_bar = $context_barp" $context_bar $context_barp } } aa_register_case \ -cats {api smoke web} \ - -libraries tclwebtest \ -procs { ad_context_bar_multirow } \ @@ -194,12 +193,17 @@ append context_barp [lindex $value 1] append context_barp "" } - ad_register_filter postauth GET /test.testf navigation::test::context_bar_multirow_filter - set server [twt::server_url] - ::twt::do_request "$server/test.testf" - aa_log "Filter page created: [tclwebtest::response url]\ shows the multirow" - set response_body [::tclwebtest::response body] - aa_equals "Context bar $context_barp" $response_body $context_barp + ns_register_proc GET /test.testf { + navigation::test::context_bar_multirow_filter + } + set d [acs::test::http /test.testf] + acs::test::reply_has_status_code $d 200 + ns_unregister_op GET /test.testf + + set response_body [dict get $d body] + ns_log notice "CONTEXT BARP $context_barp" + ns_log notice "RESPONS BODY $response_body" + aa_equals "Context bar" [ns_quotehtml $response_body] [ns_quotehtml $context_barp] } # Local variables: