Test spreadsheet 1641

[Arohasina]

Issue Description

Fixes #1641
-The spreadsheet integration tests in test_spreadsheet.py were broken after recent changes to the volunteer reporting logic. Several functions were renamed, return formats were updated, and additional event filtering logic was introduced, causing the existing tests to fail or become outdated. This PR updates the test suite to correctly reflect the current reporting behavior and restores full test coverage for the spreadsheet generation pipeline.

Changes

-Updated spreadsheet integration tests to match the current volunteerSpreadsheet.py
-Refactored tests to unpack (columns, rows) results consistently across reporting functions.
-Aligned test data with new event filtering requirements, including isService, isCanceled, and deletionDate.
-Replaced old test targets with current equivalents such as repeatParticipants, repeatParticipantsPerProgram, totalHours, and totalHoursByProgram.
-Improved test coverage for spreadsheet-related reporting functions, including retention rates, volunteer summaries, program totals, demographic breakdowns, and term exports.
-Verified spreadsheet generation runs successfully for multiple academic years using the updated test dataset.

Testing

• git checkout test_spreadsheet_1641
• Run the full spreadsheet test suite locally with pytest tests/code/test_spreadsheet.py -vv -s

[Kafui123]

All my comments have been addressed. Looks good to me.

[Meatchema]

This result occured both before and after I tried a database reset from backup
Besides run this instruction was there anything else I needed to do?: pytest tests/code/test_spreadsheet.py -vv -s

[Arohasina]

Hi @Meatchema, I think it was caused by the duplicate primary key values in the test fixtures. I just fixed it. Could you try running it again?

This result occured both before and after I tried a database reset from backup Besides run this instruction was there anything else I needed to do?: pytest tests/code/test_spreadsheet.py -vv -s

[github-actions]

View Code Coverage

[Meatchema]

It works now: