@pie-lib/rubric 2.0.21 → 2.0.22-beta.2

package/CHANGELOG.md

@@ -3,80 +3,6 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
-## [2.0.21](https://github.com/pie-framework/pie-lib/compare/@pie-lib/rubric@2.0.20...@pie-lib/rubric@2.0.21) (2026-09-22)
-
-**Note:** Version bump only for package @pie-lib/rubric
-
-## [2.0.20](https://github.com/pie-framework/pie-lib/compare/@pie-lib/rubric@2.0.19...@pie-lib/rubric@2.0.20) (2026-09-16)
-
-**Note:** Version bump only for package @pie-lib/rubric
-
-## [2.0.19](https://github.com/pie-framework/pie-lib/compare/@pie-lib/rubric@2.0.18...@pie-lib/rubric@2.0.19) (2026-09-03)
-
-**Note:** Version bump only for package @pie-lib/rubric
-
-## [2.0.18](https://github.com/pie-framework/pie-lib/compare/@pie-lib/rubric@2.0.17...@pie-lib/rubric@2.0.18) (2026-09-02)
-
-**Note:** Version bump only for package @pie-lib/rubric
-
-## [2.0.17](https://github.com/pie-framework/pie-lib/compare/@pie-lib/rubric@2.0.16...@pie-lib/rubric@2.0.17) (2026-08-26)
-
-**Note:** Version bump only for package @pie-lib/rubric
-
-## [2.0.16](https://github.com/pie-framework/pie-lib/compare/@pie-lib/rubric@2.0.15...@pie-lib/rubric@2.0.16) (2026-08-21)
-
-**Note:** Version bump only for package @pie-lib/rubric
-
-## [2.0.15](https://github.com/pie-framework/pie-lib/compare/@pie-lib/rubric@2.0.14...@pie-lib/rubric@2.0.15) (2026-08-18)
-
-**Note:** Version bump only for package @pie-lib/rubric
-
-## [2.0.14](https://github.com/pie-framework/pie-lib/compare/@pie-lib/rubric@2.0.13...@pie-lib/rubric@2.0.14) (2026-08-13)
-
-**Note:** Version bump only for package @pie-lib/rubric
-
-## [2.0.13](https://github.com/pie-framework/pie-lib/compare/@pie-lib/rubric@2.0.12...@pie-lib/rubric@2.0.13) (2026-07-31)
-
-**Note:** Version bump only for package @pie-lib/rubric
-
-## [2.0.12](https://github.com/pie-framework/pie-lib/compare/@pie-lib/rubric@2.0.11...@pie-lib/rubric@2.0.12) (2026-07-08)
-
-**Note:** Version bump only for package @pie-lib/rubric
-
-## [2.0.11](https://github.com/pie-framework/pie-lib/compare/@pie-lib/rubric@2.0.10...@pie-lib/rubric@2.0.11) (2026-07-01)
-
-**Note:** Version bump only for package @pie-lib/rubric
-
-## [2.0.10](https://github.com/pie-framework/pie-lib/compare/@pie-lib/rubric@2.0.9...@pie-lib/rubric@2.0.10) (2026-06-25)
-
-**Note:** Version bump only for package @pie-lib/rubric
-
-## [2.0.9](https://github.com/pie-framework/pie-lib/compare/@pie-lib/rubric@2.0.8...@pie-lib/rubric@2.0.9) (2026-06-19)
-
-**Note:** Version bump only for package @pie-lib/rubric
-
-## [2.0.8](https://github.com/pie-framework/pie-lib/compare/@pie-lib/rubric@2.0.7...@pie-lib/rubric@2.0.8) (2026-06-17)
-
-**Note:** Version bump only for package @pie-lib/rubric
-
-## [2.0.7](https://github.com/pie-framework/pie-lib/compare/@pie-lib/rubric@2.0.6...@pie-lib/rubric@2.0.7) (2026-06-12)
-
-### Bug Fixes
-
-- correct @emotion/style typo to @emotion/styled in package deps PIE-678 ([d529ac7](https://github.com/pie-framework/pie-lib/commit/d529ac7bc3dfa94d8037688452d4c70c66f2590b))
-
-## [2.0.6](https://github.com/pie-framework/pie-lib/compare/@pie-lib/rubric@2.0.5...@pie-lib/rubric@2.0.6) (2026-06-03)
-
-**Note:** Version bump only for package @pie-lib/rubric
-
-## [2.0.5](https://github.com/pie-framework/pie-lib/compare/@pie-lib/rubric@2.0.4...@pie-lib/rubric@2.0.5) (2026-06-01)
-
-**Note:** Version bump only for package @pie-lib/rubric
-
-## [2.0.4](https://github.com/pie-framework/pie-lib/compare/@pie-lib/rubric@2.0.3...@pie-lib/rubric@2.0.4) (2026-05-21)
-
-**Note:** Version bump only for package @pie-lib/rubric
-
 ## [2.0.3](https://github.com/pie-framework/pie-lib/compare/@pie-lib/rubric@2.0.2...@pie-lib/rubric@2.0.3) (2026-05-15)
 
 **Note:** Version bump only for package @pie-lib/rubric

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pie-lib/rubric",
-  "version": "2.0.21",
+  "version": "2.0.22-beta.2",
   "publishConfig": {
     "access": "public"
   },
@@ -13,12 +13,12 @@
     "@hello-pangea/dnd": "^18.0.1",
     "@mui/icons-material": "^7.3.4",
     "@mui/material": "^7.3.4",
-    "@pie-lib/editable-html-tip-tap": "^2.1.19",
+    "@pie-lib/editable-html-tip-tap": "^2.1.20-beta.2",
     "debug": "^4.1.1",
     "lodash-es": "^4.17.23",
     "prop-types": "^15.7.2",
     "react": "^18.2.0",
     "react-dom": "^18.2.0"
   },
-  "gitHead": "555178e597d60d604dcc8b77cbb86dfde7b86b4f"
+  "gitHead": "433bcb776c2975739e7417f133d31cd996164116"
 }

package/LICENSE.md

@@ -1,5 +0,0 @@
-Copyright 2019 CoreSpring Inc
-
-Permission to use, copy, modify, and/or distribute this software for any purpose with or without fee is hereby granted, provided that the above copyright notice and this permission notice appear in all copies.
-
-THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.

package/src/__tests__/rubric.test.jsx

@@ -1,373 +0,0 @@
-import { fireEvent, render, screen } from '@testing-library/react';
-import React from 'react';
-import { RawAuthoring } from '../authoring';
-import { takeRight } from 'lodash-es';
-import { createTheme, ThemeProvider } from '@mui/material/styles';
-
-// Mock dependencies
-jest.mock('@pie-lib/editable-html-tip-tap', () => {
-  return function EditableHtml(props) {
-    return <div data-testid="editable-html" data-markup={props.markup} />;
-  };
-});
-
-jest.mock('@pie-lib/config-ui', () => ({
-  FeedbackConfig: ({ feedback }) => <div data-testid="feedback-config">{JSON.stringify(feedback)}</div>,
-  InputContainer: ({ children, label }) => (
-    <div data-testid="input-container" data-label={label}>
-      {children}
-    </div>
-  ),
-}));
-
-jest.mock('@hello-pangea/dnd', () => ({
-  DragDropContext: ({ children }) => <div data-testid="drag-drop-context">{children}</div>,
-  Droppable: ({ children }) => children({ droppableProps: {}, innerRef: () => {} }, {}),
-  Draggable: ({ children, index }) =>
-    children(
-      {
-        innerRef: () => {},
-        draggableProps: { 'data-draggable-index': index },
-        dragHandleProps: {},
-      },
-      {},
-    ),
-}));
-
-describe('Rubric', () => {
-  const points = ['nothing right', 'a teeny bit right', 'mostly right', 'bingo'];
-  const sampleAnswers = [null, 'just right', 'not left', null];
-  const theme = createTheme();
-
-  const renderComponent = (value = {}, props = {}) => {
-    const defaultProps = {
-      classes: {},
-      onChange: jest.fn(),
-      className: 'className',
-      value: {
-        excludeZero: false,
-        points,
-        sampleAnswers,
-        ...value,
-      },
-      ...props,
-    };
-
-    return {
-      ...render(
-        <ThemeProvider theme={theme}>
-          <RawAuthoring {...defaultProps} />
-        </ThemeProvider>,
-      ),
-      onChange: defaultProps.onChange,
-      props: defaultProps,
-    };
-  };
-
-  describe('render', () => {
-    it('renders rubric title and main structure', () => {
-      renderComponent();
-      expect(screen.getByText('Rubric')).toBeInTheDocument();
-      expect(screen.getByLabelText('Max Points')).toBeInTheDocument();
-      expect(screen.getByLabelText('Exclude zeros')).toBeInTheDocument();
-    });
-
-    it('renders all point configurations', () => {
-      const { container } = renderComponent();
-      // Verify DragDropContext is rendered
-      expect(container.querySelector('[data-testid="drag-drop-context"]')).toBeInTheDocument();
-
-      // Check that point labels are rendered (4 points = "3 pts", "2 pts", "1 pt", "0 pt")
-      // Note: The PointConfig component uses singular "pt" for 0 and 1, plural "pts" for 2+
-      expect(screen.getByText('3 pts')).toBeInTheDocument();
-      expect(screen.getByText('2 pts')).toBeInTheDocument();
-      expect(screen.getByText('1 pt')).toBeInTheDocument();
-      expect(screen.getByText('0 pt')).toBeInTheDocument();
-    });
-
-    describe('draggable', () => {
-      it('renders 3 draggable items when excludeZero is true', () => {
-        // When excludeZero is true, the last point (0 pts) should not be rendered
-        expect(screen.queryByText('0 pt')).not.toBeInTheDocument();
-      });
-
-      it('renders 4 draggable items when excludeZero is false', () => {
-        const { container } = renderComponent({ excludeZero: false });
-
-        // When excludeZero is false, all points including 0 should be rendered
-        const draggableItems = container.querySelectorAll('[data-draggable-index]');
-        expect(draggableItems.length).toEqual(4);
-        expect(screen.getByText('0 pt')).toBeInTheDocument();
-
-        // Verify all point labels are rendered
-        expect(screen.getByText('3 pts')).toBeInTheDocument();
-        expect(screen.getByText('2 pts')).toBeInTheDocument();
-        expect(screen.getByText('1 pt')).toBeInTheDocument();
-        expect(screen.getByText('0 pt')).toBeInTheDocument();
-      });
-    });
-  });
-
-  describe('logic', () => {
-    describe('changeMaxPoints', () => {
-      const testChangeMaxPoints = (maxPoints, excludeZero, expectedPoints, expectedSampleAnswers) => {
-        it(`maxPoints=${maxPoints}, excludeZero=${excludeZero} calls onChange correctly`, () => {
-          const { onChange, container } = renderComponent({ excludeZero });
-
-          // Get the component instance through the container
-          // We need to call the method directly since we're testing internal behavior
-          const instance = container.querySelector('[class*="MuiBox-root"]')?._owner;
-
-          // Since we can't easily access instance methods in RTL, we'll test the behavior
-          // by verifying the onChange prop receives the correct values
-          // For now, we'll directly test the logic
-          const component = new RawAuthoring({
-            value: { excludeZero, points, sampleAnswers },
-            onChange,
-            classes: {},
-            className: 'className',
-          });
-
-          component.changeMaxPoints(maxPoints);
-
-          expect(onChange).toHaveBeenCalledWith({
-            excludeZero,
-            points: expectedPoints,
-            sampleAnswers: expectedSampleAnswers,
-            maxPoints: expectedPoints.length - 1,
-          });
-        });
-      };
-
-      testChangeMaxPoints(1, false, takeRight(points, 2), takeRight(sampleAnswers, 2));
-      testChangeMaxPoints(1, true, takeRight(points, 2), takeRight(sampleAnswers, 2));
-      testChangeMaxPoints(2, true, takeRight(points, 3), takeRight(sampleAnswers, 3));
-      testChangeMaxPoints(2, false, takeRight(points, 3), takeRight(sampleAnswers, 3));
-      testChangeMaxPoints(5, false, ['', ''].concat(points), [null, null].concat(sampleAnswers));
-    });
-
-    describe('changeSampleResponse', () => {
-      const testChangeSampleResponse = (index, clickedItem, excludeZero, expectedPoints, expectedSampleAnswers) => {
-        it(`Point ${index} with clickedItem="${clickedItem}" calls onChange correctly`, () => {
-          const { onChange } = renderComponent({ excludeZero });
-
-          // Test the logic by creating a component instance and calling the method
-          const component = new RawAuthoring({
-            value: { excludeZero, points, sampleAnswers },
-            onChange,
-            classes: {},
-            className: 'className',
-          });
-
-          component.onPointMenuChange(index, clickedItem);
-
-          expect(onChange).toHaveBeenCalledWith({
-            excludeZero,
-            points: expectedPoints,
-            sampleAnswers: expectedSampleAnswers,
-          });
-        });
-      };
-
-      testChangeSampleResponse(0, 'sample', false, points, ['', 'just right', 'not left', null]);
-      testChangeSampleResponse(3, 'sample', false, points, [null, 'just right', 'not left', '']);
-      testChangeSampleResponse(1, 'sample', true, points, [null, null, 'not left', null]);
-      testChangeSampleResponse(3, 'sample', true, points, [null, 'just right', 'not left', '']);
-    });
-  });
-
-  describe('user interactions', () => {
-    it('calls onChange when excluding zeros checkbox is toggled', () => {
-      const { onChange } = renderComponent({ excludeZero: false });
-
-      const checkbox = screen.getByLabelText('Exclude zeros');
-      fireEvent.click(checkbox);
-
-      expect(onChange).toHaveBeenCalledWith(
-        expect.objectContaining({
-          excludeZero: true,
-        }),
-      );
-    });
-
-    it('calls onChange when max points is changed', () => {
-      const { onChange } = renderComponent();
-
-      const selectInput = screen.getByLabelText('Max Points');
-      fireEvent.mouseDown(selectInput);
-
-      // The onChange should be called when a new value is selected
-      // This tests the changeMaxPoints logic indirectly
-    });
-
-    it('renders correct number of points based on excludeZero', () => {
-      const { rerender } = renderComponent({ excludeZero: false });
-      expect(screen.getByText('0 pt')).toBeInTheDocument();
-
-      // Rerender with excludeZero true
-      rerender(
-        <ThemeProvider theme={theme}>
-          <RawAuthoring
-            classes={{}}
-            onChange={jest.fn()}
-            className="className"
-            value={{ excludeZero: true, points, sampleAnswers }}
-          />
-        </ThemeProvider>,
-      );
-
-      expect(screen.queryByText('0 pt')).not.toBeInTheDocument();
-    });
-  });
-
-  describe('drag and drop', () => {
-    it('calls onChange with reordered points after drag end', () => {
-      const { onChange } = renderComponent();
-
-      const component = new RawAuthoring({
-        value: { excludeZero: false, points, sampleAnswers },
-        onChange,
-        classes: {},
-        className: 'className',
-      });
-
-      // Simulate drag from index 0 to index 2
-      component.dragEnd({
-        source: { index: 0 },
-        destination: { index: 2 },
-      });
-
-      expect(onChange).toHaveBeenCalled();
-      const result = onChange.mock.calls[0][0];
-      expect(result.points).not.toEqual(points);
-      expect(result.points.length).toBe(points.length);
-    });
-
-    it('does nothing when drag has no destination', () => {
-      const { onChange } = renderComponent();
-
-      const component = new RawAuthoring({
-        value: { excludeZero: false, points, sampleAnswers },
-        onChange,
-        classes: {},
-        className: 'className',
-      });
-
-      // Simulate drag without destination
-      component.dragEnd({
-        source: { index: 0 },
-        destination: null,
-      });
-
-      expect(onChange).not.toHaveBeenCalled();
-    });
-  });
-
-  describe('content editing', () => {
-    it('calls onChange when point content is changed', () => {
-      const { onChange } = renderComponent();
-
-      const component = new RawAuthoring({
-        value: { excludeZero: false, points, sampleAnswers },
-        onChange,
-        classes: {},
-        className: 'className',
-      });
-
-      component.changeContent(0, 'new content', 'points');
-
-      expect(onChange).toHaveBeenCalledWith(
-        expect.objectContaining({
-          points: expect.arrayContaining(['new content']),
-        }),
-      );
-    });
-
-    it('calls onChange when sample answer is changed', () => {
-      const { onChange } = renderComponent();
-
-      const component = new RawAuthoring({
-        value: { excludeZero: false, points, sampleAnswers },
-        onChange,
-        classes: {},
-        className: 'className',
-      });
-
-      component.changeContent(1, 'new sample answer', 'sampleAnswers');
-
-      expect(onChange).toHaveBeenCalledWith(
-        expect.objectContaining({
-          sampleAnswers: expect.arrayContaining(['new sample answer']),
-        }),
-      );
-    });
-
-    it('does not change content for invalid type', () => {
-      const { onChange } = renderComponent();
-
-      const component = new RawAuthoring({
-        value: { excludeZero: false, points, sampleAnswers },
-        onChange,
-        classes: {},
-        className: 'className',
-      });
-
-      component.changeContent(0, 'new content', 'invalidType');
-
-      expect(onChange).not.toHaveBeenCalled();
-    });
-  });
-
-  describe('rubricless mode', () => {
-    it('calls onChange when rubricless instruction is changed', () => {
-      const { onChange } = renderComponent({ rubriclessInstruction: '' }, { rubricless: true });
-
-      const component = new RawAuthoring({
-        value: { excludeZero: false, points, sampleAnswers, rubriclessInstruction: '' },
-        onChange,
-        classes: {},
-        className: 'className',
-        rubricless: true,
-      });
-
-      component.changeRubriclessInstruction('New instruction');
-
-      expect(onChange).toHaveBeenCalledWith(
-        expect.objectContaining({
-          rubriclessInstruction: 'New instruction',
-        }),
-      );
-    });
-  });
-
-  describe('edge cases', () => {
-    it('handles empty points array', () => {
-      const { container } = renderComponent({ points: [], sampleAnswers: [] });
-      expect(container).toBeInTheDocument();
-    });
-
-    it('handles maxPoints greater than current max', () => {
-      const { onChange } = renderComponent();
-
-      const component = new RawAuthoring({
-        value: { excludeZero: false, points, sampleAnswers },
-        onChange,
-        classes: {},
-        className: 'className',
-      });
-
-      component.changeMaxPoints(10);
-
-      expect(onChange).toHaveBeenCalled();
-      const result = onChange.mock.calls[0][0];
-      expect(result.points.length).toBe(11); // 10 + 1 for 0 points
-    });
-
-    it('handles config and pluginOpts props', () => {
-      const config = { someConfig: true };
-      const pluginOpts = { somePlugin: true };
-      const { container } = renderComponent({}, { config, pluginOpts });
-      expect(container).toBeInTheDocument();
-    });
-  });
-});