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

Activate yamatanooroti tests on CI #459

Merged
merged 2 commits into from
Nov 27, 2022

Conversation

st0012
Copy link
Member

@st0012 st0012 commented Nov 26, 2022

The env is also needed for running the tests, otherwise it won't load the vterm gem and won't do anything.

@st0012 st0012 force-pushed the fix-yamatanooroti-tests branch 3 times, most recently from 4603be0 to a30706c Compare November 26, 2022 22:41
@st0012 st0012 force-pushed the fix-yamatanooroti-tests branch from a30706c to d720470 Compare November 26, 2022 22:44
(rdbg:irb) delete 0
deleted: #0 BP - Line #{@ruby_file}:5 (line)
deleted: #0 BP - Line
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need to loose this expectation because debug can show additional (pending) info based on Ruby versions (depending on iseq support).
Since we just want to test if debug receives the command and argument correctly, matching only this part of the message should be good enough.

Copy link
Member

@k0kubun k0kubun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤦‍♂️

@k0kubun k0kubun merged commit 750cf4c into ruby:master Nov 27, 2022
matzbot pushed a commit to ruby/ruby that referenced this pull request Nov 27, 2022
(ruby/irb#459)

* Activate yamatanooroti tests on CI

* Fix delete test

ruby/irb@750cf4c480
@st0012 st0012 deleted the fix-yamatanooroti-tests branch November 27, 2022 10:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants