Text validation of URI with a surrogate pair

[artem-zakharchenko]

Previous implementation of Gavel contained a dedicated unit test for ensuring URIs with surrogate pairs in them are validated correctly by TextDiff:

gavel.js/test/unit/validators/text-diff-test.js

Lines 54 to 70 in 96f4092

	describe('when i create new instance of validator with surrogate pair in data', () => {
	validator = null;
	it('should not throw exception', () => {
	const fn = () => {
	validator = new TextDiff('text1\uD800', '\uD800text1');
	};
	assert.doesNotThrow(fn);
	});
	describe('when I run validate', () => {
	it('should not throw exception', () => {
	const fn = () => validator.validate();
	assert.doesNotThrow(fn);
	});
	});
	});

Implementation:

gavel.js/lib/validators/text-diff.js

Lines 27 to 31 in 96f4092

	const sanitizeSurrogatePairs = (data) => {
	return data
	.replace(/[\uD800-\uDBFF]/g, '')
	.replace(/[\uDC00-\uDFFF]/g, '');
	};

Originally introduced in dac0eff.

During the API redesign we have discovered that removing the surrogate pairs handling still gives a passing test suite, so I'm removing the tests and implementation until more details are known about the intended behavior.

[honzajavorek]

@tu1ly I know this is ancient history, but do you happen to remember the original context for this?