fix uri-test (do not assume URL work alike cross browser...) microsoft#93362

jrieken · jrieken · commit 49c62cbc8ff8 · 2020-04-07T10:04:30.000+02:00
diff --git a/src/vs/base/test/common/uri.test.ts b/src/vs/base/test/common/uri.test.ts
@@ -538,7 +538,7 @@ suite('URI', () => {
 		assert.throws(() => assertJoined(('foo:'), 'bazz', ''));
 		assert.throws(() => new URL('bazz', 'foo:'));
 		assert.throws(() => assertJoined(('foo://bar'), 'bazz', ''));
-		// assert.throws(() => new URL('bazz', 'foo://bar')); Edge,Chrome => throw, Safari => foo://bar/bazz, Firefox ??
+		// assert.throws(() => new URL('bazz', 'foo://bar')); Edge, Chrome => THROW, Firefox, Safari => foo://bar/bazz
 	});
 
 	test('URI#joinPath (posix)', function () {
@@ -549,8 +549,8 @@ suite('URI', () => {
 		assertJoined(('file://server/share/c:/'), '../../bazz', 'file://server/bazz', false);
 		assertJoined(('file://server/share/c:'), '../../bazz', 'file://server/bazz', false);
 
-		assertJoined(('file://ser/foo/'), '../../bazz', 'file://ser/bazz');
-		assertJoined(('file://ser/foo'), '../../bazz', 'file://ser/bazz');
+		assertJoined(('file://ser/foo/'), '../../bazz', 'file://ser/bazz', false); // Firefox -> Different, Edge, Chrome, Safar -> OK
+		assertJoined(('file://ser/foo'), '../../bazz', 'file://ser/bazz', false); // Firefox -> Different, Edge, Chrome, Safar -> OK
 	});
 
 	test('URI#joinPath (windows)', function () {
