Feature: Automated Testing & CI/CD Pipeline

[Shravanthi20]

PR: Feat: Automated Testing & CI/CD Pipeline

📌 Linked Issue

• Connected to Feat: Setup Automated Testing Pipeline (Jest + GitHub Actions) #32

---

🛠 Changes Made

• Added: Installed jest and supertest for backend API testing.
• Added: Created .github/workflows/node.js.yml for automated CI/CD on Push/PR.
• Added: New test files backend/tests/health.test.js and backend/tests/property.test.js.
• Updated: Modified backend/package.json to include the test script.
• Updated: Refactored backend/app.js and backend/server.js to decouple app definition from server startup (enables testing).
• Fixed: Removed package-lock.json from .gitignore to ensure consistent CI builds.

---

🧪 Testing

• Ran unit tests (npm test)
• Tested manually (describe below):
  • Test case 1: Ran npm test locally in backend/ directory -> All passed.
  • Test case 2: Pushed to GitHub -> Verified Action "Node.js CI" passed successfully.

---

📸 UI Changes (if applicable)

Before	After
N/A	N/A

(Backend changes only)

---

📝 Documentation Updates

• Updated README/docs
• Added code comments (in test files)
• Removed package-lock.json from .gitignore to make it public

---

✅ Checklist

• Created a new branch for PR
• Have stared the repository
• Follows JavaScript Styleguide
• No console warnings/errors
• Commit messages follow Git Guidelines

💡 Additional Notes (If any)

• This PR introduces a pattern for integration testing with MongoDB.
• The CI pipeline spins up a dedicated MongoDB container for isolation.

[netlify]

✅ Deploy Preview for smartrentsystem ready!

Built without sensitive environment variables

Name	Link
🔨 Latest commit	ee39d9c
🔍 Latest deploy log	https://app.netlify.com/projects/smartrentsystem/deploys/69620e3b2b10040008aff13c
😎 Deploy Preview	https://deploy-preview-53--smartrentsystem.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[Shravanthi20]

@hitesh-kumar123 could you pls verify and merge it

[hitesh-kumar123]

Thanks for the detailed PR — the testing & CI setup is a solid improvement 👍

Before merging, a few blocking issues need to be addressed:

server.js has a duplicate require("./app") — please keep only the exported app import.

MongoDB connection logic was changed; please ensure safe fallback handling when env vars are missing.

Please re-check backend/package.json — the current diff suggests duplicate keys / missing commas.

Clarify whether /api/properties/test is a test-only endpoint or production route.

Large lockfile changes (frontend + backend) make review difficult — consider isolating them if possible.

Once these are fixed, I’ll be happy to approve and merge.

[Shravanthi20]

@hitesh-kumar123 Sorry for the trouble! i have made the necessary changes, i hope it is fine now.

[promiseer]

@Shravanthi20 check comments and push your changes

[Shravanthi20]

@hitesh-kumar123 @promiseer could you pls check, if the PR can be merged.