Refactor tooltip test to remove void operator and improve clarity

Swyamk · Swyamk · commit 3fc4e98794ea · 2025-12-01T15:00:20.000+05:30
diff --git a/frontend/__tests__/unit/components/ContributionHeatmap.test.tsx b/frontend/__tests__/unit/components/ContributionHeatmap.test.tsx
@@ -27,9 +27,10 @@ jest.mock('react-apexcharts', () => {
             w: { config: { series: mockSeries } },
           })
           if (typeof result === 'string') {
-            const hasLightMode = result.includes('#FFFFFF') || result.includes('#E5E7EB')
-            const hasDarkMode = result.includes('#1F2937') || result.includes('#374151')
-            void (hasLightMode || hasDarkMode)
+            result.includes('#FFFFFF')
+            result.includes('#E5E7EB')
+            result.includes('#1F2937')
+            result.includes('#374151')
           }
         }
         tooltip.custom({

Original file line number	Diff line number	Diff line change
`@@ -27,9 +27,10 @@ jest.mock('react-apexcharts', () => {`
`27`	`27`	`w: { config: { series: mockSeries } },`
`28`	`28`	`})`
`29`	`29`	`if (typeof result === 'string') {`
`30`		`- const hasLightMode = result.includes('#FFFFFF') \|\| result.includes('#E5E7EB')`
`31`		`- const hasDarkMode = result.includes('#1F2937') \|\| result.includes('#374151')`
`32`		`- void (hasLightMode \|\| hasDarkMode)`
	`30`	`+ result.includes('#FFFFFF')`
	`31`	`+ result.includes('#E5E7EB')`
	`32`	`+ result.includes('#1F2937')`
	`33`	`+ result.includes('#374151')`
`33`	`34`	`}`
`34`	`35`	`}`
`35`	`36`	`tooltip.custom({`