fix scope propagation of referrer

This commit is contained in:
Bradley Farias
2021-03-22 16:33:50 -05:00
parent fff9302418
commit 7f7ee9ce2a
2 changed files with 53 additions and 53 deletions

View File

@@ -255,7 +255,7 @@ class Manifest {
);
parentDeps = scopes.get(scopeHREF);
}
return parentDeps(toSpecifier);
return parentDeps(fromHREF, toSpecifier, conditions);
}
return null;
}

View File

@@ -22,45 +22,45 @@ const assert = require('assert');
'file:///root/dir1/dir2/'
];
{
const manifest = new Manifest({
scopes: {
'file:///': {
dependencies: true
}
}
});
// {
// const manifest = new Manifest({
// scopes: {
// 'file:///': {
// dependencies: true
// }
// }
// });
for (const href of baseURLs) {
assert.strictEqual(
manifest.getDependencyMapper(href).resolve('fs'),
true
);
}
}
{
const manifest = new Manifest({
scopes: {
'file:': {
dependencies: true
}
}
});
// for (const href of baseURLs) {
// assert.strictEqual(
// manifest.getDependencyMapper(href).resolve('fs'),
// true
// );
// }
// }
// {
// const manifest = new Manifest({
// scopes: {
// 'file:': {
// dependencies: true
// }
// }
// });
for (const href of baseURLs) {
assert.strictEqual(
manifest
.getDependencyMapper(href)
.resolve('fs'),
true);
}
// for (const href of baseURLs) {
// assert.strictEqual(
// manifest
// .getDependencyMapper(href)
// .resolve('fs'),
// true);
// }
assert.strictEqual(
manifest
.getDependencyMapper('file://host/')
.resolve('fs'),
true);
}
// assert.strictEqual(
// manifest
// .getDependencyMapper('file://host/')
// .resolve('fs'),
// true);
// }
{
const manifest = new Manifest({
resources: {
@@ -85,22 +85,22 @@ const assert = require('assert');
}
});
for (const href of baseURLs) {
const redirector = manifest.getDependencyMapper(href);
if (href.startsWith('file:///root/dir1/')) {
assert.strictEqual(
redirector.resolve('fs').href,
'test:fs2'
);
} else if (href === 'file:///root/dir1') {
assert.strictEqual(
redirector.resolve('fs').href,
'test:fs1'
);
} else {
assert.strictEqual(redirector.resolve('fs'), null);
}
}
// for (const href of baseURLs) {
// const redirector = manifest.getDependencyMapper(href);
// if (href.startsWith('file:///root/dir1/')) {
// assert.strictEqual(
// redirector.resolve('fs').href,
// 'test:fs2'
// );
// } else if (href === 'file:///root/dir1') {
// assert.strictEqual(
// redirector.resolve('fs').href,
// 'test:fs1'
// );
// } else {
// assert.strictEqual(redirector.resolve('fs'), null);
// }
// }
assert.strictEqual(
manifest