Feature/add missing unit tests

.github/workflows/nextjs.yml

@@ -48,105 +48,11 @@ jobs:
               - '**/*.{css,scss}'
               - 'tailwind.config.{js,ts}'
 
-  # Lint and format job
-  lint:
-    runs-on: ubuntu-latest
-    needs: changes
-    if: ${{ needs.changes.outputs.code == 'true' || github.event_name == 'pull_request' }}
-    # Grant specific permissions needed for lint job
-    permissions:
-      checks: write
-      contents: write
-      pull-requests: write
-    steps:
-      - name: Checkout
-        uses: actions/checkout@v4
-        with:
-          # Make sure the actual branch is checked out when running on pull requests
-          ref: ${{ github.head_ref }}
-          # Full git history is needed for proper linting
-          fetch-depth: 0
-
-      - name: Prettify code
-        uses: creyD/[email protected]
-        with:
-          prettier_options: --write --tab-width=2 --use-tabs .
-          # Ensure Prettier config is respected or created if missing
-          prettier_plugins: prettier-plugin-tailwindcss
-          github_token: ${{ secrets.GITHUB_TOKEN }}
-          commit_message: "🧹 Automated code formatting"
-          # Skip committing on main branch to avoid repository rule violations
-          same_commit: true
-          # Only commit changes when not on main branch
-          commit_options: ${{ github.ref != 'refs/heads/main' && '--no-verify' || '' }}
-
-      - name: Detect package manager
-        id: detect-package-manager
-        run: |
-          if [ -f "${{ github.workspace }}/yarn.lock" ]; then
-            echo "manager=yarn" >> $GITHUB_OUTPUT
-            echo "command=install" >> $GITHUB_OUTPUT
-            echo "runner=yarn" >> $GITHUB_OUTPUT
-            exit 0
-          elif [ -f "${{ github.workspace }}/package.json" ]; then
-            echo "manager=npm" >> $GITHUB_OUTPUT
-            echo "command=ci" >> $GITHUB_OUTPUT
-            echo "runner=npx --no-install" >> $GITHUB_OUTPUT
-            exit 0
-          else
-            echo "Unable to determine package manager"
-            exit 1
-          fi
-
-      - name: Setup Node
-        uses: actions/setup-node@v4
-        with:
-          node-version: "20"
-          cache: ${{ steps.detect-package-manager.outputs.manager }}
-
-      # Optimized caching strategy
-      - name: Cache node_modules
-        uses: actions/cache@v4
-        id: node-modules-cache
-        with:
-          path: node_modules
-          key: ${{ runner.os }}-${{ steps.detect-package-manager.outputs.manager }}-${{ hashFiles('**/package-lock.json', '**/yarn.lock') }}-v2
-
-      - name: Install dependencies
-        if: steps.node-modules-cache.outputs.cache-hit != 'true'
-        run: ${{ steps.detect-package-manager.outputs.manager }} ${{ steps.detect-package-manager.outputs.command }}
-
-      - name: Install additional dependencies for linting
-        run: |
-          ${{ steps.detect-package-manager.outputs.manager }} add --save-dev eslint-plugin-prettier prettier-plugin-tailwindcss
-
-      # Enhanced linting strategy - auto-fix but don't fail on warnings
-      - name: Run ESLint with auto-fix
-        id: eslint-fix
-        continue-on-error: true
-        run: |
-          echo "Running ESLint with auto-fix..."
-          ${{ steps.detect-package-manager.outputs.runner }} next lint --fix
-
-      # Check for errors, but allow warnings
-      - name: Check for ESLint errors
-        run: |
-          echo "Checking for ESLint errors (allowing warnings)..."
-          ${{ steps.detect-package-manager.outputs.runner }} next lint --quiet || {
-            echo "::error::ESLint found errors that must be fixed. Please review the errors above.";
-            exit 1;
-          }
-
-      # Report warnings but don't fail the build
-      - name: Report ESLint warnings
-        run: |
-          echo "Checking for ESLint warnings (won't fail build)..."
-          ${{ steps.detect-package-manager.outputs.runner }} next lint || echo "::warning::ESLint found warnings, but continuing build. Please consider fixing these warnings when possible."
-
-  # Build job - preserved your logic with optimizations
+  # Build job
   build:
     runs-on: ubuntu-latest
-    needs: [lint]
+    needs: [changes]
+    if: ${{ needs.changes.outputs.code == 'true' || github.event_name == 'pull_request' }}
     timeout-minutes: 15
     # Grant specific permissions needed for build job
     permissions:
@@ -179,7 +85,7 @@ jobs:
           node-version: "20"
           cache: ${{ steps.detect-package-manager.outputs.manager }}
 
-      # Reusing improved caching from above
+      # Optimized caching strategy
       - name: Cache node_modules
         uses: actions/cache@v4
         id: node-modules-cache
@@ -202,7 +108,6 @@ jobs:
         if: steps.node-modules-cache.outputs.cache-hit != 'true'
         run: ${{ steps.detect-package-manager.outputs.manager }} ${{ steps.detect-package-manager.outputs.command }} --no-audit --force
 
-      # Preserved from original workflow
       - name: Add Amplify outputs file
         run: touch amplify_outputs.json
 
@@ -261,7 +166,6 @@ jobs:
         if: steps.node-modules-cache.outputs.cache-hit != 'true'
         run: ${{ steps.detect-package-manager.outputs.manager }} ${{ steps.detect-package-manager.outputs.command }} --no-audit --force
 
-      # Added Amplify file creation steps
       - name: Add Amplify outputs file
         run: touch amplify_outputs.json
 

.gitignore

@@ -63,3 +63,7 @@ amplifyconfiguration*
 # Additional build caches
 .swc/
 .turbo/
+
+# repomix
+/repomix-output.xml
+/repomix.config.json

.husky/pre-commit

@@ -1,16 +1,9 @@
-#!/bin/sh
-#. "$(dirname "$0")/_/husky.sh"
+# Run TypeScript check (excluding problematic test files)
+echo "🧪 Checking TypeScript..."
+npx tsc --noEmit
 
-echo "🧪 Running TypeScript type check..."
-# Run TypeScript but don't block commit on errors
-#npx tsc --noEmit || echo "⚠️ TypeScript check found issues. Consider fixing them, but proceeding with commit."
+# Run Prettier + ESLint on staged files
+echo "🧹 Formatting and linting staged files..."
+npx lint-staged
 
-# Run Prettier to format code
-echo "🧹 Running Prettier..."
-#npx prettier --write --tab-width=2 --use-tabs .
-
-# Run ESLint but don't block commit on errors
-echo "🛠️ Running ESLint..."
-#npx next lint --fix || echo "⚠️ ESLint found issues. Consider fixing them, but proceeding with commit."
-
-echo "✅ Pre-commit checks completed! (Warnings were allowed to proceed)"
+echo "✅ Pre-commit checks passed!"

.prettierrc

@@ -4,6 +4,7 @@
 	"useTabs": true,
 	"semi": true,
 	"singleQuote": false,
+	"jsxSingleQuote": false,
 	"trailingComma": "all",
 	"bracketSpacing": true,
 	"bracketSameLine": false,

__mocks__/jquery-ui.js

@@ -0,0 +1,8 @@
+// Mock implementation of jQuery UI for testing
+module.exports = {
+	ui: {
+		widgets: {
+			autocomplete: {},
+		},
+	},
+};

__mocks__/jquery-ui/ui/widgets/autocomplete.js

@@ -0,0 +1,2 @@
+// Mock implementation of jQuery UI autocomplete widget for testing
+module.exports = {};

__mocks__/jquery.js

@@ -0,0 +1,17 @@
+// Mock implementation of jQuery for testing
+const mockAutocomplete = jest.fn();
+const mockOn = jest.fn();
+const mockVal = jest.fn().mockReturnValue("");
+
+const mockJQuery = jest.fn((selector) => ({
+	autocomplete: mockAutocomplete,
+	on: mockOn,
+	val: mockVal,
+}));
+
+// Expose the mock functions for test access
+mockJQuery.mockAutocomplete = mockAutocomplete;
+mockJQuery.mockOn = mockOn;
+mockJQuery.mockVal = mockVal;
+
+module.exports = mockJQuery;

__tests__/__mocks__/mockRegistry.tsx

@@ -86,21 +86,46 @@ export const MockNavBar = ({
 /**
  * Mock for the Footer component
  */
-export const MockFooter = () => {
+export function MockFooter() {
 	return (
-		<footer data-testid="footer" role="contentinfo" aria-label="Site Footer">
-			<div data-testid="footer-nav" role="navigation" aria-label="Footer Navigation">
-				<a href="/privacy" role="link" aria-label="Privacy Policy">
-					Privacy Policy
-				</a>
-				<a href="/terms" role="link" aria-label="Terms of Service">
-					Terms of Service
-				</a>
+		<div className="footer" data-testid="mock-footer">
+			<div className="flex flex-col items-center justify-center w-full min-h-fit">
+				<div className="flex flex-col md:flex-row items-start justify-left w-full">
+					<div className="logo">
+						<img src="/mock-logo.png" alt="HackRPI Logo" data-testid="footer-logo" />
+					</div>
+					<div className="info">
+						<h2 data-testid="footer-title">
+							<span className="font-bold">HackRPI</span> Retro V. Modern
+						</h2>
+						<div className="address">
+							<p>Darrin Communications Center @ Rensselaer Polytechnic Institute</p>
+							<p>110 8th St, Troy, NY 12180</p>
+						</div>
+					</div>
+					<div className="links">
+						<div className="registration-link">Register Now</div>
+						<div className="social-links">
+							<a href="#" aria-label="Discord">
+								Discord
+							</a>
+							<a href="#" aria-label="Twitter">
+								Twitter
+							</a>
+							<a href="#" aria-label="Instagram">
+								Instagram
+							</a>
+						</div>
+					</div>
+				</div>
+				<div className="copyright">
+					<p>Made with ❤️ by HackRPI.</p>
+					<p>&copy; 2025 HackRPI</p>
+				</div>
 			</div>
-			<p>© 2025 HackRPI. All rights reserved.</p>
-		</footer>
+		</div>
 	);
-};
+}
 
 /**
  * Mock for the Title component