svn commit: r1418111 - /subversion/branches/in-repo-authz/BRANCH-README

[br...@apache.org]

Author: breser
Date: Thu Dec  6 22:15:34 2012
New Revision: 1418111

URL: http://svn.apache.org/viewvc?rev=1418111&view=rev
Log:
On the in-repo-authz branch: Update the BRANCH-README.

I've implemented C API level tests, I think the testing of the svnserve/dav 
code requires major changes to our test infrastructure that isn't really in
scope for this change so I'm hesistant to start doing it on this branch.

Also the relative path test doesn't really matter since we technically only 
ever use absolute paths, upstream callers to svn_repos_authz_read2() may 
convert relative paths to absolute paths though.  There's no doesn't exist
test for OS file system paths, but that didn't exist in the existing tests.

* BRANCH-README: Remove testing and refactor entries.

Modified:
    subversion/branches/in-repo-authz/BRANCH-README

Modified: subversion/branches/in-repo-authz/BRANCH-README
URL: http://svn.apache.org/viewvc/subversion/branches/in-repo-authz/BRANCH-README?rev=1418111&r1=1418110&r2=1418111&view=diff
==============================================================================
--- subversion/branches/in-repo-authz/BRANCH-README (original)
+++ subversion/branches/in-repo-authz/BRANCH-README Thu Dec  6 22:15:34 2012
@@ -8,27 +8,6 @@ TODO
 
 * svnserve - I have partial support written, but need to deal with the whole
     --config-file issue.
-* tests - Need to finish writing the tests for svn_repos_authz_read2().
-    It's not really feasable to handle DAV testing with the way our test
-    suite is currently written.  We have two authz directives and 4 different
-    styles of authz paths.  So we'd need 8 different Location's to be setup,
-    which seems absurd to me.  One alternative is to migrate to always using
-    something akin to davautocheck.
 * errors - DAV error messages kinda suck since it guesses what it thinks is
     the best error message at current.  When it should display the whole chain.
-* refactor svn_repos_authz_read* functions to share a common implementation. 
 * documentation - as in the user level docs.
-
-TESTS
-
-Following tests should be developed (+ means we should look for success,
-- means we should look for failure).
-
-+ absolute path on OS file system.
-+ relative path on OS file system.
-+ file:// url to repo path.
-+ relative path in repo (^/)
-- http:// url
-- svn:// url
-- OS path that does not exist
-- repo path that does not exist