Skip to content

Trim white space when flushing assertions#93

Open
gazayas wants to merge 1 commit intomainfrom
fixes/extra-space
Open

Trim white space when flushing assertions#93
gazayas wants to merge 1 commit intomainfrom
fixes/extra-space

Conversation

@gazayas
Copy link
Copy Markdown

@gazayas gazayas commented Jun 23, 2023

Fixes #51.

pry(#<BasicsTest>)> "#{last["action"]} #{last["target"]}#{last["options"]}" + "\n"
=> "assert page.has_content? 'Foo was successfully created.' \n"

Since we're tagging on a new line at the end with a +, just calling strip here should suffice.

@gazayas gazayas changed the title Fix indentation when flushing assertions Trim white space when flushing assertions Jun 23, 2023
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.

Unnecessary space on assertions

1 participant