added test

Author: petermsouzajr

src/google/sheetDataMethods/__mocks__/getSheetInfo.js

@@ -1 +1,14 @@
-export const getTabIdFromWeeklyTitle = jest.fn();
+// src/google/sheetDataMethods/__mocks__/getSheetInfo.js
+// This is the manual mock for ../google/sheetDataMethods/getSheetInfo.js
+
+const mockInstance = jest.fn(() => Promise.resolve([])); // Default implementation
+
+// Add a property to identify this specific mock instance for debugging
+mockInstance.isManualMock = true;
+
+export const getExistingTabTitlesInRange = mockInstance;
+
+// If the original getSheetInfo.js has other named exports,
+// you would define and export them here as well, e.g.:
+// export const anotherFunction = jest.fn();
+// anotherFunction.isManualMock = true; // and mark them too if needed

src/index.test.js

@@ -65,10 +65,42 @@ jest.mock('./index.js', () => {
 // Import after mocking
 import { main } from './index.js';
 
+let mockProcessExit;
+
 describe('main function', () => {
+  beforeAll(() => {
+    mockProcessExit = jest.spyOn(process, 'exit').mockImplementation((code) => {
+      // console.warn(`Test Warning: process.exit(${code}) was called and suppressed.`);
+      // Optionally throw an error here if a test should explicitly fail when exit is called:
+      // throw new Error(`process.exit(${code}) called`);
+    });
+
+    global.shadowConfigDetails = {
+      googleSpreadsheetUrl:
+        'https://docs.google.com/spreadsheets/d/default_mock_id/edit',
+      googleKeyFilePath: 'default/mock/path/to/keyfile.json',
+      testTypes: ['unit', 'integration', 'e2e'],
+      testCategories: ['smoke', 'regression', 'performance'],
+      weeklySummaryStartDay: 'Monday',
+      weeklySummaryEnabled: true,
+      teamNames: ['team1', 'team2'],
+      csvDownloadsPath: './cypress/downloads',
+      // Ensure all fields that might be accessed by constants.js are here
+      // For TEST_DATA, it might need specific fields like testDataPath or similar
+      testDataPath: 'mock/path/to/test-data.json', // Assuming TEST_DATA needs something like this
+      resultsPath: 'mock/path/to/results.json', // Assuming TEST_DATA needs something like this
+    };
+  });
+
+  afterAll(() => {
+    mockProcessExit.mockRestore();
+    delete global.shadowConfigDetails;
+  });
+
   beforeEach(() => {
     // Reset all mocks before each test
     jest.clearAllMocks();
+    // mockProcessExit.mockClear(); // Clear calls to process.exit if needed per test
 
     // Mock console methods
     console.log = jest.fn();

src/sharedMethods/summaryRequired.js

@@ -77,13 +77,64 @@ const isSummaryNeeded = (tabTitles) => {
  * @param {boolean} options.csv - If true, outputs in CSV format.
  * @returns {Promise<boolean>} True if a summary is required, false otherwise.
  */
-export const isSummaryRequired = async ({ csv }) => {
-  if (csv) {
-    return false;
-  }
+export const isSummaryRequired = async (options = {}) => {
+  if (options.csv) return false;
   try {
+    const currentDate = new Date();
+    // console.log('[DEBUG] SUT currentDate:', currentDate.toISOString());
+
+    const lastMonth = getFormattedMonth('lastMonth');
+    // console.log('[DEBUG] SUT lastMonth:', lastMonth);
+
+    const currentMonth = getFormattedMonth();
+    // console.log('[DEBUG] SUT currentMonth:', currentMonth);
+
+    let yearForSummary = getFormattedYear();
+    if (currentMonth === 'Jan' && lastMonth === 'Dec') {
+      yearForSummary = getFormattedYear('lastYear');
+    }
+    // console.log('[DEBUG] SUT yearForSummary:', yearForSummary);
+
+    const targetSummaryTitle = createSummaryTitle(
+      lastMonth,
+      yearForSummary.toString()
+    );
+    // console.log('[DEBUG] SUT targetSummaryTitle:', targetSummaryTitle);
+
     const existingTabTitles = await getExistingTabTitlesInRange();
-    return isSummaryNeeded(existingTabTitles);
+    // console.log('[DEBUG] SUT existingTabTitles raw:', existingTabTitles);
+
+    if (!existingTabTitles) {
+      // console.error('Error: existingTabTitles is undefined');
+      return false;
+    }
+
+    const summaryExists = existingTabTitles.some(
+      (title) =>
+        title.trim().toLowerCase() === targetSummaryTitle.trim().toLowerCase()
+    );
+    // console.log('[DEBUG] SUT summaryExists:', summaryExists);
+
+    if (summaryExists) {
+      return false;
+    }
+
+    const searchPatternMonth = lastMonth;
+    const searchPatternYear = yearForSummary.toString();
+    // console.log('[DEBUG] SUT searchPatternMonth:', searchPatternMonth);
+    // console.log('[DEBUG] SUT searchPatternYear:', searchPatternYear);
+
+    const sheetsFromLastMonthExist = existingTabTitles.some((title) => {
+      const titleIncludesMonth = title.includes(searchPatternMonth);
+      const titleIncludesYear = title.includes(searchPatternYear);
+      return titleIncludesMonth && titleIncludesYear;
+    });
+    // console.log(
+    //   '[DEBUG] SUT sheetsFromLastMonthExist:',
+    //   sheetsFromLastMonthExist
+    // );
+
+    return sheetsFromLastMonthExist;
   } catch (error) {
     console.error('Error in isSummaryRequired:', error);
     return false;
@@ -142,12 +193,19 @@ const isWeeklySummaryNeeded = (tabTitles) => {
       return false;
     }
     const expectedSummaryTitle = createWeeklySummaryTitle();
-    const summaryExists = tabTitles.some(
-      (title) => title.startsWith('Weekly') && title === expectedSummaryTitle
-    );
+    // console.log('[SUT DEBUG] isWeeklySummaryNeeded - expectedSummaryTitle:', expectedSummaryTitle);
+    // console.log('[SUT DEBUG] isWeeklySummaryNeeded - tabTitles:', JSON.stringify(tabTitles));
+
+    const summaryExists = tabTitles.some((title) => {
+      const starts = title.startsWith('Weekly');
+      const matches = title === expectedSummaryTitle;
+      // console.log(`[SUT DEBUG] Checking title: "${title}", startsWeekly: ${starts}, matchesExpected: ${matches}`);
+      return starts && matches;
+    });
+    // console.log('[SUT DEBUG] isWeeklySummaryNeeded - summaryExists:', summaryExists);
     return !summaryExists;
   } catch (error) {
-    console.error('Error in isSummaryNeeded:', error);
+    console.error('Error in isWeeklySummaryNeeded:', error);
     return false;
   }
 };