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

Convert t/mojo/reactor_ev.t to use subtests #1970

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

misch2
Copy link
Contributor

@misch2 misch2 commented Aug 2, 2022

Summary

Tests in t/mojo/reactor_ev.t converted to use subtests.

Motivation

Help convert remaining subtests.

References

#1520

@kraih
Copy link
Member

kraih commented Aug 2, 2022

Please squash commits.

@misch2 misch2 force-pushed the michals/convert-tests-to-use-subtests-reactor-ev branch from fc2f924 to e67753f Compare August 3, 2022 12:11
@misch2
Copy link
Contributor Author

misch2 commented Aug 3, 2022

Commits squashed

marcusramberg
marcusramberg previously approved these changes Aug 14, 2022
Copy link
Member

@marcusramberg marcusramberg left a comment

Choose a reason for hiding this comment

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

LGTM

t/mojo/reactor_ev.t Outdated Show resolved Hide resolved
t/mojo/reactor_ev.t Outdated Show resolved Hide resolved
@jhthorsen
Copy link
Member

I also think you might want to squash the commits and write a more descriptive commit message.

@misch2 misch2 force-pushed the michals/convert-tests-to-use-subtests-reactor-ev branch from 6060e16 to 5a8c74d Compare August 18, 2022 16:21
@misch2
Copy link
Contributor Author

misch2 commented Aug 18, 2022

Commits squashed.

t/mojo/reactor_ev.t Outdated Show resolved Hide resolved
t/mojo/reactor_ev.t Outdated Show resolved Hide resolved
@misch2 misch2 force-pushed the michals/convert-tests-to-use-subtests-reactor-ev branch from dc9e7e4 to ed0a5e9 Compare August 22, 2022 17:40
Sign up for free to 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.

None yet

4 participants