Skip to content

63 task migrate the database into new arcitecture#64

Merged
Aravinda-HWK merged 6 commits intomainfrom
63-task-migrate-the-database-into-new-arcitecture
Oct 17, 2025
Merged

63 task migrate the database into new arcitecture#64
Aravinda-HWK merged 6 commits intomainfrom
63-task-migrate-the-database-into-new-arcitecture

Conversation

@Aravinda-HWK
Copy link
Copy Markdown
Collaborator

📌 Description

This PR is to change the database architecture to the new design.


🔍 Changes Made

  1. Update the sqlite.go file to initialize the database and all the tables according to the new database architecture.
  2. Update all the IMAP commands according to that.
  3. Change the message parser and the saving queries in the delivery service.
  4. Update all the test cases based on the new architecture.

✅ Checklist (Email System)

  • Core IMAP commands tested (LOGIN, CAPABILITY, LIST, SELECT, FETCH, LOGOUT).
  • Authentication is tested.
  • Docker build & run validated.
  • Configuration loading verified for default and custom paths.
  • Persistent storage with Docker volume verified.
  • Error handling and logging verified
  • Documentation updated (README, config samples).

🧪 Testing Instructions

To test the server, use the instructions in the README in the test directory.


📷 Screenshots / Logs (if applicable)


⚠️ Notes for Reviewers

…e insertion

- Updated test files in `select_test.go`, `status_test.go`, and `subscribe_test.go` to replace direct instantiation of `models.ClientState` with `helpers.SetupAuthenticatedState`.
- Simplified message insertion in tests by utilizing `helpers.InsertTestMail` and `helpers.CreateTestUser`.
- Improved readability and maintainability of tests by reducing repetitive code.
- Ensured consistent user and mailbox creation across tests.
@Aravinda-HWK Aravinda-HWK merged commit f173fd1 into main Oct 17, 2025
2 checks passed
@Aravinda-HWK Aravinda-HWK deleted the 63-task-migrate-the-database-into-new-arcitecture branch October 17, 2025 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[TASK] Migrate the database into new arcitecture

2 participants