Add missing unit test coverage and fix SIP URI to pass tests. Mainly

fixing the params() setter but make the code more readable in general.

1 files changed, 29 insertions, 15 deletions

diff --git a/t/sip.t b/t/sip.t
index a12f41c..dca67e4 100644
--- a/t/sip.t
+++ b/t/sip.t
@@ -1,50 +1,64 @@
 use strict;
 use warnings;
 
-use Test::More tests => 11;
+use Test::More tests => 13;
 
 use URI ();
 
 my $u = URI->new('sip:phone@domain.ext');
 ok($u->user eq 'phone' &&
-	$u->host eq 'domain.ext' &&
-	$u->port eq '5060' &&
-	$u eq 'sip:phone@domain.ext');
+   $u->host eq 'domain.ext' &&
+   $u->port eq '5060' &&
+   $u->host_port eq 'domain.ext:5060' &&
+   $u->authority eq 'phone@domain.ext' &&
+   $u eq 'sip:phone@domain.ext');
 
 $u->host_port('otherdomain.int:9999');
 ok($u->host eq 'otherdomain.int' &&
    $u->port eq '9999' &&
+   $u->host_port eq 'otherdomain.int:9999' &&
+   $u->authority eq 'phone@otherdomain.int:9999' &&
    $u eq 'sip:phone@otherdomain.int:9999');
 
 $u->port('5060');
 $u = $u->canonical;
-ok($u->host eq 'otherdomain.int' &&
-   $u->port eq '5060' &&
+ok($u->port eq '5060' &&
+   $u->host_port eq 'otherdomain.int:5060' &&
+   $u->authority eq 'phone@otherdomain.int' &&
    $u eq 'sip:phone@otherdomain.int');
 
 $u->user('voicemail');
 ok($u->user eq 'voicemail' &&
+   $u->authority eq 'voicemail@otherdomain.int' &&
    $u eq 'sip:voicemail@otherdomain.int');
 
+$u->authority('fax@gateway.ext');
+ok($u->user eq 'fax' &&
+   $u->host eq 'gateway.ext' &&
+   $u->host_port eq 'gateway.ext:5060' &&
+   $u->authority eq 'fax@gateway.ext' &&
+   $u eq 'sip:fax@gateway.ext');
+
 $u = URI->new('sip:phone@domain.ext?Subject=Meeting&Priority=Urgent');
-ok($u->host eq 'domain.ext' &&
-   $u->query eq 'Subject=Meeting&Priority=Urgent');
+ok($u->query eq 'Subject=Meeting&Priority=Urgent');
 
 $u->query_form(Subject => 'Lunch', Priority => 'Low');
 my @q = $u->query_form;
-ok($u->host eq 'domain.ext' &&
-   $u->query eq 'Subject=Lunch&Priority=Low' &&
-   @q == 4 && "@q" eq "Subject Lunch Priority Low");
+ok($u->query eq 'Subject=Lunch&Priority=Low' &&
+   @q == 4 && "@q" eq 'Subject Lunch Priority Low' &&
+   $u eq 'sip:phone@domain.ext?Subject=Lunch&Priority=Low');
 
 $u = URI->new('sip:phone@domain.ext;maddr=127.0.0.1;ttl=16');
-ok($u->host eq 'domain.ext' &&
-   $u->params eq 'maddr=127.0.0.1;ttl=16');
+ok($u->params eq 'maddr=127.0.0.1;ttl=16');
+
+$u->params('maddr=127.0.0.1;ttl=16;x-addedparam=1');
+ok($u->params eq 'maddr=127.0.0.1;ttl=16;x-addedparam=1' && 
+   $u eq 'sip:phone@domain.ext;maddr=127.0.0.1;ttl=16;x-addedparam=1');
 
 $u = URI->new('sip:phone@domain.ext?Subject=Meeting&Priority=Urgent');
 $u->params_form(maddr => '127.0.0.1', ttl => '16');
 my @p = $u->params_form;
-ok($u->host eq 'domain.ext' &&
-   $u->query eq 'Subject=Meeting&Priority=Urgent' &&
+ok($u->query eq 'Subject=Meeting&Priority=Urgent' &&
    $u->params eq 'maddr=127.0.0.1;ttl=16' &&
    @p == 4 && "@p" eq "maddr 127.0.0.1 ttl 16");