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

Many test failures due to X::Method::NotFound instead of X::Assignment::RO #5957

Closed
p6rt opened this issue Dec 31, 2016 · 8 comments
Closed
Labels
JVM Related to Rakudo-JVM

Comments

@p6rt
Copy link

p6rt commented Dec 31, 2016

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

Searchable as RT130470$

@p6rt
Copy link
Author

p6rt commented Dec 31, 2016

From @usev6

There are many test failures for rakudo-j because
X​::Method​::NotFound is returned instead of X​::Assignment​::RO.

I'm going to fudge those tests and will look into the issue.

@p6rt
Copy link
Author

p6rt commented Dec 31, 2016

From @lizmat

What is the method that isn’t found? Maybe we need to add it for the JVM ?

On 31 Dec 2016, at 19​:41, Christian Bartolomaeus (via RT) <perl6-bugs-followup@​perl.org> wrote​:

# New Ticket Created by Christian Bartolomaeus
# Please include the string​: [perl #​130470]
# in the subject line of all future correspondence about this issue.
# <URL​: https://rt-archive.perl.org/perl6/Ticket/Display.html?id=130470 >

There are many test failures for rakudo-j because
X​::Method​::NotFound is returned instead of X​::Assignment​::RO.

I'm going to fudge those tests and will look into the issue.

@p6rt
Copy link
Author

p6rt commented Dec 31, 2016

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

@p6rt
Copy link
Author

p6rt commented Jan 1, 2017

From @usev6

On Sat, 31 Dec 2016 11​:04​:34 -0800, elizabeth wrote​:

There are many test failures for rakudo-j because
X​::Method​::NotFound is returned instead of X​::Assignment​::RO.

What is the method that isn’t found? Maybe we need to add it for the
JVM ?

I suspect the changes from rakudo/rakudo#967
(and the mentioned nqp commit Raku/nqp@49d6beb34a)
to be the cause for the failures.

Here is one example of the newly failing code​:

$ ./perl6-j -e 'my $m = <a>.Mix; $m<a> = 42.1'
Method 'STORE' not found for invocant of class 'Int'
  in method ASSIGN-KEY at gen/jvm/CORE.setting line 2352
  in sub postcircumfix​:<{ }> at gen/jvm/CORE.setting line 8761
  in block <unit> at -e line 1

$ ./perl6-j --ll-exception -e 'my $m = <a>.Mix; $m<a> = 42.1'
Method 'STORE' not found for invocant of class 'Int'
  in ASSIGN-KEY (gen/jvm/CORE.setting​:2352)
  in ASSIGN-KEY (gen/jvm/CORE.setting​:2350)
  in postcircumfix​:<{ }> (gen/jvm/CORE.setting​:8761)
  in postcircumfix​:<{ }> (gen/jvm/CORE.setting​:8755)
  in <unit> (-e​:1)
  in <unit-outer> (-e​:1)
  in eval (gen/jvm/stage2/NQPHLL.nqp​:1165)
  in (gen/jvm/stage2/NQPHLL.nqp​:1255)
  in command_eval (gen/jvm/stage2/NQPHLL.nqp​:1252)
  in command_eval (src/Perl6/Compiler.nqp​:27)
  in command_line (gen/jvm/stage2/NQPHLL.nqp​:1236)
  in MAIN (gen/jvm/main.nqp​:47)
  in <mainline> (gen/jvm/main.nqp​:38)
  in (gen/jvm/main.nqp)

@p6rt
Copy link
Author

p6rt commented Jan 1, 2017

From @usev6

On Sun, 01 Jan 2017 06​:58​:30 -0800, bartolin@​gmx.de wrote​:

I suspect the changes from rakudo/rakudo#967
(and the mentioned nqp commit
Raku/nqp@49d6beb34a)
to be the cause for the failures.

And I was right on that. I opened a PR to fix the failures​:
rakudo/rakudo#984

@p6rt
Copy link
Author

p6rt commented Jan 1, 2017

From @usev6

The tests are passing again. I'm closing this ticket.

1 similar comment
@p6rt
Copy link
Author

p6rt commented Jan 1, 2017

From @usev6

The tests are passing again. I'm closing this ticket.

@p6rt
Copy link
Author

p6rt commented Jan 1, 2017

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

@p6rt p6rt closed this as completed Jan 1, 2017
@p6rt p6rt added the JVM Related to Rakudo-JVM label Jan 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
JVM Related to Rakudo-JVM
Projects
None yet
Development

No branches or pull requests

1 participant