Linh - Added unit test for popupEditorController

[linh2020]

Description

This PR adds full unit test coverage for the popupEditorController in the backend, ensuring its core functionalities are properly validated. Tests include success and failure paths, permissions handling, and input validation.

Related PRS (if any):

No related PRs

Main changes explained:

1. Added 15 Jest unit test cases covering:

• getAllPopupEditors – success and error handling
• getPopupEditorById – success and failure scenarios
• createPopupEditor – with/without permissions, missing fields, save errors
• updatePopupEditor – with/without permissions, invalid/missing content, save errors

2. Updated popupEditorController.js:

• Improved error handling in updatePopupEditor to catch null or undefined popup objects
• Added error catch block if findById returns no result

How to test:

1. Check out this branch locally.
2. Run npm install if needed.
3. Run the test suite: npm test src/controllers/popupEditorController.spec.js
4. Ensure all 15 test cases pass.

Screenshots or videos of changes:

creenshots or videos of changes:

Note:

• This PR ensures full backend test coverage for the popup editor controller.
• The logic was enhanced for better error clarity.
• Please review and approve so the team can confidently build on top of this controller.

[abdel-lall]

LGTM!

[ManviKishore]

All test cases are passing successfully.

[linkhanthtel]

All test cases passed as expected. Nice work!

[Anusha-Gali]

Hi Linh,

I have reviewed your PR locally and it all the test case pass as per requirement.

Note: Code still points to old Db.js,

[HemanthNidamanuru]

Hi Linh,

I checked out this branch locally and verified the changes. All tests are passing successfully.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[one-community]

Thank you all, merging!