Skip to content

Add more rdbg tests - #374

Merged
peterzhu2118 merged 4 commits into
ruby:masterfrom
st0012:add-more-rdbg-tests
Jul 1, 2022
Merged

Add more rdbg tests#374
peterzhu2118 merged 4 commits into
ruby:masterfrom
st0012:add-more-rdbg-tests

Conversation

@st0012

Copy link
Copy Markdown
Member

This PR adds a few new rdbg tests. But more importantly, it excludes user-defined ~/.rdbgrc from tests and restore it later.

Comment threadtest/support/test_case.rb Outdated
@st0012
st0012force-pushed the add-more-rdbg-tests branch 2 times, most recently from 9213b1e to 10e6926CompareDecember 21, 2021 20:57
Comment threadtest/support/test_case.rb Outdated
@st0012
st0012force-pushed the add-more-rdbg-tests branch 2 times, most recently from 22bfe2e to 7aa6b84CompareMarch 21, 2022 10:46
@st0012
st0012 requested a review from ko1March 21, 2022 11:16
@st0012

Copy link
Copy Markdown
MemberAuthor

@ko1 I think this one is ready too.

@st0012
st0012force-pushed the add-more-rdbg-tests branch 2 times, most recently from a48ab85 to 0847292CompareApril 13, 2022 20:46
Comment threadtest/support/utils.rb Outdated
@st0012
st0012force-pushed the add-more-rdbg-tests branch 3 times, most recently from e19bbc9 to 9bfa598CompareApril 17, 2022 18:43
@st0012
st0012force-pushed the add-more-rdbg-tests branch from a1db82a to d71ddeaCompareMay 23, 2022 08:44
@st0012
st0012force-pushed the add-more-rdbg-tests branch from d71ddea to f029421CompareJuly 1, 2022 14:02

@peterzhu2118peterzhu2118 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good, thanks!

@peterzhu2118
peterzhu2118 merged commit f0407ee into ruby:masterJul 1, 2022
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@st0012@ko1@peterzhu2118