android-test icon indicating copy to clipboard operation
android-test copied to clipboard

Adds nbsp highlighting for text matchers report

Open Kutashov opened this issue 3 years ago • 4 comments

Overview

Currently, you can see errors like "expected: some string; was: some string" while working with strings that contain nbsp symbols. The reason being is an nsbp and a regular space-symbol are displayed equally. Apparently, this message looks confusing, it takes time to find out the root cause. It is even a bigger problem when there is multiple nbsp in string, debug time grows up dramatically.

Proposed Changes

I like the way the AS handles this, we can do it the same way. image

Kutashov avatar Nov 04 '21 11:11 Kutashov

@brettchabot Hi, there! Here is a copybara check, which got failed. Would you mind helping me, how could I fix this error?

Kutashov avatar Nov 12 '21 16:11 Kutashov

@TWiStErRob Hi! Since we got no reply from @adazh for long time, shall we merge this version?

Kutashov avatar Jun 15 '22 20:06 Kutashov

I'm not Google, just following the repo.

TWiStErRob avatar Jun 15 '22 21:06 TWiStErRob

Sorry for the delay! Left a couple of comments.

adazh avatar Jun 16 '22 22:06 adazh