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

Ease removal of whitespace for Powershell Write-Output and VBScript Echo #592

Merged
merged 2 commits into from
Mar 26, 2016

Conversation

chris-rock
Copy link
Contributor

This PR implements a new strip method to handle stdout

Powershell

# Write-Output comes with a newline
describe powershell("Write-Output 'hello'") do
  its('stdout') { should eq "hello\r\n" }
  # remove whitespace \r\n from stdout
  its('strip') { should eq "hello" }
  its('stderr') { should eq '' }
end

VBScript:

describe vbscript('WScript.Echo "hello"') do
  its('stdout') { should eq "hello\r\n" }
  # remove whitespace \r\n from stdout
  its('strip') { should eq "hello" }
end

@chris-rock chris-rock changed the title Ease the remove of whitespace for Write-Output and Echo in Powershell / VBScript Ease removal of whitespace for Powershell Write-Output and VBScript Echo Mar 26, 2016
@arlimus
Copy link
Contributor

arlimus commented Mar 26, 2016

Thank you @chris-rock !

@arlimus arlimus merged commit 1c60908 into master Mar 26, 2016
@arlimus arlimus deleted the chris-rock/strip branch March 26, 2016 22:20
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

3 participants