Skip to content
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

Implement .name for the Bool constants in Rakudo #1059

Closed
p6rt opened this issue Jun 14, 2009 · 6 comments
Closed

Implement .name for the Bool constants in Rakudo #1059

p6rt opened this issue Jun 14, 2009 · 6 comments
Labels

Comments

@p6rt
Copy link

p6rt commented Jun 14, 2009

Migrated from rt.perl.org#66576 (status was 'resolved')

Searchable as RT66576$

@p6rt
Copy link
Author

p6rt commented Jun 14, 2009

From @masak

<masak> rakudo​: say False.name
<p6eval> rakudo 77f9d7​: OUTPUT«Method 'name' not found for invocant of
class [...]
* masak submits rakudobug

@p6rt
Copy link
Author

p6rt commented Nov 16, 2009

From @kyleha

This is an automatically generated mail to inform you that tests are now available in at least one of these files​: misc/pm.txt, t/spec/S02-builtin_data_types/bool.t

commit 6654f90c9d0eb5d302cf716ba8c9a60073ad1a2d
Author​: kyle <kyle@​c213334d-75ef-0310-aa23-eaa082d1ae64>
Date​: Mon Nov 16 15​:31​:30 2009 +0000

  [t/spec] Test for RT 66576​: .name method on bool values
 
  git-svn-id​: http://svn.pugscode.org/pugs@&#8203;29103 c213334d-75ef-0310-aa23-eaa082d1ae64

Inline Patch
diff --git a/misc/pm.txt b/misc/pm.txt
index f464e5f..9765f5c 100644
--- a/misc/pm.txt
+++ b/misc/pm.txt
@@ -14,10 +14,6 @@ is meant to be used for e.g. map, which needs to work on lists like
 (1,2,3), it can't cut constants out, so it seems no more constraining
 than :(\$x) too. Should one of them go away?
 
-Kh-1: Is it true that Bool::True.name should work?  Should it return
-    the string 'True'?
-    I as because of http://rt.perl.org/rt3/Public/Bug/Display.html?id=66576
-
 ==========
 
 Answered questions:
@@ -116,4 +112,3 @@ declare my or our to make the & alias in the current lexpad, then
 "is export" does no more magic than it ordinarily does.  I suppose
 method aliases are always considered multi when they show up in a
 symbol table.
-
diff --git a/t/spec/S02-builtin_data_types/bool.t b/t/spec/S02-builtin_data_types/bool.t
index 95d408e..d5610d9 100644
--- a/t/spec/S02-builtin_data_types/bool.t
+++ b/t/spec/S02-builtin_data_types/bool.t
@@ -60,6 +60,12 @@ is(--$bool, Bool::False, 'Decrement of Bool::False produces Bool::False');
     is ('RT65514' but Bool::False), 'RT65514', 'Bool::False works with "but"';
 }
 
+#?rakudo skip 'RT 66576: .name method on bool values'
+{
+    is Bool::True.name, 'True', 'Bool::True.name works (is "True")';
+    is Bool::False.name, 'False', 'Bool::False.name works (is "False")';
+}
+
 done_testing;
 
 # vim: ft=perl6

@p6rt
Copy link
Author

p6rt commented Nov 16, 2009

The RT System itself - Status changed from 'new' to 'open'

@p6rt
Copy link
Author

p6rt commented May 2, 2011

From @tadzik

Fixed in rakudo/rakudo@5e81848ad0

@p6rt
Copy link
Author

p6rt commented May 2, 2011

From @moritz

fixed by tadzik++ (but with .key and not .name, as is current spec).

@p6rt
Copy link
Author

p6rt commented May 2, 2011

@moritz - Status changed from 'open' to 'resolved'

@p6rt p6rt closed this as completed May 2, 2011
@p6rt p6rt added the Todo label Jan 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant