test(surveys): add unit tests for skipSurvey() function

src/tests/lib/surveysUtils.test.js

@@ -6,15 +6,16 @@ import {
 	getMapSurvey,
 	submitHeroQuestion,
 	updateSurveyResponse,
-	getPrioritySurvey
+	getPrioritySurvey,
+	skipSurvey
 } from '../../lib/Surveys/surveyUtils';
 
 beforeEach(() => {
 	let store = {};
 	vi.stubGlobal('localStorage', {
 		getItem: vi.fn((key) => store[key] || null),
 		setItem: vi.fn((key, value) => {
-			store[key] = value;
+			store[key] = String(value);
 		}),
 		removeItem: vi.fn((key) => {
 			delete store[key];
@@ -318,3 +319,35 @@ describe('Survey Visibility and Multiple Responses', () => {
 		expect(selectedSurvey).toEqual(surveys[2]);
 	});
 });
+
+describe('skipSurvey', () => {
+	it('should set only _skipped_timestamp for recurring surveys', () => {
+		const survey = { id: 1, allows_multiple_responses: true, always_visible: true };
+
+		skipSurvey(survey);
+
+		expect(localStorage.getItem('survey_1_skipped_timestamp')).not.toBeNull();
+		expect(localStorage.getItem('survey_1_skipped')).toBeNull();
+	});
+
+	it('should set only _skipped for one-time surveys', () => {
+		const survey = { id: 2, allows_multiple_responses: false, always_visible: false };
+
+		skipSurvey(survey);
+
+		expect(localStorage.getItem('survey_2_skipped')).toBe('true');
+		expect(localStorage.getItem('survey_2_skipped_timestamp')).toBeNull();
+	});
+
+	it('should remove stale _skipped flag when skipping a recurring survey', () => {
+		const survey = { id: 3, allows_multiple_responses: true, always_visible: true };
+
+		localStorage.setItem('survey_3_skipped', 'true');
+		expect(localStorage.getItem('survey_3_skipped')).toBe('true');
+
+		skipSurvey(survey);
+
+		expect(localStorage.getItem('survey_3_skipped')).toBeNull();
+		expect(localStorage.getItem('survey_3_skipped_timestamp')).not.toBeNull();
+	});
+});