Skip to content

Skip tests that require stty when it's not available #597

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

Merged
merged 1 commit into from
Jun 21, 2017
Merged

Skip tests that require stty when it's not available #597

merged 1 commit into from
Jun 21, 2017

Conversation

javiereguiluz
Copy link
Member

This fixes #530.

@javiereguiluz javiereguiluz merged commit 1be9abb into symfony:master Jun 21, 2017
javiereguiluz added a commit that referenced this pull request Jun 21, 2017
…eguiluz)

This PR was merged into the master branch.

Discussion
----------

Skip tests that require stty when it's not available

This fixes #530.

Commits
-------

1be9abb Skip tests that require stty when it's not available
@stof
Copy link
Member

stof commented Jun 21, 2017

you should also skip it on windows all the time (your code may do it already, not sure), as the windows implementation is not supported by your test either AFAIK

@dzuelke
Copy link
Contributor

dzuelke commented Jun 22, 2017

Thanks @javiereguiluz !

javiereguiluz added a commit that referenced this pull request Jun 25, 2017
This PR was merged into the master branch.

Discussion
----------

Exclude some tests on Windows too

This continues #597 according to @stof comment: #597 (comment)

Commits
-------

515d490 Exclude some tests on Windows too
sayjun0505 added a commit to sayjun0505/sym_proj that referenced this pull request Apr 16, 2023
This PR was merged into the master branch.

Discussion
----------

Exclude some tests on Windows too

This continues #597 according to @stof comment: symfony/demo#597 (comment)

Commits
-------

515d490 Exclude some tests on Windows too
spider-yamet added a commit to spider-yamet/sym_proj that referenced this pull request Apr 16, 2023
This PR was merged into the master branch.

Discussion
----------

Exclude some tests on Windows too

This continues #597 according to @stof comment: symfony/demo#597 (comment)

Commits
-------

515d490 Exclude some tests on Windows too
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Tests fail for AddUserCommand if shell is not a TTY
3 participants