feat: Add Dependabot configuration for GitHub Actions and npm updates; update action versions in workflows

chrisreddington · web-flow · commit 2bdc26995987 · 2025-05-08T16:13:11.000Z
diff --git a/.github/dependabot.yml b/.github/dependabot.yml
@@ -0,0 +1,26 @@
+version: 2
+updates:
+  - package-ecosystem: github-actions
+    directory: /
+    schedule:
+      interval: weekly
+    groups:
+      actions-minor:
+        update-types:
+          - minor
+          - patch
+
+  - package-ecosystem: npm
+    directory: /
+    schedule:
+      interval: weekly
+    groups:
+      npm-development:
+        dependency-type: development
+        update-types:
+          - minor
+          - patch
+      npm-production:
+        dependency-type: production
+        update-types:
+          - patch
diff --git a/.github/workflows/attestations.yml b/.github/workflows/attestations.yml
@@ -16,15 +16,15 @@ jobs:
     steps:
       # Checkout the repository
       - name: Checkout repository
-        uses: actions/checkout@v4
+        uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683
 
       # Install dependencies
       - name: Install dependencies
         run: npm install
 
       # Generate SBOM from the dependencies (scanning the workspace directory)
       - name: Generate SBOM
-        uses: anchore/sbom-action@v0
+        uses: anchore/sbom-action@9f7302141466aa6482940f15371237e9d9f4c34a
         with:
           upload-artifact: false
           upload-release-assets: false
@@ -46,14 +46,14 @@ jobs:
       #     subject-path: "dist.tar.gz"
 
       # Complete an attestation of the SBOM and the build
-      - uses: actions/attest-sbom@v1
+      - uses: actions/attest-sbom@115c3be05ff3974bcbd596578934b3f9ce39bf68
         with:
           subject-path: 'dist.tar.gz'
           sbom-path: 'sbom.spdx.json'
 
       # Publish the SBOM (Zipped per https://github.com/actions/upload-artifact?tab=readme-ov-file#zip-archives)
       - name: Publish the SBOM
-        uses: actions/upload-artifact@v4
+        uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02
         with:
           name: sbom
           path: sbom.spdx.json
@@ -66,7 +66,7 @@ jobs:
       #    path: dist.tar.gz
           
       - name: "Deploy to GitHub Pages"
-        uses: actions/upload-pages-artifact@v3.0.1
+        uses: actions/upload-pages-artifact@56afc609e74202658d3ffba0e8f6dda462b719fa
         with:
           path: dist
 
@@ -90,4 +90,4 @@ jobs:
    steps:
      - name: Deploy to GitHub Pages
        id: deployment
-       uses: actions/deploy-pages@v4 # or specific "vX.X.X" version tag for this action
+       uses: actions/deploy-pages@d6db90164ac5ed86f2b6aed7e0febac5b3c0c03e # or specific "vX.X.X" version tag for this action
diff --git a/src/features/lists/components/ListAddTask.test.jsx b/src/features/lists/components/ListAddTask.test.jsx
@@ -5,21 +5,13 @@ import ListAddTask from './ListAddTask';
 
 // Mock the context hooks
 const mockAddTask = vi.fn();
-const mockAddTag = vi.fn();
 
 vi.mock('../../../context/TaskContext', () => ({
   useTaskContext: () => ({
     addTask: mockAddTask
   })
 }));
 
-vi.mock('../../../context/TagContext', () => ({
-  useTagContext: () => ({
-    tags: ['work', 'personal', 'urgent'],
-    addTag: mockAddTag
-  })
-}));
-
 describe('ListAddTask Component', () => {
   const mockOnCancel = vi.fn();
   const mockListFilters = [
@@ -28,16 +20,14 @@ describe('ListAddTask Component', () => {
   
   beforeEach(() => {
     mockAddTask.mockClear();
-    mockAddTag.mockClear();
     mockOnCancel.mockClear();
   });
 
   test('renders the form correctly', () => {
     render(<ListAddTask onCancel={mockOnCancel} listFilters={mockListFilters} />);
     
-    expect(screen.getByTestId('list-add-task')).toBeInTheDocument();
+    expect(screen.getByTestId('list-add-task-form')).toBeInTheDocument();
     expect(screen.getByTestId('list-task-input')).toBeInTheDocument();
-    expect(screen.getByTestId('list-tag-input')).toBeInTheDocument();
     expect(screen.getByTestId('list-cancel-button')).toBeInTheDocument();
     expect(screen.getByTestId('list-submit-button')).toBeInTheDocument();
   });
@@ -51,63 +41,29 @@ describe('ListAddTask Component', () => {
     expect(taskInput.value).toBe('New Test Task');
   });
 
-  test('handles tag input change', () => {
-    render(<ListAddTask onCancel={mockOnCancel} listFilters={mockListFilters} />);
-    
-    const tagInput = screen.getByTestId('list-tag-input');
-    fireEvent.change(tagInput, { target: { value: 'personal' } });
-    
-    expect(tagInput.value).toBe('personal');
-  });
-
-  test('includes tags from list filters by default', () => {
+  test('displays auto-applied tags from list filters', () => {
     render(<ListAddTask onCancel={mockOnCancel} listFilters={mockListFilters} />);
     
-    expect(screen.getByTestId('list-selected-tags')).toBeInTheDocument();
-    expect(screen.getByText('work')).toBeInTheDocument();
+    expect(screen.getByTestId('auto-applied-tags')).toBeInTheDocument();
+    expect(screen.getByText(/will be tagged with/i)).toBeInTheDocument();
+    expect(screen.getByText(/work/i)).toBeInTheDocument();
   });
 
-  test('adds a tag when Enter key is pressed', () => {
-    render(<ListAddTask onCancel={mockOnCancel} listFilters={mockListFilters} />);
-    
-    const tagInput = screen.getByTestId('list-tag-input');
-    fireEvent.change(tagInput, { target: { value: 'new-tag' } });
-    fireEvent.keyDown(tagInput, { key: 'Enter' });
-    
-    expect(screen.getByText('new-tag')).toBeInTheDocument();
-    expect(tagInput.value).toBe('');
-  });
-
-  test('removes a tag when remove button is clicked', () => {
-    render(<ListAddTask onCancel={mockOnCancel} listFilters={mockListFilters} />);
-    
-    const tagToRemove = screen.getByText('work');
-    const removeButton = screen.getByTestId('list-remove-tag-work');
-    fireEvent.click(removeButton);
-    
-    expect(screen.queryByText('work')).not.toBeInTheDocument();
-  });
-
-  test('submits the form with task and tags', () => {
+  test('submits the form with task and tags from list filters', () => {
     render(<ListAddTask onCancel={mockOnCancel} listFilters={mockListFilters} />);
     
     // Fill in task
     const taskInput = screen.getByTestId('list-task-input');
     fireEvent.change(taskInput, { target: { value: 'Submit Test Task' } });
     
-    // Add another tag
-    const tagInput = screen.getByTestId('list-tag-input');
-    fireEvent.change(tagInput, { target: { value: 'urgent' } });
-    fireEvent.keyDown(tagInput, { key: 'Enter' });
-    
     // Submit the form
     const submitButton = screen.getByTestId('list-submit-button');
     fireEvent.click(submitButton);
     
     expect(mockAddTask).toHaveBeenCalledWith({
       text: 'Submit Test Task',
       isCompleted: false,
-      tags: ['work', 'urgent']
+      tags: ['work']
     });
     expect(mockOnCancel).toHaveBeenCalled();
   });
diff --git a/src/features/lists/components/TaskListConfig.test.jsx b/src/features/lists/components/TaskListConfig.test.jsx
@@ -3,12 +3,13 @@ import { render, screen, fireEvent } from '@testing-library/react';
 import { vi } from 'vitest';
 import TaskListConfig from './TaskListConfig';
 
-// Mock the tag context
-const mockTags = ['work', 'personal', 'urgent', 'home'];
+// Mock variables to control the mocked context values
+let mockTagsValue = ['work', 'personal', 'urgent', 'home'];
 
+// Mock the tag context
 vi.mock('../../../context/TagContext', () => ({
   useTagContext: () => ({
-    tags: mockTags
+    tags: mockTagsValue
   })
 }));
 
@@ -27,6 +28,8 @@ describe('TaskListConfig Component', () => {
   beforeEach(() => {
     mockOnSave.mockClear();
     mockOnCancel.mockClear();
+    // Reset the mock tags to the default value before each test
+    mockTagsValue = ['work', 'personal', 'urgent', 'home'];
   });
 
   test('renders the task list config correctly', () => {
@@ -189,9 +192,8 @@ describe('TaskListConfig Component', () => {
   });
 
   test('displays message when no tags are available', () => {
-    vi.spyOn(require('../../../context/TagContext'), 'useTagContext').mockReturnValue({
-      tags: []
-    });
+    // Set mock tags to an empty array for this test
+    mockTagsValue = [];
 
     render(
       <TaskListConfig 
diff --git a/src/features/tags/components/TagManager.test.jsx b/src/features/tags/components/TagManager.test.jsx
@@ -10,9 +10,12 @@ const mockDeleteTag = vi.fn();
 const mockUpdateTasksWithEditedTag = vi.fn();
 const mockUpdateTasksWithDeletedTag = vi.fn();
 
+// Mock data to control context values in tests
+let mockTagsValue = ['work', 'personal', 'urgent'];
+
 vi.mock('../../../context/TagContext', () => ({
   useTagContext: () => ({
-    tags: ['work', 'personal', 'urgent'],
+    tags: mockTagsValue,
     addTag: mockAddTag,
     editTag: mockEditTag,
     deleteTag: mockDeleteTag
@@ -36,6 +39,8 @@ describe('TagManager Component', () => {
     mockUpdateTasksWithEditedTag.mockClear();
     mockUpdateTasksWithDeletedTag.mockClear();
     mockOnClose.mockClear();
+    // Reset the mock tags to the default value before each test
+    mockTagsValue = ['work', 'personal', 'urgent'];
   });
 
   test('renders the tag manager correctly', () => {
@@ -137,12 +142,8 @@ describe('TagManager Component', () => {
   });
 
   test('renders no tags message when there are no tags', () => {
-    vi.spyOn(require('../../../context/TagContext'), 'useTagContext').mockReturnValue({
-      tags: [],
-      addTag: mockAddTag,
-      editTag: mockEditTag,
-      deleteTag: mockDeleteTag
-    });
+    // Set mock tags to an empty array for this test
+    mockTagsValue = [];
 
     render(<TagManager onClose={mockOnClose} />);
     
diff --git a/src/setupTests.js b/src/setupTests.js
@@ -1,6 +1,7 @@
+import React from 'react';
 import { expect, vi, afterEach } from 'vitest';
 import { cleanup } from '@testing-library/react';
-import matchers from '@testing-library/jest-dom/matchers';
+import * as matchers from '@testing-library/jest-dom/matchers';
 
 // Add custom jest-dom matchers
 expect.extend(matchers);
@@ -12,14 +13,12 @@ afterEach(() => {
 
 // Mock framer-motion to avoid animation-related test issues
 vi.mock('framer-motion', () => {
-  const actual = vi.importActual('framer-motion');
   return {
-    ...actual,
     motion: {
-      div: ({ children, ...props }) => <div {...props}>{children}</div>,
-      button: ({ children, ...props }) => <button {...props}>{children}</button>,
-      p: ({ children, ...props }) => <p {...props}>{children}</p>,
+      div: ({ children, ...props }) => React.createElement('div', props, children),
+      button: ({ children, ...props }) => React.createElement('button', props, children),
+      p: ({ children, ...props }) => React.createElement('p', props, children)
     },
-    AnimatePresence: ({ children }) => <>{children}</>,
+    AnimatePresence: ({ children }) => children,
   };
 });
