refactored lines 46 to 70 to have helper functions and simplify the code, src/messaging/rooms.js

[safa-amin]

This resolves issue #12

I refactored the 'modifyRoomData' function to be more efficient. In my new refactored code, the function essentially processes an array of room data objects, updating various fields based on certain conditions. The function goes through the integer fields, checks the room name (leaves), determines if the room is public and updates the notification settings if required.

I also added helper functions 'shouldUpdateNotificationSetting' and 'shouldUpdateGroups' to simplify the original code.

[coveralls]

Pull Request Test Coverage Report for Build 10718553140

Details

• 14 of 16 (87.5%) changed or added relevant lines in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+0.002%) to 82.673%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
src/messaging/rooms.js	14	16	87.5%

Totals
Change from base Build 10550029021:	0.002%
Covered Lines:	22327
Relevant Lines:	25586

---

💛 - Coveralls

[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 SonarCloud