updatin
[gitmo/Moose.git] / t / 004_basic.t
index 5f6f253..a3a36da 100644 (file)
@@ -3,8 +3,16 @@
 use strict;
 use warnings;
 
-use Test::More tests => 64;
+use Test::More; 
+
+BEGIN {
+    eval "use Regexp::Common; use Locale::US;";
+    plan skip_all => "Regexp::Common & Locale::US required for this test" if $@;        
+    plan tests => 81;    
+}
+
 use Test::Exception;
+use Scalar::Util 'isweak';
 
 BEGIN {
     use_ok('Moose');           
@@ -15,6 +23,7 @@ BEGIN {
     use strict;
     use warnings;
     use Moose;
+    use Moose::Util::TypeConstraints;
     
     use Locale::US;
     use Regexp::Common 'zip';
@@ -33,26 +42,27 @@ BEGIN {
             /^$RE{zip}{US}{-extended => 'allow'}$/            
         };
     
-    has 'street'   => (is => 'rw', isa => Str());
-    has 'city'     => (is => 'rw', isa => Str());
-    has 'state'    => (is => 'rw', isa => USState());
-    has 'zip_code' => (is => 'rw', isa => USZipCode());   
+    has 'street'   => (is => 'rw', isa => 'Str');
+    has 'city'     => (is => 'rw', isa => 'Str');
+    has 'state'    => (is => 'rw', isa => 'USState');
+    has 'zip_code' => (is => 'rw', isa => 'USZipCode');   
     
     package Company;
     use strict;
     use warnings;
     use Moose;
+    use Moose::Util::TypeConstraints;    
     
-    has 'name'      => (is => 'rw', isa => Str());
+    has 'name'      => (is => 'rw', isa => 'Str', required => 1);
     has 'address'   => (is => 'rw', isa => 'Address'); 
     has 'employees' => (is => 'rw', isa => subtype ArrayRef => where { 
-        ($_->isa('Employee') || return) for @$_; 1 
+        (blessed($_) && $_->isa('Employee') || return) for @$_; 1 
     });    
     
     sub BUILD {
-        my ($self, %params) = @_;
-        if ($params{employees}) {
-            foreach my $employee (@{$params{employees}}) {
+        my ($self, $params) = @_;
+        if ($params->{employees}) {
+            foreach my $employee (@{$params->{employees}}) {
                 $employee->company($self);
             }
         }
@@ -60,14 +70,27 @@ BEGIN {
     
     sub get_employee_count { scalar @{(shift)->employees} }
     
+    after 'employees' => sub {
+        my ($self, $employees) = @_;
+        # if employees is defined, it 
+        # has already been type checked
+        if (defined $employees) {
+            # make sure each gets the 
+            # weak ref to the company
+            foreach my $employee (@{$employees}) {
+                $employee->company($self);
+            }            
+        }
+    };
+    
     package Person;
     use strict;
     use warnings;
     use Moose;
     
-    has 'first_name'     => (is => 'rw', isa => Str());
-    has 'last_name'      => (is => 'rw', isa => Str());       
-    has 'middle_initial' => (is => 'rw', isa => Str(), predicate => 'has_middle_initial');  
+    has 'first_name'     => (is => 'rw', isa => 'Str', required => 1);
+    has 'last_name'      => (is => 'rw', isa => 'Str', required => 1);       
+    has 'middle_initial' => (is => 'rw', isa => 'Str', predicate => 'has_middle_initial');  
     has 'address'        => (is => 'rw', isa => 'Address');
     
     sub full_name {
@@ -84,13 +107,18 @@ BEGIN {
     
     extends 'Person';
     
-    has 'title'   => (is => 'rw', isa => Str());
+    has 'title'   => (is => 'rw', isa => 'Str', required => 1);
     has 'company' => (is => 'rw', isa => 'Company', weak_ref => 1);  
+    
+    override 'full_name' => sub {
+        my $self = shift;
+        super() . ', ' . $self->title
+    };
 }
 
 my $ii;
 lives_ok {
-    $ii = Company->new(
+    $ii = Company->new({
         name    => 'Infinity Interactive',
         address => Address->new(
             street   => '565 Plandome Rd., Suite 307',
@@ -125,7 +153,7 @@ lives_ok {
                 address        => Address->new(city => 'Marysville', state => 'OH')
             ),        
         ]
-    );
+    });
 } '... created the entire company successfully';
 isa_ok($ii, 'Company');
 
@@ -148,9 +176,10 @@ is($ii->employees->[0]->first_name, 'Jeremy', '... got the right first name');
 is($ii->employees->[0]->last_name, 'Shao', '... got the right last name');
 ok(!$ii->employees->[0]->has_middle_initial, '... no middle initial');
 is($ii->employees->[0]->middle_initial, undef, '... got the right middle initial value');
-is($ii->employees->[0]->full_name, 'Jeremy Shao', '... got the right full name');
+is($ii->employees->[0]->full_name, 'Jeremy Shao, President / Senior Consultant', '... got the right full name');
 is($ii->employees->[0]->title, 'President / Senior Consultant', '... got the right title');
 is($ii->employees->[0]->company, $ii, '... got the right company');
+ok(isweak($ii->employees->[0]->{company}), '... the company is a weak-ref');
 
 isa_ok($ii->employees->[0]->address, 'Address');
 is($ii->employees->[0]->address->city, 'Manhasset', '... got the right city');
@@ -165,9 +194,10 @@ is($ii->employees->[1]->first_name, 'Tommy', '... got the right first name');
 is($ii->employees->[1]->last_name, 'Lee', '... got the right last name');
 ok(!$ii->employees->[1]->has_middle_initial, '... no middle initial');
 is($ii->employees->[1]->middle_initial, undef, '... got the right middle initial value');
-is($ii->employees->[1]->full_name, 'Tommy Lee', '... got the right full name');
+is($ii->employees->[1]->full_name, 'Tommy Lee, Vice President / Senior Developer', '... got the right full name');
 is($ii->employees->[1]->title, 'Vice President / Senior Developer', '... got the right title');
 is($ii->employees->[1]->company, $ii, '... got the right company');
+ok(isweak($ii->employees->[1]->{company}), '... the company is a weak-ref');
 
 isa_ok($ii->employees->[1]->address, 'Address');
 is($ii->employees->[1]->address->city, 'New York', '... got the right city');
@@ -182,9 +212,10 @@ is($ii->employees->[2]->first_name, 'Stevan', '... got the right first name');
 is($ii->employees->[2]->last_name, 'Little', '... got the right last name');
 ok($ii->employees->[2]->has_middle_initial, '... got middle initial');
 is($ii->employees->[2]->middle_initial, 'C', '... got the right middle initial value');
-is($ii->employees->[2]->full_name, 'Stevan C. Little', '... got the right full name');
+is($ii->employees->[2]->full_name, 'Stevan C. Little, Senior Developer', '... got the right full name');
 is($ii->employees->[2]->title, 'Senior Developer', '... got the right title');
 is($ii->employees->[2]->company, $ii, '... got the right company');
+ok(isweak($ii->employees->[2]->{company}), '... the company is a weak-ref');
 
 isa_ok($ii->employees->[2]->address, 'Address');
 is($ii->employees->[2]->address->city, 'Madison', '... got the right city');
@@ -199,17 +230,42 @@ is($ii->employees->[3]->first_name, 'Rob', '... got the right first name');
 is($ii->employees->[3]->last_name, 'Kinyon', '... got the right last name');
 ok(!$ii->employees->[3]->has_middle_initial, '... got middle initial');
 is($ii->employees->[3]->middle_initial, undef, '... got the right middle initial value');
-is($ii->employees->[3]->full_name, 'Rob Kinyon', '... got the right full name');
+is($ii->employees->[3]->full_name, 'Rob Kinyon, Developer', '... got the right full name');
 is($ii->employees->[3]->title, 'Developer', '... got the right title');
 is($ii->employees->[3]->company, $ii, '... got the right company');
+ok(isweak($ii->employees->[3]->{company}), '... the company is a weak-ref');
 
 isa_ok($ii->employees->[3]->address, 'Address');
 is($ii->employees->[3]->address->city, 'Marysville', '... got the right city');
 is($ii->employees->[3]->address->state, 'OH', '... got the right state');
 
+# create new company
+
+my $new_company = Company->new(name => 'Infinity Interactive International');
+isa_ok($new_company, 'Company');
+
+my $ii_employees = $ii->employees;
+foreach my $employee (@$ii_employees) {
+    is($employee->company, $ii, '... has the ii company');
+}
+
+$new_company->employees($ii_employees);
+
+foreach my $employee (@{$new_company->employees}) {
+    is($employee->company, $new_company, '... has the different company now');
+}
+
 ## check some error conditions for the subtypes
 
 dies_ok {
+    Address->new(street => {}),    
+} '... we die correctly with bad args';
+
+dies_ok {
+    Address->new(city => {}),    
+} '... we die correctly with bad args';
+
+dies_ok {
     Address->new(state => 'British Columbia'),    
 } '... we die correctly with bad args';
 
@@ -226,10 +282,18 @@ lives_ok {
 } '... we live correctly with good args';
 
 dies_ok {
-    Company->new(employees => [ Person->new ]),    
+    Company->new(),    
+} '... we die correctly without good args';
+
+lives_ok {
+    Company->new(name => 'Foo'),    
+} '... we live correctly without good args';
+
+dies_ok {
+    Company->new(name => 'Foo', employees => [ Person->new ]),    
 } '... we die correctly with good args';
 
 lives_ok {
-    Company->new(employees => []),    
+    Company->new(name => 'Foo', employees => []),    
 } '... we live correctly with good args';