Skip to content

Commit 8d45159

Browse files
committed
tests: Fix tab sync tests by making sure extension:// URLs are ignored
Signed-off-by: Marcel Klehr <[email protected]>
1 parent 36f854c commit 8d45159

File tree

1 file changed

+21
-1
lines changed

1 file changed

+21
-1
lines changed

src/lib/adapters/Caching.ts

Lines changed: 21 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ import {
1212
UnknownMoveTargetError
1313
} from '../../errors/Error'
1414
import { BulkImportResource, ICapabilities, IHashSettings } from '../interfaces/Resource'
15+
import { isTest } from '../isTest'
1516

1617
export default class CachingAdapter implements Adapter, BulkImportResource<TItemLocation> {
1718
protected highestId: number
@@ -43,7 +44,26 @@ export default class CachingAdapter implements Adapter, BulkImportResource<TItem
4344
return false
4445
}
4546
try {
46-
return Boolean(['https:', 'http:', 'ftp:', 'data:', 'javascript:', 'file:', 'chrome:', 'edge:', 'about:', 'chrome-extension:', 'moz-extension:'].includes(
47+
let schemes = [
48+
'https:',
49+
'http:',
50+
'ftp:',
51+
'data:',
52+
'javascript:',
53+
'file:',
54+
'chrome:',
55+
'edge:',
56+
'about:'
57+
]
58+
59+
if (!isTest) {
60+
schemes = schemes.concat([
61+
'chrome-extension:',
62+
'moz-extension:'
63+
])
64+
}
65+
66+
return Boolean(schemes.includes(
4767
new URL(bm.url).protocol
4868
))
4969
} catch (e) {

0 commit comments

Comments
 (0)