Re: [Exist-development] [Exist-commits] SF.net SVN: exist:[11485] trunk/eXist/test/src/org/exist/validation

[Dannes W. <da...@ex...>]

Hi,

On Sat, Mar 20, 2010 at 7:24 AM,  <sha...@us...> wrote:
> [ignore] use SecurityManager.GUEST instead of UserImpl.DEFAULT

>    trunk/eXist/test/src/org/exist/validation/DatabaseInsertResources_NoValidation_Test.java
>    trunk/eXist/test/src/org/exist/validation/DatabaseInsertResources_WithValidation_Test.java
> -            broker = pool.get(UserImpl.DEFAULT);
> +            broker = pool.get(SecurityManager.GUEST);

I was a bit surprised that this little change made the junit test
fail. Do we have any idea why we did not see this before? Did we run
the test suite locally?

Anyway, what are the risks here? At this moment we found this issue in
the not related validation tests by accident... did we check whether
there are more locations where this can happen?

Couldn't these UserImpl.DEFAULT and SecurityManager.GUEST be made
identical, making it a pure replacement, so there is no risk?

regards

Dannes

-- 
eXist-db Native XML Database - http://exist-db.org
Join us on linked-in: http://www.linkedin.com/groups?gid=35624