Skip to content

Add basic #invokeOn:with: test#85

Open
smarr wants to merge 1 commit into
SOM-st:masterfrom
smarr:add-invoke-on-test
Open

Add basic #invokeOn:with: test#85
smarr wants to merge 1 commit into
SOM-st:masterfrom
smarr:add-invoke-on-test

Conversation

@smarr

@smarr smarr commented Dec 5, 2021

Copy link
Copy Markdown
Member

There was no usage of #invokeOn:with: yet in the code base, so, here a very minimal test.

Signed-off-by: Stefan Marr <git@stefan-marr.de>
@smarr smarr added the enhancement Improves the implementation with something noteworthy label Dec 5, 2021
@smarr

smarr commented Dec 5, 2021

Copy link
Copy Markdown
Member Author

Hmpf. Only TruffleSOM even implements the primitive as it seems...

@Hirevo

Hirevo commented Dec 5, 2021

Copy link
Copy Markdown

SOM-rs also implements the primitive.
I ran the new test code in the REPL and it gives me the correct outputs.
The test failure seems to indicate that it is something else that evaluated incorrectly, I'll look into it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Improves the implementation with something noteworthy

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants