Skip to content

PR Group 4 - Liban & Kaalid #12

Open
brittLiban wants to merge 23 commits intokellerflint:mainfrom
brittLiban:main
Open

PR Group 4 - Liban & Kaalid #12
brittLiban wants to merge 23 commits intokellerflint:mainfrom
brittLiban:main

Conversation

@brittLiban
Copy link
Copy Markdown

Testing Setup

  • Added Jest unit tests for the main frontend components.
  • Set up Jest with Next.js, jsdom, and extended matchers.
  • Added backend integration tests using Jest, Supertest, and a MySQL test environment.
  • Updated the README with clear instructions for running all tests.

Cypress E2E Tests

  • Added Cypress and all necessary configuration.
  • Created tests for the key user flows:
    • Home page navigation
    • Creating patterns
    • Deleting patterns
    • Viewing patterns

kellerflint and others added 23 commits November 10, 2025 18:38
Added healthcheck for the database service and modified the dependency condition for the app service.
…d up. Essentially it was a result of the codebase manually creatting the DB with a user that didn’t have those permissions once the container had already initialized it. We dropped that extra CREATE step, pointed Docker and db.js at the environment-managed MySQL setup, and added an npm start script to package.json so the container boots

essentially now it runs perfectly when starting up docker compose up build :)
…d model tests for getAllPatterns function. 1 simple test good to go.
- Updated package.json to include testing dependencies:
- Created a new test file test the rendering of the NavBar component,
- Added jest.config.mjs to configure Jest with Next.js support and set the test environment to jsdom.
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.

3 participants