Skip to content

Qa/appointment - #153

Merged
dashaaaa21 merged 5 commits into
developfrom
QA/appointment
Mar 8, 2026
Merged

dashaaaa21 merged 5 commits into
developfrom
QA/appointment

Conversation

@dashaaaa21

Copy link
Copy Markdown
Contributor

QA appointments test

Copilot AI review requested due to automatic review settings March 8, 2026 10:57
@HackYourFutures
HackYourFutures temporarily deployed to study-bridge-qa-appoint-pyuap1 March 8, 2026 10:58 Inactive
@HackYourFutures
HackYourFutures temporarily deployed to study-bridge-qa-appoint-pyuap1 March 8, 2026 11:02 Inactive

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds initial QA/Jest coverage for appointment business validation and sets up a TypeScript/ESM Jest environment for the server.

Changes:

  • Extend AppointmentTypeDB with optional “populated” fields used by appointment response formatting (teacherName, studentName, studentProfileImageUrl).
  • Switch server Jest configuration to ts-jest ESM and restrict test discovery to **/__tests__/**/*.test.ts.
  • Add MongoDB in-memory test database setup + helpers and introduce basic smoke tests + appointment business validation/CRUD tests.

Reviewed changes

Copilot reviewed 10 out of 11 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
server/src/db/schemes/types/appointment.types.ts Adds optional “enriched” appointment fields to the DB type.
server/jest.config.js Reconfigures Jest to run TypeScript tests in ESM via ts-jest.
server/tests/tsconfig.json Adds a dedicated TS config for tests.
server/tests/simple.test.ts Adds a basic Jest smoke test suite.
server/tests/setup/database.setup.ts Adds mongodb-memory-server + mongoose lifecycle helpers.
server/tests/helpers/test.helpers.ts Adds factories for student/teacher/appointment and date helpers.
server/tests/appointment/appointment.service.test.ts Adds appointment business validation + AppointmentModel CRUD tests.
Files not reviewed (1)
  • server/package-lock.json: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

Comment thread server/__tests__/tsconfig.json Resolved
Comment thread server/jest.config.js Resolved
@dashaaaa21
dashaaaa21 merged commit ce1a09b into develop Mar 8, 2026
2 checks passed

This branch was previously deployed

1 inactive deployment
study-bridge-qa-appoint-pyuap1 — 5adee04a Deployed Mar 8, 2026 by HackYourFutures
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.

4 participants