Re: [siesta-dev] Plugins/Subscribe.pm

[prev] [thread] [next] [lurker] [Date index for 2002/10/10]

From: Richard Clamp
Subject: Re: [siesta-dev] Plugins/Subscribe.pm
Date: 12:27 on 10 Oct 2002
On Thursday, Oct 10, 2002, at 03:52 Europe/London, Gavin Estey wrote:

> I think this looks right, I've been poking about a bit :)

Thanks, applied.

> I couldn't work out how to fix the mock'd list object into getting the
> test to work. I've not used Test::MockObject before I'm afraid.

Yeah, that test is a little dense.

--- 10plugin_subscribe.t.~1.7.~ Tue Oct  1 11:06:56 2002
+++ 10plugin_subscribe.t        Thu Oct 10 12:26:37 2002
@@ -19,7 +19,7 @@ my $list =
    Test::MockObject->new->mock( id => sub { 'escapees' } )
    ->mock( owner => sub { 'houdini@elsewhere' } )
    ->mock( add_member => sub { $_[1] eq 'suzanne@lab' || $_[1] eq 
'foo@bar' } )
-  ->mock( members => sub { return ($juser) } );
+  ->mock( is_member => sub { $_[1] eq $juser->id  } );

  my $mail =

> One thing I'd like to add to the TODO is seperating the email templates
> away from the modules so they can be easily edited. Maybe have
> $list->get_template('already-on') etc?
>

I'll add that too.

-- 
Richard Clamp <richardc@xxxxxxxxx.xxx>


Generated at 13:56 on 01 Jul 2004 by mariachi 0.52