• last updated 7 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Creating a user without a username is possible if the authority is local, include the test authority among those for which we generate a username, as it is also local as in "hosted locally"

Fixes acs-authentication.auth_use_email_for_login_p automated test

reduce public footprint

  1. … 1 more file in changeset.
fixed bug: exception in test case auth_use_email_for_login_p caused test to terminate

By adding an "ad_try", we can see that the intention to create

a user with no username failed. Probably this was working at some time,

not sure, if it still should.

This is could be a general problem with "aa_run_with_teardown" to

terminate too early in such cases.

    • -10
    • +18
    ./tcl/test/acs-authentication-procs.tcl
- Rework acs-api-browser.callgraph__bad_calls automated test so that acs-service-contract calling contract implementation is considered OK

- Reinstate authentication service contracts as private

- Move test authority implementation to acs-authentication to remove intra-package dependency

acs-api-browser.callgraph__bad_call still fails because proc acs-tcl.install::xml::action::create-user calls private acs-authentication.auth::local::registration::Register... to be continued

  1. … 2 more files in changeset.
file test-authority-procs.tcl was initially added on branch oacs-5-10.

    • -0
    • +0
    ./tcl/test-authority-procs.tcl
Make api public, complies with acs-api-browser.graph__bad_calls automated test

  1. … 1 more file in changeset.
Make service contract implementation public

Complies with acs-api-browser.callgraph__bad_calls

improve listing of test coverage

    • -1
    • +27
    ./tcl/test/acs-authentication-procs.tcl
  1. … 16 more files in changeset.
make listing of tested procs more complete

    • -12
    • +30
    ./tcl/test/acs-authentication-procs.tcl
  1. … 17 more files in changeset.
whitespace changes

set default value only in case of failure

    • -6
    • +7
    ./tcl/authentication-procs-naviserver.tcl
Care about the parameter cache after test case

    • -1
    • +3
    ./tcl/test/acs-authentication-procs.tcl
Set a bunch of test cases to 'production_safe'

    • -7
    • +7
    ./tcl/test/acs-authentication-procs.tcl
Merge 'auth_authority_api' and 'auth_authority_delete' test cases, covering almost the same api

    • -195
    • +122
    ./tcl/test/acs-authentication-procs.tcl
Fix typo

    • -2
    • +2
    ./tcl/test/acs-authentication-procs.tcl
don't rely on uppercase header field names

  1. … 10 more files in changeset.
prefer db_0or1row +"where exists" over db_string + "select case"

  1. … 3 more files in changeset.
Improve sql portability: oracle cannot select a boolean directly using exists

Reduce divergencies between oracle and postgres codebase

    • -9
    • +0
    ./tcl/test/acs-authentication-procs-oracle.xql
    • -10
    • +0
    ./tcl/test/acs-authentication-procs-postgresql.xql
    • -2
    • +6
    ./tcl/test/acs-authentication-procs.tcl
prefer "ns_conn isconnected" over "ad_conn isconnected", since the latter might not notice changes in the connect state

  1. … 18 more files in changeset.
mprove spelling: move closer to the linux documentation recommendations

    • -2
    • +2
    ./tcl/test/acs-authentication-procs.tcl
  1. … 15 more files in changeset.
Fallback to + #

Add missing authority_id on the 'auth_authenticate' testcase

    • -1
    • +2
    ./tcl/test/acs-authentication-procs.tcl
Replace deprecated idiom

break overlong lines

Replace deprecated idiom

Improve documentation

Deprecate duplicated and untested proc auth::driver::GetParameters

Fix reported api, extend test

    • -3
    • +57
    ./tcl/test/acs-authentication-procs.tcl
Extend test to cover more api

    • -4
    • +37
    ./tcl/test/acs-authentication-procs.tcl