jonbrunt / ai-portfolio

Personal AI Engineering Portfolio Page

https://ai-portfolio-henna.vercel.app

MIT License

0 stars 0 forks source link

LOCAL UNIT TESTING AND CI PIPELINE #17

Closed jonbrunt closed 7 months ago

jonbrunt commented 7 months ago

Feature: Pre-Push Hook (Local) and Branch Protection (CI Pipeline)

Description

This PR introduces pre-push unit tests (local) and PR/merge protections for the main branch (linting and unit tests). Will be made more robust with integration testing, e2e testing, and CD shortly.

Changes

Frontend:

Added YAML file for CI
Added branch protection for main branch in GH Actions

Testing

Manual testing was conducted in the dev server/browser, and more importantly, in a mock test in the branch to ensure the actions are occurring.
Unit tests all pass
Integration tests are coming soon

Notes

This addresses current needs for CI and preparation for CD. Robust coverage is now in place on the local side (pre-commit hook for linting and pre-push hook for testing). We might consider moving linting to pre-push, to avoid issues with items not staged for commit that are in changes. Enforces linting/testing for all branches (feature, issue, hotfix, etc). Enforces linting and unit tests before merging.

Checklist

[x] Code compiles correctly
[x] All tests passing
[x] No merge conflicts

Reviewers

Requesting review from @jonbrunt

Tags

#enhancement

jonbrunt commented 7 months ago

We are building excellent coverage on the local side and the remote/merge side. Let's continue developing this pipeline as we finish the last three major components, add unit tests to the page and layout files, and incorporate integration and e2e tests. From there we will make an initial deployment and configure/adjust DevOps settings accordingly.