Use 'Unnamed Manifest' as catchall bucket
This commit is contained in:
@@ -113,7 +113,7 @@ test('groups dependencies with empty manifest paths together', () => {
|
||||
summary.addScannedDependencies(changesWithEmptyManifests)
|
||||
const text = core.summary.stringify()
|
||||
|
||||
expect(text).toContain('<summary> </summary>')
|
||||
expect(text).toContain('<summary>Unnamed Manifest</summary>')
|
||||
expect(text).toContain('castore')
|
||||
expect(text).toContain('connection')
|
||||
expect(text).toContain('<summary>python/dist-info/METADATA</summary>')
|
||||
|
||||
+4
-4
@@ -976,9 +976,9 @@ function groupDependenciesByManifest(changes) {
|
||||
var _a;
|
||||
const dependencies = new Map();
|
||||
for (const change of changes) {
|
||||
// If the manifest is null or empty, use a space as the key to avoid
|
||||
// If the manifest is null or empty, give it a name now to avoid
|
||||
// breaking the HTML rendering later
|
||||
const manifestName = change.manifest || ' ';
|
||||
const manifestName = change.manifest || 'Unnamed Manifest';
|
||||
if (dependencies.get(manifestName) === undefined) {
|
||||
dependencies.set(manifestName, []);
|
||||
}
|
||||
@@ -45235,9 +45235,9 @@ function groupDependenciesByManifest(changes) {
|
||||
var _a;
|
||||
const dependencies = new Map();
|
||||
for (const change of changes) {
|
||||
// If the manifest is null or empty, use a space as the key to avoid
|
||||
// If the manifest is null or empty, give it a name now to avoid
|
||||
// breaking the HTML rendering later
|
||||
const manifestName = change.manifest || ' ';
|
||||
const manifestName = change.manifest || 'Unnamed Manifest';
|
||||
if (dependencies.get(manifestName) === undefined) {
|
||||
dependencies.set(manifestName, []);
|
||||
}
|
||||
|
||||
+1
-1
File diff suppressed because one or more lines are too long
+2
-2
@@ -8,9 +8,9 @@ export function groupDependenciesByManifest(
|
||||
): Map<string, Changes> {
|
||||
const dependencies: Map<string, Changes> = new Map()
|
||||
for (const change of changes) {
|
||||
// If the manifest is null or empty, use a space as the key to avoid
|
||||
// If the manifest is null or empty, give it a name now to avoid
|
||||
// breaking the HTML rendering later
|
||||
const manifestName = change.manifest || ' '
|
||||
const manifestName = change.manifest || 'Unnamed Manifest'
|
||||
|
||||
if (dependencies.get(manifestName) === undefined) {
|
||||
dependencies.set(manifestName, [])
|
||||
|
||||
Reference in New Issue
Block a user