New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Signatures don't advertise required parameters as required when serialized with .perl in Rakudo #1225
Comments
From @masak<masak> rakudo: sub foo(:$a!) {}; say &foo.signature.perl |
From @kylehaThis is an automatically generated mail to inform you that tests are now available in t/spec/S06-signature/named-parameters.t commit 5ca328e331568e1e30309e8ecef1151fecba019b [t/spec] Test for RT #68524 Inline Patchdiff --git a/t/spec/S06-signature/named-parameters.t b/t/spec/S06-signature/named-parameters.t
index 83efe31..8fff210 100644
--- a/t/spec/S06-signature/named-parameters.t
+++ b/t/spec/S06-signature/named-parameters.t
@@ -1,7 +1,7 @@
use v6;
use Test;
-plan 76;
+plan 77;
# L<S06/Required parameters/"Passing a named argument that cannot be bound to
# a normal subroutine is also a fatal error.">
@@ -220,4 +220,12 @@ ok(%fellowship<dwarf> ~~ undef, "dwarf arg was not given");
#?rakudo todo 'RT #68086'
eval_dies_ok 'sub rt68086( $a, $a ) { }', 'two sub params with the same name';
+# RT #68524
+#?rakudo todo 'RT 68524'
+{
+ sub rt68524( :$a! ) {}
+ ok( &rt68524.signature.perl ~~ m/\!/,
+ '.signature.perl with required parameter includes requirement' );
+}
+
# vim: ft=perl6 |
The RT System itself - Status changed from 'new' to 'open' |
From @jnthnOn Fri Aug 14 14:07:16 2009, masak wrote:
Fixed in git 174e0d6, during re-write of Signature.perl into the Thanks, Jonathan |
From @kylehaThe test for this passes now, so I'm marking it resolved. |
@kyleha - Status changed from 'open' to 'resolved' |
Migrated from rt.perl.org#68524 (status was 'resolved')
Searchable as RT68524$
The text was updated successfully, but these errors were encountered: