• last updated 21 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Rework of this tests

- use dict idioms instead of array, so we don't have to unset all the time

- use new user api when appropriate

- fix test when we try to reuse the username of somebody who was previously banned.

This is no longer possible and is now reflected in the test

Revert massive replacement of empty list creation sentences. The use of '[list]' instead of '{}' adds semantics that could be used for performance improvements in the future, such as using a different internal representation. There is already work in this direction, avoiding the generation of the string representation during comparison of empty strings (huge thanks to Stefan Sobernig for the pointer: https://core.tcl.tk/tcl/info/44527c632ed609c2).

  1. … 475 more files in changeset.
Prefer '{}' to '[list]' when creating empty lists

  1. … 71 more files in changeset.
Fix typos

  1. … 62 more files in changeset.
merged changes from the oacs-5-9 branch and resolved conflicts

  1. … 7833 more files in changeset.
- fix test case

- add editor hints to keep spaces/tabs in the furture more consistent

  1. … 753 more files in changeset.

ad_proc reform:

- use nsf::proc for defining ad_procs, when nsf::proc is available

- nsf::proc is part of the environment of XOTcl2 and provides

an efficient, C-implemented argument parser that can replace

the scripted *__arg_parser of OpenACS.

- The benefits are performance and reduced memory consumption.

* Simple functions (where the calling overhead was a large

part of the execution times can become several times faster

(e.g. db_name is 3x faster, cookie handling is 2x faster),

for functions with much db-activities the percentage will

be less.

* The memory consumption (RSS) of a site like OpenACS.org

is reduced by about 15%.

NEW (with nsf::proc)

::ad_get_cookie x: 10.93 microseconds per iteration

::ad_get_cookie ad_session_id: 21.95 microseconds per iteration

::dt_systime: 24.89 microseconds per iteration

::db_name: 8.70 microseconds per iteration

::db_string dbqd.null.g "select version() from dual": 316.66 microseconds per iteration

::ad_convert_to_html "hello world": 102.29 microseconds per iteration

OLD (without nsf::proc)

::ad_get_cookie x: 19.42 microseconds per iteration

::ad_get_cookie ad_session_id: 30.10 microseconds per iteration

::dt_systime: 27.55 microseconds per iteration

::db_name: 32.28 microseconds per iteration

::db_string dbqd.null.g "select version() from dual": 366.69 microseconds per iteration

::ad_convert_to_html "hello world": 160.51 microseconds per iteration

======================================================================

proc mtime cmd {set c 5000

set t [time [list time $cmd $c]];

regexp {^(-?[0-9]+) +} $t _ mS1;

set ms [expr {$mS1*1.0/$c}];

return "[format %6.2f $ms] microseconds per iteration"

}

set tests {

{::ad_get_cookie x} {::ad_get_cookie ad_session_id} {::dt_systime}

{::db_name} {::db_string dbqd.null.g "select version() from dual"}

{::ad_convert_to_html "hello world"}

}

foreach t $tests {append ::_ $t ": " [mtime $t] \n}

set ::_

======================================================================

  1. … 8 more files in changeset.
Merging back to HEAD all changes that happened in branch oacs-5-8 between tags: vg-merge-oacs-5-8-from-20141027 and vg-merge-oacs-5-8-from-20150427

  1. … 520 more files in changeset.
Merging back to HEAD branch oacs-5-8 (using tag vg-merge-oacs-5-8-from-20141027).

  1. … 2546 more files in changeset.
- simplify code

  1. … 3 more files in changeset.
- use $::acs::rootdir rather than [acs_root_dir] or [get_server_root]

  1. … 13 more files in changeset.
- brace expressions to allow for tcl compilation

- use "in" or not in (ni) operator instead of lsearch (for readability and performance)

  1. … 10 more files in changeset.
- use "in" or not in (ni) operator instead of lsearch (for readability and performance)

  1. … 2 more files in changeset.
- make exists_and_not_null explicit

  1. … 5 more files in changeset.
stub acs_mail_lite::send since ns_sendmail is not always the ultimate transport. (the second one)

stub acs_mail_lite::send since ns_sendmail is not always the ultimate transport.

make test case a bit more informative about failure mode

Running these test twice would lead to errors, due to test users not being

deleted. The tear-down stuff knows how to rollback the database, but caching

is a bit beyond its automated capability at the moment.

Normalizing string comparison operations into tcl 8.4 operations, enabling byte-code compilation for expressions and many conditionals

  1. … 366 more files in changeset.
Someone changed the local auth implementation to handle the "RetrievePassword"

operation (by e-mailing a note to the user which links to a reset password

page). This is cool. What's not cool is to change the behavior without

running tests on the package and correcting the one that looked for the

local auth implementation to return "not_supported" ...

Should've been more thorough. Turns out every single test case was registered with backslash-space on the first line, causing an error

Fixed broken case: space after backslash

Added categories to the automated test(s).

  1. … 2 more files in changeset.
Added local file system auth_retrieve_doc driver

  1. … 2 more files in changeset.
Fixed password email change test case - turned out to be interaction between two test cases and the cache

Fixed batch sync test cases

  1. … 1 more file in changeset.
Fixed broken test cases

  1. … 2 more files in changeset.
Fixed parameters test case to expose problem with unique constraint/primary key on auth_driver_params on Oracle, and to do a better job of testing the parameters framework