ryanmwangi
/
CalMerger
1
0
Fork 1
Code Issues 2 Pull Requests Releases Activity

fix: Improve cleanup process in test suite 

- Ensured the server is properly closed before executing cleanup.
- Added a delay to allow for the release of any lingering file handles.
- Enhanced the cleanup logic to safely remove the merged calendars directory after tests.
- Prevented potential EBUSY errors during directory removal.
This commit is contained in:
Ryan Mwangi 2024-11-06 15:23:30 +03:00
parent f5ff331f2f
commit 8a522377c8
1 changed files with 12 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
calendar.test.js
View File

@ -17,8 +17,10 @@ describe('Calendar Merging API', () => {
beforeAll(async () => { beforeAll(async () => {
// Change the working directory to the test-specific directory // Change the working directory to the test-specific directory
process.chdir(path.join(__dirname, 'temp_test_calendar')); process.chdir(path.join(__dirname, 'temp_test_calendar'));
// Start the server // Start the server
server = app.listen(0); server = app.listen(0);
// Ensure the test merged calendars directory exists // Ensure the test merged calendars directory exists
if (!fs.existsSync(TEST_MERGED_CALENDARS_DIR)) { if (!fs.existsSync(TEST_MERGED_CALENDARS_DIR)) {
fs.mkdirSync(TEST_MERGED_CALENDARS_DIR, { recursive: true }); fs.mkdirSync(TEST_MERGED_CALENDARS_DIR, { recursive: true });
@ -26,13 +28,16 @@ describe('Calendar Merging API', () => {
}); });
afterAll(async () => { afterAll(async () => {
// // Clean up the merged calendars directory after tests // Ensure the server is closed before cleanup
// if (fs.existsSync(TEST_MERGED_CALENDARS_DIR)) {
// fs.rmSync(TEST_MERGED_CALENDARS_DIR, { recursive: true, force: true });
// }
// Close the server
await new Promise(resolve => server.close(resolve)); await new Promise(resolve => server.close(resolve));
// Optional: Add a delay to ensure all handles are released
await new Promise(resolve => setTimeout(resolve, 100));
// Clean up the merged calendars directory after tests
if (fs.existsSync(TEST_MERGED_CALENDARS_DIR)) {
fs.rmSync(TEST_MERGED_CALENDARS_DIR, { recursive: true, force: true });
}
}); });
const loadCalendarFile = (filename) => { const loadCalendarFile = (filename) => {