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

Special symbols in test run name cause search failure #78

Closed
okainov opened this Issue Oct 27, 2017 · 2 comments

Comments

Projects
None yet
2 participants
@okainov
Contributor

okainov commented Oct 27, 2017

Description of problem

Looks like special symbols in test run name is not being escaped properly

Component (web, API, etc)

web

Version or commit hash (if applicable)

04ee633

How often reproducible

Always

Steps to Reproduce

  1. Create Test run with "\" symbols in the name i.e. "Test Product with Windows\Chrome"
  2. Try to search this test run by product
  3. Make sure error will appear
    image

Actual results

Error appears

Expected results

Test run should be found

Additional info

Maybe the same error exists for other fields as well

@atodorov

This comment has been minimized.

Show comment
Hide comment
@atodorov

atodorov Oct 27, 2017

Member

Confirmed the problem, however the quotes are not the issue, the backslash character is! As a workaround you can use forward slash (/) or something else as a delimiter.

Member

atodorov commented Oct 27, 2017

Confirmed the problem, however the quotes are not the issue, the backslash character is! As a workaround you can use forward slash (/) or something else as a delimiter.

@okainov

This comment has been minimized.

Show comment
Hide comment
@okainov

okainov Oct 28, 2017

Contributor

Yes, it was a GitHub bug. In my initial text there was quote-slash-quote symbols, but slash disappeared for some reason. Now I've updated the text

Contributor

okainov commented Oct 28, 2017

Yes, it was a GitHub bug. In my initial text there was quote-slash-quote symbols, but slash disappeared for some reason. Now I've updated the text

@atodorov atodorov closed this Nov 2, 2017

atodorov added a commit that referenced this issue Nov 2, 2017

atodorov added a commit that referenced this issue Nov 2, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment