This repository has been archived by the owner. It is now read-only.

Multiple spaces are reduced to a single space when reporting text comparison failures #1013

Closed
lukeis opened this Issue Mar 2, 2016 · 7 comments

Comments

Projects
None yet
1 participant
@lukeis
Member

lukeis commented Mar 2, 2016

Originally reported on Google Code with ID 1013

What steps will reproduce the problem?

1. Open http://seleniumhq.org
2. Assert title is 'Selenium  web application testing system' - note the extra space
between 'Selenium' and 'web' that is not present in the actual page title.

What is the expected output? What do you see instead?

Expected: Assertion should fail with a message showing the expected and actual values,
for example:

"[error] Actual value 'Selenium  web application testing system' did not match 'Selenium
web application testing system'"

Ideally we would also highlight the differences, for example:

"[error] Actual value 'Selenium[  ]web application testing system' did not match 'Selenium[
]web application testing system'"

Actual:

The actual error message reduces the multiple spaces to a single space, therefore making
an incorrect statement:

"[error] Actual value 'Selenium web application testing system' did not match 'Selenium
web application testing system'"

What version of the product are you using? On what operating system?

Selenium IDE 1.0.8
OS: Any

Reported by dave.hunt on 2010-11-17 17:42:55

@lukeis lukeis self-assigned this Mar 2, 2016

@lukeis

This comment has been minimized.

Member

lukeis commented Mar 2, 2016

Done! :-)

Reported by Samit.Badle on 2010-11-17 20:49:53


- _Attachment: [patch.txt](https://storage.googleapis.com/google-code-attachments/selenium/issue-1013/comment-1/patch.txt)_
@lukeis

This comment has been minimized.

Member

lukeis commented Mar 2, 2016

checked in. samit's patch was soooo much simpler than what i was trying to do... apparently
loading html in a xul iframe doesn't actually render html entities.

Reported by adamgoucher on 2010-11-17 22:00:51

  • Status changed: fix
@lukeis

This comment has been minimized.

Member

lukeis commented Mar 2, 2016

Fix for the wrap bug spotted by Dave!

Reported by Samit.Badle on 2010-11-19 10:59:38


- _Attachment: [patch2.txt](https://storage.googleapis.com/google-code-attachments/selenium/issue-1013/comment-3/patch2.txt)_
@lukeis

This comment has been minimized.

Member

lukeis commented Mar 2, 2016

Does this 2nd patch still need to be applied? if not can someone close this please?
:)

Reported by theautomatedtester on 2010-12-05 09:58:14

@lukeis

This comment has been minimized.

Member

lukeis commented Mar 2, 2016

Hi,

I see that the patch has already been applied by Adam in r10374. So, I am closing this
issue.

Dave,

If something is still amiss, let me know / reopen the issue.

Cheers,
Samit

Reported by Samit.Badle on 2010-12-06 10:25:57

  • Status changed: Fixed
@lukeis

This comment has been minimized.

Member

lukeis commented Mar 2, 2016

ghbdtn

Reported by mashaalex2010 on 2014-11-16 10:50:21

@lukeis

This comment has been minimized.

Member

lukeis commented Mar 2, 2016

Reported by luke.semerau on 2015-09-17 18:11:37

  • Labels added: Restrict-AddIssueComment-Commit

@lukeis lukeis closed this Mar 2, 2016

@SeleniumHQ SeleniumHQ locked and limited conversation to collaborators Mar 4, 2016

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