Commit | Line | Data |
db1ab48d |
1 | #!/usr/bin/perl |
2 | |
3 | use strict; |
4 | use warnings; |
5 | |
7ff56534 |
6 | use Test::More tests => 87; # it's really 124 with kolibre's tests; |
db1ab48d |
7 | use Test::Exception; |
8 | |
db1ab48d |
9 | =pod |
10 | |
11 | Mutually recursive roles. |
12 | |
13 | =cut |
14 | |
15 | { |
16 | package Role::Foo; |
db1ab48d |
17 | use Moose::Role; |
18 | |
19 | requires 'foo'; |
20 | |
21 | sub bar { 'Role::Foo::bar' } |
22 | |
23 | package Role::Bar; |
db1ab48d |
24 | use Moose::Role; |
25 | |
26 | requires 'bar'; |
27 | |
28 | sub foo { 'Role::Bar::foo' } |
29 | } |
30 | |
31 | { |
32 | package My::Test1; |
db1ab48d |
33 | use Moose; |
34 | |
35 | ::lives_ok { |
36 | with 'Role::Foo', 'Role::Bar'; |
37 | } '... our mutually recursive roles combine okay'; |
38 | |
39 | package My::Test2; |
db1ab48d |
40 | use Moose; |
41 | |
42 | ::lives_ok { |
43 | with 'Role::Bar', 'Role::Foo'; |
44 | } '... our mutually recursive roles combine okay (no matter what order)'; |
45 | } |
46 | |
47 | my $test1 = My::Test1->new; |
48 | isa_ok($test1, 'My::Test1'); |
49 | |
50 | ok($test1->does('Role::Foo'), '... $test1 does Role::Foo'); |
51 | ok($test1->does('Role::Bar'), '... $test1 does Role::Bar'); |
52 | |
53 | can_ok($test1, 'foo'); |
54 | can_ok($test1, 'bar'); |
55 | |
56 | is($test1->foo, 'Role::Bar::foo', '... $test1->foo worked'); |
57 | is($test1->bar, 'Role::Foo::bar', '... $test1->bar worked'); |
58 | |
59 | my $test2 = My::Test2->new; |
60 | isa_ok($test2, 'My::Test2'); |
61 | |
62 | ok($test2->does('Role::Foo'), '... $test2 does Role::Foo'); |
63 | ok($test2->does('Role::Bar'), '... $test2 does Role::Bar'); |
64 | |
65 | can_ok($test2, 'foo'); |
66 | can_ok($test2, 'bar'); |
67 | |
68 | is($test2->foo, 'Role::Bar::foo', '... $test2->foo worked'); |
69 | is($test2->bar, 'Role::Foo::bar', '... $test2->bar worked'); |
70 | |
71 | # check some meta-stuff |
72 | |
73 | ok(Role::Foo->meta->has_method('bar'), '... it still has the bar method'); |
74 | ok(Role::Foo->meta->requires_method('foo'), '... it still has the required foo method'); |
75 | |
76 | ok(Role::Bar->meta->has_method('foo'), '... it still has the foo method'); |
77 | ok(Role::Bar->meta->requires_method('bar'), '... it still has the required bar method'); |
78 | |
79 | =pod |
80 | |
81 | Role method conflicts |
82 | |
83 | =cut |
84 | |
85 | { |
86 | package Role::Bling; |
db1ab48d |
87 | use Moose::Role; |
88 | |
89 | sub bling { 'Role::Bling::bling' } |
90 | |
91 | package Role::Bling::Bling; |
db1ab48d |
92 | use Moose::Role; |
93 | |
94 | sub bling { 'Role::Bling::Bling::bling' } |
95 | } |
96 | |
97 | { |
98 | package My::Test3; |
db1ab48d |
99 | use Moose; |
100 | |
101 | ::throws_ok { |
102 | with 'Role::Bling', 'Role::Bling::Bling'; |
103 | } qr/requires the method \'bling\' to be implemented/, '... role methods conflicted and method was required'; |
104 | |
105 | package My::Test4; |
db1ab48d |
106 | use Moose; |
107 | |
108 | ::lives_ok { |
109 | with 'Role::Bling'; |
110 | with 'Role::Bling::Bling'; |
111 | } '... role methods didnt conflict when manually combined'; |
112 | |
113 | package My::Test5; |
db1ab48d |
114 | use Moose; |
115 | |
116 | ::lives_ok { |
117 | with 'Role::Bling::Bling'; |
118 | with 'Role::Bling'; |
119 | } '... role methods didnt conflict when manually combined (in opposite order)'; |
120 | |
121 | package My::Test6; |
db1ab48d |
122 | use Moose; |
123 | |
124 | ::lives_ok { |
125 | with 'Role::Bling::Bling', 'Role::Bling'; |
126 | } '... role methods didnt conflict when manually resolved'; |
127 | |
128 | sub bling { 'My::Test6::bling' } |
129 | } |
130 | |
131 | ok(!My::Test3->meta->has_method('bling'), '... we didnt get any methods in the conflict'); |
132 | ok(My::Test4->meta->has_method('bling'), '... we did get the method when manually dealt with'); |
133 | ok(My::Test5->meta->has_method('bling'), '... we did get the method when manually dealt with'); |
134 | ok(My::Test6->meta->has_method('bling'), '... we did get the method when manually dealt with'); |
135 | |
d79e62fd |
136 | ok(!My::Test3->does('Role::Bling'), '... our class does() the correct roles'); |
137 | ok(!My::Test3->does('Role::Bling::Bling'), '... our class does() the correct roles'); |
138 | ok(My::Test4->does('Role::Bling'), '... our class does() the correct roles'); |
139 | ok(My::Test4->does('Role::Bling::Bling'), '... our class does() the correct roles'); |
140 | ok(My::Test5->does('Role::Bling'), '... our class does() the correct roles'); |
141 | ok(My::Test5->does('Role::Bling::Bling'), '... our class does() the correct roles'); |
142 | ok(My::Test6->does('Role::Bling'), '... our class does() the correct roles'); |
143 | ok(My::Test6->does('Role::Bling::Bling'), '... our class does() the correct roles'); |
144 | |
db1ab48d |
145 | is(My::Test4->bling, 'Role::Bling::bling', '... and we got the first method that was added'); |
146 | is(My::Test5->bling, 'Role::Bling::Bling::bling', '... and we got the first method that was added'); |
147 | is(My::Test6->bling, 'My::Test6::bling', '... and we got the local method'); |
148 | |
149 | # check how this affects role compostion |
150 | |
151 | { |
152 | package Role::Bling::Bling::Bling; |
db1ab48d |
153 | use Moose::Role; |
154 | |
155 | with 'Role::Bling::Bling'; |
156 | |
157 | sub bling { 'Role::Bling::Bling::Bling::bling' } |
158 | } |
159 | |
160 | ok(Role::Bling::Bling->meta->has_method('bling'), '... still got the bling method in Role::Bling::Bling'); |
d79e62fd |
161 | ok(Role::Bling::Bling->meta->does_role('Role::Bling::Bling'), '... our role correctly does() the other role'); |
fb1e11d5 |
162 | ok(Role::Bling::Bling::Bling->meta->has_method('bling'), '... dont have the bling method in Role::Bling::Bling::Bling'); |
d05cd563 |
163 | is(Role::Bling::Bling::Bling->meta->get_method('bling')->(), |
164 | 'Role::Bling::Bling::Bling::bling', |
165 | '... still got the bling method in Role::Bling::Bling::Bling'); |
db1ab48d |
166 | |
fb1e11d5 |
167 | |
db1ab48d |
168 | =pod |
169 | |
170 | Role attribute conflicts |
171 | |
172 | =cut |
173 | |
174 | { |
175 | package Role::Boo; |
db1ab48d |
176 | use Moose::Role; |
177 | |
178 | has 'ghost' => (is => 'ro', default => 'Role::Boo::ghost'); |
179 | |
180 | package Role::Boo::Hoo; |
db1ab48d |
181 | use Moose::Role; |
182 | |
183 | has 'ghost' => (is => 'ro', default => 'Role::Boo::Hoo::ghost'); |
184 | } |
185 | |
186 | { |
187 | package My::Test7; |
db1ab48d |
188 | use Moose; |
189 | |
190 | ::throws_ok { |
191 | with 'Role::Boo', 'Role::Boo::Hoo'; |
fb1e11d5 |
192 | } qr/We have encountered an attribute conflict/, |
db1ab48d |
193 | '... role attrs conflicted and method was required'; |
194 | |
195 | package My::Test8; |
db1ab48d |
196 | use Moose; |
197 | |
198 | ::lives_ok { |
199 | with 'Role::Boo'; |
200 | with 'Role::Boo::Hoo'; |
201 | } '... role attrs didnt conflict when manually combined'; |
202 | |
203 | package My::Test9; |
db1ab48d |
204 | use Moose; |
205 | |
206 | ::lives_ok { |
207 | with 'Role::Boo::Hoo'; |
208 | with 'Role::Boo'; |
209 | } '... role attrs didnt conflict when manually combined'; |
210 | |
211 | package My::Test10; |
db1ab48d |
212 | use Moose; |
213 | |
214 | has 'ghost' => (is => 'ro', default => 'My::Test10::ghost'); |
215 | |
216 | ::throws_ok { |
217 | with 'Role::Boo', 'Role::Boo::Hoo'; |
fb1e11d5 |
218 | } qr/We have encountered an attribute conflict/, |
db1ab48d |
219 | '... role attrs conflicted and cannot be manually disambiguted'; |
220 | |
221 | } |
222 | |
223 | ok(!My::Test7->meta->has_attribute('ghost'), '... we didnt get any attributes in the conflict'); |
224 | ok(My::Test8->meta->has_attribute('ghost'), '... we did get an attributes when manually composed'); |
225 | ok(My::Test9->meta->has_attribute('ghost'), '... we did get an attributes when manually composed'); |
226 | ok(My::Test10->meta->has_attribute('ghost'), '... we did still have an attribute ghost (conflict does not mess with class)'); |
227 | |
d79e62fd |
228 | ok(!My::Test7->does('Role::Boo'), '... our class does() the correct roles'); |
229 | ok(!My::Test7->does('Role::Boo::Hoo'), '... our class does() the correct roles'); |
230 | ok(My::Test8->does('Role::Boo'), '... our class does() the correct roles'); |
231 | ok(My::Test8->does('Role::Boo::Hoo'), '... our class does() the correct roles'); |
232 | ok(My::Test9->does('Role::Boo'), '... our class does() the correct roles'); |
233 | ok(My::Test9->does('Role::Boo::Hoo'), '... our class does() the correct roles'); |
234 | ok(!My::Test10->does('Role::Boo'), '... our class does() the correct roles'); |
235 | ok(!My::Test10->does('Role::Boo::Hoo'), '... our class does() the correct roles'); |
236 | |
db1ab48d |
237 | can_ok('My::Test8', 'ghost'); |
238 | can_ok('My::Test9', 'ghost'); |
239 | can_ok('My::Test10', 'ghost'); |
240 | |
241 | is(My::Test8->new->ghost, 'Role::Boo::ghost', '... got the expected default attr value'); |
242 | is(My::Test9->new->ghost, 'Role::Boo::Hoo::ghost', '... got the expected default attr value'); |
243 | is(My::Test10->new->ghost, 'My::Test10::ghost', '... got the expected default attr value'); |
244 | |
d05cd563 |
245 | =pod |
246 | |
247 | Role override method conflicts |
248 | |
249 | =cut |
250 | |
0558683c |
251 | { |
252 | package Role::Plot; |
253 | use Moose::Role; |
254 | |
255 | override 'twist' => sub { |
256 | super() . ' -> Role::Plot::twist'; |
257 | }; |
258 | |
259 | package Role::Truth; |
260 | use Moose::Role; |
261 | |
262 | override 'twist' => sub { |
263 | super() . ' -> Role::Truth::twist'; |
264 | }; |
265 | } |
266 | |
267 | { |
268 | package My::Test::Base; |
269 | use Moose; |
270 | |
271 | sub twist { 'My::Test::Base::twist' } |
272 | |
273 | package My::Test11; |
274 | use Moose; |
275 | |
276 | extends 'My::Test::Base'; |
277 | |
278 | ::lives_ok { |
279 | with 'Role::Truth'; |
280 | } '... composed the role with override okay'; |
281 | |
282 | package My::Test12; |
283 | use Moose; |
284 | |
285 | extends 'My::Test::Base'; |
286 | |
287 | ::lives_ok { |
288 | with 'Role::Plot'; |
289 | } '... composed the role with override okay'; |
290 | |
291 | package My::Test13; |
292 | use Moose; |
293 | |
294 | ::dies_ok { |
295 | with 'Role::Plot'; |
296 | } '... cannot compose it because we have no superclass'; |
297 | |
298 | package My::Test14; |
299 | use Moose; |
300 | |
301 | extends 'My::Test::Base'; |
302 | |
303 | ::throws_ok { |
304 | with 'Role::Plot', 'Role::Truth'; |
305 | } qr/Two \'override\' methods of the same name encountered/, |
306 | '... cannot compose it because we have no superclass'; |
307 | } |
308 | |
309 | ok(My::Test11->meta->has_method('twist'), '... the twist method has been added'); |
310 | ok(My::Test12->meta->has_method('twist'), '... the twist method has been added'); |
311 | ok(!My::Test13->meta->has_method('twist'), '... the twist method has not been added'); |
312 | ok(!My::Test14->meta->has_method('twist'), '... the twist method has not been added'); |
313 | |
314 | ok(!My::Test11->does('Role::Plot'), '... our class does() the correct roles'); |
315 | ok(My::Test11->does('Role::Truth'), '... our class does() the correct roles'); |
316 | ok(!My::Test12->does('Role::Truth'), '... our class does() the correct roles'); |
317 | ok(My::Test12->does('Role::Plot'), '... our class does() the correct roles'); |
318 | ok(!My::Test13->does('Role::Plot'), '... our class does() the correct roles'); |
319 | ok(!My::Test14->does('Role::Truth'), '... our class does() the correct roles'); |
320 | ok(!My::Test14->does('Role::Plot'), '... our class does() the correct roles'); |
321 | |
322 | is(My::Test11->twist(), 'My::Test::Base::twist -> Role::Truth::twist', '... got the right method return'); |
323 | is(My::Test12->twist(), 'My::Test::Base::twist -> Role::Plot::twist', '... got the right method return'); |
324 | ok(!My::Test13->can('twist'), '... no twist method here at all'); |
325 | is(My::Test14->twist(), 'My::Test::Base::twist', '... got the right method return (from superclass)'); |
326 | |
327 | { |
328 | package Role::Reality; |
329 | use Moose::Role; |
330 | |
331 | ::throws_ok { |
332 | with 'Role::Plot'; |
333 | } qr/A local method of the same name as been found/, |
334 | '... could not compose roles here, it dies'; |
335 | |
336 | sub twist { |
337 | 'Role::Reality::twist'; |
338 | } |
339 | } |
340 | |
341 | ok(Role::Reality->meta->has_method('twist'), '... the twist method has not been added'); |
fb1e11d5 |
342 | #ok(!Role::Reality->meta->does_role('Role::Plot'), '... our role does() the correct roles'); |
0558683c |
343 | is(Role::Reality->meta->get_method('twist')->(), |
344 | 'Role::Reality::twist', |
345 | '... the twist method returns the right value'); |
3c5fd53a |
346 | |
347 | =pod |
348 | |
349 | Role conflicts between attributes and methods |
350 | |
21716c07 |
351 | [15:23] <kolibrie> when class defines method and role defines method, class wins |
352 | [15:24] <kolibrie> when class 'has' method and role defines method, class wins |
353 | [15:24] <kolibrie> when class defines method and role 'has' method, role wins |
354 | [15:24] <kolibrie> when class 'has' method and role 'has' method, role wins |
355 | [15:24] <kolibrie> which means when class 'has' method and two roles 'has' method, no tiebreak is d |
356 | [15:24] <kolibrie> etected |
357 | [15:24] <perigrin> this is with role and has declaration in the exact same order in every case? |
358 | [15:25] <kolibrie> yes |
359 | [15:25] <perigrin> interesting |
360 | [15:25] <kolibrie> that's what I thought |
361 | [15:26] <kolibrie> does that sound like something I should write a test for? |
362 | [15:27] <perigrin> stevan, ping? |
363 | [15:27] <perigrin> I'm not sure what the right answer for composition is. |
364 | [15:27] <perigrin> who should win |
365 | [15:27] <perigrin> if I were to guess I'd say the class should always win. |
366 | [15:27] <kolibrie> that would be my guess, but I thought I would ask to make sure |
367 | [15:29] <stevan> kolibrie: please write a test |
368 | [15:29] <stevan> I am not exactly sure who should win either,.. but I suspect it is not working correctly right now |
369 | [15:29] <stevan> I know exactly why it is doing what it is doing though |
370 | |
371 | Now I have to decide actually what happens, and how to fix it. |
372 | - SL |
3c5fd53a |
373 | |
374 | { |
375 | package Role::Method; |
376 | use Moose::Role; |
377 | |
378 | sub ghost { 'Role::Method::ghost' } |
379 | |
380 | package Role::Method2; |
381 | use Moose::Role; |
382 | |
383 | sub ghost { 'Role::Method2::ghost' } |
384 | |
385 | package Role::Attribute; |
386 | use Moose::Role; |
387 | |
388 | has 'ghost' => (is => 'ro', default => 'Role::Attribute::ghost'); |
389 | |
390 | package Role::Attribute2; |
391 | use Moose::Role; |
392 | |
393 | has 'ghost' => (is => 'ro', default => 'Role::Attribute2::ghost'); |
394 | } |
395 | |
396 | { |
397 | package My::Test15; |
398 | use Moose; |
399 | |
400 | ::lives_ok { |
401 | with 'Role::Method'; |
402 | } '... composed the method role into the method class'; |
403 | |
404 | sub ghost { 'My::Test15::ghost' } |
405 | |
406 | package My::Test16; |
407 | use Moose; |
408 | |
409 | ::lives_ok { |
410 | with 'Role::Method'; |
411 | } '... composed the method role into the attribute class'; |
412 | |
413 | has 'ghost' => (is => 'ro', default => 'My::Test16::ghost'); |
414 | |
415 | package My::Test17; |
416 | use Moose; |
417 | |
418 | ::lives_ok { |
419 | with 'Role::Attribute'; |
420 | } '... composed the attribute role into the method class'; |
421 | |
422 | sub ghost { 'My::Test17::ghost' } |
423 | |
424 | package My::Test18; |
425 | use Moose; |
426 | |
427 | ::lives_ok { |
428 | with 'Role::Attribute'; |
429 | } '... composed the attribute role into the attribute class'; |
430 | |
431 | has 'ghost' => (is => 'ro', default => 'My::Test18::ghost'); |
432 | |
433 | package My::Test19; |
434 | use Moose; |
435 | |
436 | ::lives_ok { |
437 | with 'Role::Method', 'Role::Method2'; |
438 | } '... composed method roles into class with method tiebreaker'; |
439 | |
440 | sub ghost { 'My::Test19::ghost' } |
441 | |
442 | package My::Test20; |
443 | use Moose; |
444 | |
445 | ::lives_ok { |
446 | with 'Role::Method', 'Role::Method2'; |
447 | } '... composed method roles into class with attribute tiebreaker'; |
448 | |
449 | has 'ghost' => (is => 'ro', default => 'My::Test20::ghost'); |
450 | |
451 | package My::Test21; |
452 | use Moose; |
453 | |
454 | ::lives_ok { |
455 | with 'Role::Attribute', 'Role::Attribute2'; |
456 | } '... composed attribute roles into class with method tiebreaker'; |
457 | |
458 | sub ghost { 'My::Test21::ghost' } |
459 | |
460 | package My::Test22; |
461 | use Moose; |
462 | |
463 | ::lives_ok { |
464 | with 'Role::Attribute', 'Role::Attribute2'; |
465 | } '... composed attribute roles into class with attribute tiebreaker'; |
466 | |
467 | has 'ghost' => (is => 'ro', default => 'My::Test22::ghost'); |
468 | |
469 | package My::Test23; |
470 | use Moose; |
471 | |
472 | ::lives_ok { |
473 | with 'Role::Method', 'Role::Attribute'; |
474 | } '... composed method and attribute role into class with method tiebreaker'; |
475 | |
476 | sub ghost { 'My::Test23::ghost' } |
477 | |
478 | package My::Test24; |
479 | use Moose; |
480 | |
481 | ::lives_ok { |
482 | with 'Role::Method', 'Role::Attribute'; |
483 | } '... composed method and attribute role into class with attribute tiebreaker'; |
484 | |
485 | has 'ghost' => (is => 'ro', default => 'My::Test24::ghost'); |
486 | |
487 | package My::Test25; |
488 | use Moose; |
489 | |
490 | ::lives_ok { |
491 | with 'Role::Attribute', 'Role::Method'; |
492 | } '... composed attribute and method role into class with method tiebreaker'; |
493 | |
494 | sub ghost { 'My::Test25::ghost' } |
495 | |
496 | package My::Test26; |
497 | use Moose; |
498 | |
499 | ::lives_ok { |
500 | with 'Role::Attribute', 'Role::Method'; |
501 | } '... composed attribute and method role into class with attribute tiebreaker'; |
502 | |
503 | has 'ghost' => (is => 'ro', default => 'My::Test26::ghost'); |
504 | } |
505 | |
506 | my $test15 = My::Test15->new; |
507 | isa_ok($test15, 'My::Test15'); |
508 | is($test15->ghost, 'My::Test15::ghost', '... we access the method from the class and ignore the role method'); |
509 | |
510 | my $test16 = My::Test16->new; |
511 | isa_ok($test16, 'My::Test16'); |
512 | is($test16->ghost, 'My::Test16::ghost', '... we access the attribute from the class and ignore the role method'); |
513 | |
514 | my $test17 = My::Test17->new; |
515 | isa_ok($test17, 'My::Test17'); |
516 | is($test17->ghost, 'My::Test17::ghost', '... we access the method from the class and ignore the role attribute'); |
517 | |
518 | my $test18 = My::Test18->new; |
519 | isa_ok($test18, 'My::Test18'); |
520 | is($test18->ghost, 'My::Test18::ghost', '... we access the attribute from the class and ignore the role attribute'); |
521 | |
522 | my $test19 = My::Test19->new; |
523 | isa_ok($test19, 'My::Test19'); |
524 | is($test19->ghost, 'My::Test19::ghost', '... we access the method from the class and ignore the role methods'); |
525 | |
526 | my $test20 = My::Test20->new; |
527 | isa_ok($test20, 'My::Test20'); |
528 | is($test20->ghost, 'My::Test20::ghost', '... we access the attribute from the class and ignore the role methods'); |
529 | |
530 | my $test21 = My::Test21->new; |
531 | isa_ok($test21, 'My::Test21'); |
532 | is($test21->ghost, 'My::Test21::ghost', '... we access the method from the class and ignore the role attributes'); |
533 | |
534 | my $test22 = My::Test22->new; |
535 | isa_ok($test22, 'My::Test22'); |
536 | is($test22->ghost, 'My::Test22::ghost', '... we access the attribute from the class and ignore the role attributes'); |
537 | |
538 | my $test23 = My::Test23->new; |
539 | isa_ok($test23, 'My::Test23'); |
540 | is($test23->ghost, 'My::Test23::ghost', '... we access the method from the class and ignore the role method and attribute'); |
541 | |
542 | my $test24 = My::Test24->new; |
543 | isa_ok($test24, 'My::Test24'); |
544 | is($test24->ghost, 'My::Test24::ghost', '... we access the attribute from the class and ignore the role method and attribute'); |
545 | |
546 | my $test25 = My::Test25->new; |
547 | isa_ok($test25, 'My::Test25'); |
548 | is($test25->ghost, 'My::Test25::ghost', '... we access the method from the class and ignore the role attribute and method'); |
549 | |
550 | my $test26 = My::Test26->new; |
551 | isa_ok($test26, 'My::Test26'); |
552 | is($test26->ghost, 'My::Test26::ghost', '... we access the attribute from the class and ignore the role attribute and method'); |
553 | |
21716c07 |
554 | =cut |