diff options
author | David Phillips <david@yeah.nah.nz> | 2020-04-04 19:34:10 +1300 |
---|---|---|
committer | David Phillips <david@yeah.nah.nz> | 2020-04-04 19:36:42 +1300 |
commit | cba83d93d8b47cc044e8d093049d23aba5d8b6ed (patch) | |
tree | 042fe5146cfef32a10c132a48292db8f29be7539 | |
parent | cdae7275d29c9c370ec7e714dd71af2a1076c0d0 (diff) | |
download | idalius-cba83d93d8b47cc044e8d093049d23aba5d8b6ed.tar.xz |
Add test plans for Thanks, Vote
-rw-r--r-- | Mock/CommandRegistry.pm | 3 | ||||
-rwxr-xr-x | test.pl | 10 | ||||
-rw-r--r-- | test/TODO.md | 4 | ||||
-rwxr-xr-x | test/test_thanks.t | 35 | ||||
-rwxr-xr-x | test/test_vote.t | 273 |
5 files changed, 316 insertions, 9 deletions
diff --git a/Mock/CommandRegistry.pm b/Mock/CommandRegistry.pm index 72c84cf..6531fd7 100644 --- a/Mock/CommandRegistry.pm +++ b/Mock/CommandRegistry.pm @@ -28,6 +28,7 @@ sub run_owned { # XXX plugins need to eventually use the ctx hash directly. Much more # readable than remembering the long chain of magic positional args each # time you write a new cmd plugin + my @cmd_args = split /\s+/, $ctx{rest} if $ctx{rest}; my @args = ( $ctx{irc}, $ctx{logger}, @@ -36,7 +37,7 @@ sub run_owned { $ctx{ided}, $ctx{rest}, $ctx{no_reenter}, - $ctx{args} + @cmd_args ); $self->{_commands}->{$owner}->{$command}->(@args); @@ -2,10 +2,8 @@ use Test::Harness; -my @t = ( - "test/test_autojoin.t", - "test/test_echo.t", - "test/test_ping.t", -); +my @tests = map {"test/test_$_.t"} qw/ + autojoin echo ping thanks vote +/; -runtests(@t); +runtests(@tests); diff --git a/test/TODO.md b/test/TODO.md index 4c4f14a..b8d66bf 100644 --- a/test/TODO.md +++ b/test/TODO.md @@ -21,12 +21,10 @@ * Quote_Grab.pm * Random.pm * Source.pm -* Thanks.pm * Timezone.pm * Titillate.pm * Topic.pm * URL_Title.pm -* Vote.pm ### The following have sufficient tests for now: @@ -34,3 +32,5 @@ * Autojoin.pm * Echo.pm * Ping.pm +* Thanks.pm +* Vote.pm diff --git a/test/test_thanks.t b/test/test_thanks.t new file mode 100755 index 0000000..680c235 --- /dev/null +++ b/test/test_thanks.t @@ -0,0 +1,35 @@ +#!/usr/bin/env perl + +use strict; +use warnings; +use Test::Simple tests => 8; +use Mock::CommandRegistry; + +use Plugin::Thanks; + +my @expected_commands = qw/thanks thanks. thanks! thanks?/; + +my $cr = new Mock::CommandRegistry; +my $register = sub { $cr->register(@_); }; +Plugin::Thanks->configure($register, undef, undef, undef); +for my $cmd (@expected_commands) { + my $is_registered = $cr->is_registered_to_owner("Plugin::Thanks", $cmd); + ok($is_registered, "registered command $cmd"); +} + +my $thanker = "somelad"; +my %ctx = ( + irc => undef, + logger => undef, + who => "$thanker!who\@example.com", + where => undef, + ided => undef, + rest => undef, + no_reenter => undef, + args => undef +); +my $expected = "$thanker: pong"; +for my $cmd (@expected_commands) { + my $response = $cr->run_owned("Plugin::Thanks", $cmd, %ctx); + ok($response =~ m/^\Q$thanker\E: /, "response addressed to thanker"); +} diff --git a/test/test_vote.t b/test/test_vote.t new file mode 100755 index 0000000..43537c5 --- /dev/null +++ b/test/test_vote.t @@ -0,0 +1,273 @@ +#!/usr/bin/env perl + +use strict; +use warnings; +use Test::More tests => 42; +use Mock::CommandRegistry; + +use Plugin::Vote; + + +my %ctx_a = ( + irc => undef, + logger => undef, + who => 'user!who@example.com', + where => "#channel_a", + ided => undef, + rest => undef, + no_reenter => undef, +); + +my %ctx_b = %ctx_a; +$ctx_b{where} = "#channel_b"; + +my $cr = new Mock::CommandRegistry; +my $register = sub { $cr->register(@_); }; +Plugin::Vote->configure($register, undef, undef, undef); + +# Test all expected commands registered +{ + my @expected_commands = ("vote on", "vote yes", "vote no", "vote end"); + for my $cmd (@expected_commands) { + my $is_registered = $cr->is_registered_to_owner("Plugin::Vote", $cmd); + ok($is_registered, "registered command $cmd"); + } +} + + +# Test ending a vote when none is in progress produces error message +{ + my %ctx = %ctx_a; + my $r = $cr->run_owned("Plugin::Vote", "vote end", %ctx); + ok($r =~ m/no vote.*progress/i, "no vote error message"); +} + + +sub ok_vote_match { + my ($result, $expect_ayes, $expect_noes) = @_; + + if ($result =~ m/The votes are in.*Ayes: (\d+).*Noes: (\d+)/) { + is($1, $expect_ayes, "ayes match"); + is($2, $expect_noes, "ayes match"); + } else { + die("'$result' can't be taken apart"); + } +} + +sub vote_started { + my ($result) = @_; + $result =~ m/^Call to vote.*/; +} + +# - Start a vote, end a vote, 0,0 +{ + my %ctx = %ctx_a; + $ctx{rest} = "topic1"; + my $r = $cr->run_owned("Plugin::Vote", "vote on", %ctx); + note($r); + ok(vote_started($r), "vote started"); + + $ctx{rest} = ""; + $r = $cr->run_owned("Plugin::Vote", "vote end", %ctx); + note($r); + ok_vote_match($r, 0, 0); +} + +# - Start a vote, start another, error message, end a vote, 0,0 +{ + my %ctx = %ctx_a; + $ctx{rest} = "topic1"; + my $r = $cr->run_owned("Plugin::Vote", "vote on", %ctx); + note($r); + ok(vote_started($r), "first vote started"); + + $ctx{rest} = "topic2"; + $r = $cr->run_owned("Plugin::Vote", "vote on", %ctx); + note($r); + ok(!vote_started($r), "second vote not started"); + + $ctx{rest} = ""; + $r = $cr->run_owned("Plugin::Vote", "vote end", %ctx); + note($r); + ok_vote_match($r, 0, 0); +} + +# - Start a vote, vote yes, end a vote, 1,0 +{ + my %ctx = %ctx_a; + $ctx{rest} = "topic1"; + my $r = $cr->run_owned("Plugin::Vote", "vote on", %ctx); + note($r); + ok(vote_started($r), "vote started"); + + $ctx{rest} = ""; + $r = $cr->run_owned("Plugin::Vote", "vote yes", %ctx); + note($r); + + $r = $cr->run_owned("Plugin::Vote", "vote end", %ctx); + note($r); + ok_vote_match($r, 1, 0); +} + +# - Start a vote, vote no, end a vote, 0,1 +{ + my %ctx = %ctx_a; + $ctx{rest} = "topic1"; + my $r = $cr->run_owned("Plugin::Vote", "vote on", %ctx); + note($r); + ok(vote_started($r), "vote started"); + + $ctx{rest} = ""; + $r = $cr->run_owned("Plugin::Vote", "vote no", %ctx); + note($r); + + $r = $cr->run_owned("Plugin::Vote", "vote end", %ctx); + note($r); + ok_vote_match($r, 0, 1); +} + +# - Start a vote, vote yes twice, error message, end the vote, 1,0 +{ + my %ctx = %ctx_a; + $ctx{rest} = "topic1"; + my $r = $cr->run_owned("Plugin::Vote", "vote on", %ctx); + note($r); + ok(vote_started($r), "vote started"); + + $ctx{rest} = ""; + $r = $cr->run_owned("Plugin::Vote", "vote yes", %ctx); + note($r); + + $r = $cr->run_owned("Plugin::Vote", "vote yes", %ctx); + note($r); + ok($r =~ m/already voted/, "user told about double vote"); + + $r = $cr->run_owned("Plugin::Vote", "vote end", %ctx); + note($r); + ok_vote_match($r, 1, 0); +} + +# - Start a vote, vote no twice, error message, end the vote, 0,1 +{ + my %ctx = %ctx_a; + $ctx{rest} = "topic1"; + my $r = $cr->run_owned("Plugin::Vote", "vote on", %ctx); + note($r); + ok(vote_started($r), "vote started"); + + $ctx{rest} = ""; + $r = $cr->run_owned("Plugin::Vote", "vote no", %ctx); + note($r); + + $r = $cr->run_owned("Plugin::Vote", "vote no", %ctx); + note($r); + ok($r =~ m/already voted/, "user told about double vote"); + + $r = $cr->run_owned("Plugin::Vote", "vote end", %ctx); + note($r); + ok_vote_match($r, 0, 1); +} + +# - Start a vote, vote no then yes, error message, end the vote, 0,1 +{ + my %ctx = %ctx_a; + $ctx{rest} = "topic1"; + my $r = $cr->run_owned("Plugin::Vote", "vote on", %ctx); + note($r); + ok(vote_started($r), "vote started"); + + $ctx{rest} = ""; + $r = $cr->run_owned("Plugin::Vote", "vote no", %ctx); + note($r); + + $r = $cr->run_owned("Plugin::Vote", "vote yes", %ctx); + note($r); + ok($r =~ m/already voted/, "user told about double vote"); + + $r = $cr->run_owned("Plugin::Vote", "vote end", %ctx); + note($r); + ok_vote_match($r, 0, 1); +} + +# - Start a vote, two people vote yes, one no, end the vote, 2,1 +{ + my %ctx = %ctx_a; + $ctx{rest} = "topic1"; + my $r = $cr->run_owned("Plugin::Vote", "vote on", %ctx); + note($r); + ok(vote_started($r), "vote started"); + + $ctx{rest} = ""; + $r = $cr->run_owned("Plugin::Vote", "vote yes", %ctx); + note($r); + + my $orig_who = $ctx{who}; + $ctx{who} = "a$orig_who"; + $r = $cr->run_owned("Plugin::Vote", "vote yes", %ctx); + note($r); + + $ctx{who} = "b$orig_who"; + $r = $cr->run_owned("Plugin::Vote", "vote no", %ctx); + note($r); + + $ctx{who} = $orig_who; + $r = $cr->run_owned("Plugin::Vote", "vote end", %ctx); + note($r); + ok_vote_match($r, 2, 1); +} + +# Test cases cross-chan +# - Start a vote #a, vote on #b, error message, end a vote, 0,0 +{ + my %ctx = %ctx_a; + + # Start a vote on channel A + $ctx{rest} = "topic1"; + my $r = $cr->run_owned("Plugin::Vote", "vote on", %ctx); + note($r); + ok(vote_started($r), "vote started"); + + # Vote yes on channel B - should fail since votes are channel-bound + $r = $cr->run_owned("Plugin::Vote", "vote yes", %ctx_b); + note($r); + + $r = $cr->run_owned("Plugin::Vote", "vote end", %ctx); + note($r); + ok_vote_match($r, 0, 0); +} + +# - Start a vote #a, start identical vote #b, vote #a yes, vote #b no, end #a vote, 1,0, end #b vote, 0,1 +{ + my %ctx_m_a = %ctx_a; + my %ctx_m_b = %ctx_b; + + # Start a vote on channel A + $ctx_m_a{rest} = "topic1"; + my $r = $cr->run_owned("Plugin::Vote", "vote on", %ctx_m_a); + note($r); + ok(vote_started($r), "vote A started"); + + # Start a vote on channel B + $ctx_m_b{rest} = "topic1"; + $r = $cr->run_owned("Plugin::Vote", "vote on", %ctx_m_b); + note($r); + ok(vote_started($r), "vote B started"); + + # Vote yes on channel A + $r = $cr->run_owned("Plugin::Vote", "vote yes", %ctx_m_a); + note($r); + + # Vote no on channel B + $r = $cr->run_owned("Plugin::Vote", "vote no", %ctx_m_b); + note($r); + + # Gather votes from A, expect 1 for, 0 against + $r = $cr->run_owned("Plugin::Vote", "vote end", %ctx_m_a); + note($r); + ok_vote_match($r, 1, 0); + + # Gather votes from B, expect 0 for, 1 against + $r = $cr->run_owned("Plugin::Vote", "vote end", %ctx_m_b); + note($r); + ok_vote_match($r, 0, 1); +} |