Build failed in Jenkins: cyrus-imapd-master #507

[Greg Banks]

On Tue, Apr 10, 2012, at 05:09 PM, Jenkins wrote:
> See <http://ci.cyrusimap.org/job/cyrus-imapd-master/507/>
> 

> Test failures and errors summary
> ================================
> 
> Cassandane::Cyrus::Lsub.lsub_toplevel
>     http://ci.cyrusimap.org/job/cyrus-imapd-master/507//testReport/%28root%29/Cassandane__Cyrus__Lsub/test_lsub_toplevel/
> 


The Lsub tests were using this syntax to create a user

    # sub folders of another user - one is subscribable
    $self->{instance}->create_user("other",
				   subdirs => [ qw(sub sub.folder) ]);

A recent change to support running tests with unixhierarchysep=on made
the checking in create_user() stricter and it now rejects 'sub.folder'
because it contains the default hierarchy separator.  Creating a 2-level
folder is a perfectly reasonable thing to want to do here, so I fixed
this by adding a new syntax to support that without the ambiguity:

    # sub folders of another user - one is subscribable
    $self->{instance}->create_user("other",
				   subdirs => [ 'sub', ['sub', 'folder']
				   ]);

Fixed in
http://git.cyrusimap.org/cassandane/commit/?id=cb4a4480ec1a90e40e08482130432062469e558d

-- 
Greg.