Skip to content

Commit 85b5936

Browse files
authored
Merge pull request #41 from adobe/cluster-fixes
Fix Cluster Facet
2 parents f10bf9d + fcaab3a commit 85b5936

File tree

2 files changed

+5
-1
lines changed

2 files changed

+5
-1
lines changed

distiller.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -435,7 +435,7 @@ export class DataChunks {
435435
this.addFacet(facetName, (bundle) => {
436436
const facetMatch = facetValues.find((f) => f.entries.some((e) => e.id === bundle.id));
437437
const clusters = producer(facetMatch.value);
438-
return [facetMatch, ...clusters.filter((cluster) => sortedClusters.includes(cluster))];
438+
return [facetMatch.value, ...clusters.filter((cluster) => sortedClusters.includes(cluster))];
439439
});
440440
}
441441

test/distiller.test.js

+4
Original file line numberDiff line numberDiff line change
@@ -945,6 +945,10 @@ describe('DataChunks.addClusterFacet()', () => {
945945
const { facets } = d;
946946

947947
assert.ok(facets.urlCluster.length > 0);
948+
assert.notEqual(facets.urlCluster[0].value, '[object Object]');
949+
assert.equal(facets.urlCluster[0].value, '/developer');
950+
assert.equal(facets.urlCluster[1].value, 'https://www.aem.live/home');
951+
assert.equal(facets.urlCluster[2].value, 'https://www.aem.live/developer/tutorial');
948952
});
949953

950954
it('should handle empty facet values gracefully', () => {

0 commit comments

Comments
 (0)