@pie-lib/charting 6.1.1-next.0 → 6.2.0-next.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (85) hide show
  1. package/lib/actions-button.js +0 -5
  2. package/lib/actions-button.js.map +1 -1
  3. package/lib/axes.js +0 -23
  4. package/lib/axes.js.map +1 -1
  5. package/lib/bars/bar.js +0 -2
  6. package/lib/bars/bar.js.map +1 -1
  7. package/lib/bars/common/bars.js +0 -13
  8. package/lib/bars/common/bars.js.map +1 -1
  9. package/lib/bars/common/correct-check-icon.js +0 -1
  10. package/lib/bars/common/correct-check-icon.js.map +1 -1
  11. package/lib/bars/histogram.js +0 -2
  12. package/lib/bars/histogram.js.map +1 -1
  13. package/lib/chart-setup.js +0 -19
  14. package/lib/chart-setup.js.map +1 -1
  15. package/lib/chart-type.js +0 -1
  16. package/lib/chart-type.js.map +1 -1
  17. package/lib/chart-types.js +0 -1
  18. package/lib/chart-types.js.map +1 -1
  19. package/lib/chart.js +2 -18
  20. package/lib/chart.js.map +1 -1
  21. package/lib/common/correctness-indicators.js +0 -4
  22. package/lib/common/correctness-indicators.js.map +1 -1
  23. package/lib/common/drag-handle.js +0 -1
  24. package/lib/common/drag-handle.js.map +1 -1
  25. package/lib/common/drag-icon.js +0 -2
  26. package/lib/common/drag-icon.js.map +1 -1
  27. package/lib/common/styles.js +0 -1
  28. package/lib/common/styles.js.map +1 -1
  29. package/lib/grid.js +0 -4
  30. package/lib/grid.js.map +1 -1
  31. package/lib/index.js +0 -1
  32. package/lib/index.js.map +1 -1
  33. package/lib/key-legend.js +0 -1
  34. package/lib/key-legend.js.map +1 -1
  35. package/lib/line/common/drag-handle.js +0 -2
  36. package/lib/line/common/drag-handle.js.map +1 -1
  37. package/lib/line/common/line.js +2 -19
  38. package/lib/line/common/line.js.map +1 -1
  39. package/lib/line/line-cross.js +0 -16
  40. package/lib/line/line-cross.js.map +1 -1
  41. package/lib/line/line-dot.js +0 -2
  42. package/lib/line/line-dot.js.map +1 -1
  43. package/lib/mark-label.js +0 -21
  44. package/lib/mark-label.js.map +1 -1
  45. package/lib/plot/common/plot.js +0 -12
  46. package/lib/plot/common/plot.js.map +1 -1
  47. package/lib/plot/dot.js +0 -3
  48. package/lib/plot/dot.js.map +1 -1
  49. package/lib/plot/line.js +0 -3
  50. package/lib/plot/line.js.map +1 -1
  51. package/lib/tool-menu.js +0 -11
  52. package/lib/tool-menu.js.map +1 -1
  53. package/lib/utils.js +0 -12
  54. package/lib/utils.js.map +1 -1
  55. package/package.json +12 -9
  56. package/src/__tests__/actions-button.test.jsx +280 -0
  57. package/src/__tests__/axes.test.jsx +557 -16
  58. package/src/__tests__/chart-setup.test.jsx +495 -10
  59. package/src/__tests__/chart.test.jsx +1 -1
  60. package/src/__tests__/grid.test.jsx +2 -2
  61. package/src/__tests__/key-legend.test.jsx +223 -0
  62. package/src/__tests__/tool-menu.test.jsx +522 -0
  63. package/src/__tests__/utils.js +1 -1
  64. package/src/axes.jsx +10 -7
  65. package/src/bars/common/bars.jsx +32 -50
  66. package/src/chart-setup.jsx +6 -9
  67. package/src/chart-type.js +3 -6
  68. package/src/chart.jsx +2 -2
  69. package/src/common/__tests__/correctness-indicators.test.jsx +720 -0
  70. package/src/common/__tests__/drag-handle.test.jsx +0 -1
  71. package/src/common/correctness-indicators.jsx +8 -13
  72. package/src/common/drag-handle.jsx +2 -12
  73. package/src/grid.jsx +1 -1
  74. package/src/line/__tests__/line-cross.test.jsx +423 -1
  75. package/src/line/__tests__/utils.js +1 -1
  76. package/src/line/common/__tests__/drag-handle.test.jsx +1 -2
  77. package/src/line/common/drag-handle.jsx +2 -11
  78. package/src/line/common/line.jsx +2 -2
  79. package/src/line/line-cross.js +7 -13
  80. package/src/mark-label.jsx +3 -3
  81. package/src/plot/__tests__/dot.test.jsx +300 -1
  82. package/src/plot/__tests__/line.test.jsx +331 -1
  83. package/src/plot/common/plot.jsx +14 -13
  84. package/src/utils.js +0 -1
  85. package/NEXT.CHANGELOG.json +0 -16
@@ -0,0 +1,280 @@
1
+ import React from 'react';
2
+ import { fireEvent, render } from '@testing-library/react';
3
+ import { createTheme, ThemeProvider } from '@mui/material/styles';
4
+ import { ActionsButton } from '../actions-button';
5
+
6
+ jest.mock('@pie-lib/translator', () => ({
7
+ __esModule: true,
8
+ default: {
9
+ translator: {
10
+ t: jest.fn((key) => {
11
+ const translations = {
12
+ 'charting.add': 'Add Category',
13
+ 'charting.delete': 'Delete',
14
+ 'charting.newLabel': 'New Category',
15
+ };
16
+ return translations[key] || key;
17
+ }),
18
+ },
19
+ },
20
+ }));
21
+
22
+ let theme;
23
+
24
+ beforeAll(() => {
25
+ theme = createTheme();
26
+ });
27
+
28
+ describe('ActionsButton', () => {
29
+ const addCategory = jest.fn();
30
+ const deleteCategory = jest.fn();
31
+
32
+ beforeEach(() => {
33
+ jest.clearAllMocks();
34
+ });
35
+
36
+ const renderComponent = (extras = {}) => {
37
+ const defaults = {
38
+ addCategory,
39
+ deleteCategory,
40
+ categories: [],
41
+ language: 'en',
42
+ };
43
+ const props = { ...defaults, ...extras };
44
+ return render(
45
+ <ThemeProvider theme={theme}>
46
+ <ActionsButton {...props} />
47
+ </ThemeProvider>,
48
+ );
49
+ };
50
+
51
+ describe('rendering', () => {
52
+ it('renders without crashing', () => {
53
+ const { container } = renderComponent();
54
+ expect(container).toBeInTheDocument();
55
+ });
56
+
57
+ it('renders Actions button', () => {
58
+ const { getByText } = renderComponent();
59
+ expect(getByText('Actions')).toBeInTheDocument();
60
+ });
61
+
62
+ it('renders with custom language', () => {
63
+ const { container } = renderComponent({ language: 'es' });
64
+ expect(container).toBeInTheDocument();
65
+ });
66
+
67
+ it('renders with empty categories', () => {
68
+ const { container } = renderComponent({ categories: [] });
69
+ expect(container).toBeInTheDocument();
70
+ });
71
+
72
+ it('renders with multiple categories', () => {
73
+ const categories = [
74
+ { label: 'A', deletable: true },
75
+ { label: 'B', deletable: true },
76
+ { label: 'C', deletable: false },
77
+ ];
78
+ const { container } = renderComponent({ categories });
79
+ expect(container).toBeInTheDocument();
80
+ });
81
+ });
82
+
83
+ describe('popover interactions', () => {
84
+ it('opens popover when Actions button is clicked', () => {
85
+ const { getByText } = renderComponent();
86
+ const actionsButton = getByText('Actions');
87
+
88
+ fireEvent.click(actionsButton);
89
+
90
+ expect(getByText(/Add Category/i)).toBeInTheDocument();
91
+ });
92
+
93
+ it('closes popover when clicking outside', () => {
94
+ const { getByText } = renderComponent();
95
+ const actionsButton = getByText('Actions');
96
+
97
+ fireEvent.click(actionsButton);
98
+ expect(getByText(/Add Category/i)).toBeInTheDocument();
99
+
100
+ const backdrop = document.querySelector('.MuiPopover-root');
101
+ if (backdrop) {
102
+ fireEvent.click(backdrop);
103
+ }
104
+ });
105
+
106
+ it('has correct accessibility attributes', () => {
107
+ const { getByText } = renderComponent();
108
+ const actionsButton = getByText('Actions');
109
+
110
+ expect(actionsButton).toHaveAttribute('role', 'button');
111
+ expect(actionsButton).toHaveAttribute('tabIndex', '0');
112
+ });
113
+ });
114
+
115
+ describe('add category functionality', () => {
116
+ it('calls addCategory when add button is clicked', () => {
117
+ const { getByText } = renderComponent();
118
+
119
+ fireEvent.click(getByText('Actions'));
120
+ fireEvent.click(getByText(/Add Category/i));
121
+
122
+ expect(addCategory).toHaveBeenCalledTimes(1);
123
+ });
124
+
125
+ it('closes popover after adding category', () => {
126
+ const { getByText } = renderComponent();
127
+
128
+ fireEvent.click(getByText('Actions'));
129
+ fireEvent.click(getByText(/Add Category/i));
130
+
131
+ expect(addCategory).toHaveBeenCalled();
132
+ });
133
+ });
134
+
135
+ describe('delete category functionality', () => {
136
+ it('shows delete buttons for deletable categories', () => {
137
+ const categories = [
138
+ { label: 'Category A', deletable: true },
139
+ { label: 'Category B', deletable: true },
140
+ ];
141
+ const { getByText } = renderComponent({ categories });
142
+
143
+ fireEvent.click(getByText('Actions'));
144
+
145
+ expect(getByText(/Delete <Category A>/i)).toBeInTheDocument();
146
+ expect(getByText(/Delete <Category B>/i)).toBeInTheDocument();
147
+ });
148
+
149
+ it('does not show delete button for non-deletable categories', () => {
150
+ const categories = [{ label: 'Category A', deletable: false }];
151
+ const { getByText, queryByText } = renderComponent({ categories });
152
+
153
+ fireEvent.click(getByText('Actions'));
154
+
155
+ expect(queryByText(/Delete <Category A>/i)).not.toBeInTheDocument();
156
+ });
157
+
158
+ it('does not show delete button for categories with correctness', () => {
159
+ const categories = [{ label: 'Category A', deletable: true, correctness: { value: 'correct' } }];
160
+ const { getByText, queryByText } = renderComponent({ categories });
161
+
162
+ fireEvent.click(getByText('Actions'));
163
+
164
+ expect(queryByText(/Delete <Category A>/i)).not.toBeInTheDocument();
165
+ });
166
+
167
+ it('calls deleteCategory with correct index', () => {
168
+ const categories = [
169
+ { label: 'Category A', deletable: true },
170
+ { label: 'Category B', deletable: true },
171
+ ];
172
+ const { getByText } = renderComponent({ categories });
173
+
174
+ fireEvent.click(getByText('Actions'));
175
+ fireEvent.click(getByText(/Delete <Category B>/i));
176
+
177
+ expect(deleteCategory).toHaveBeenCalledWith(1);
178
+ });
179
+
180
+ it('closes popover after deleting category', () => {
181
+ const categories = [{ label: 'Category A', deletable: true }];
182
+ const { getByText } = renderComponent({ categories });
183
+
184
+ fireEvent.click(getByText('Actions'));
185
+ fireEvent.click(getByText(/Delete <Category A>/i));
186
+
187
+ expect(deleteCategory).toHaveBeenCalled();
188
+ });
189
+
190
+ it('shows "New Category" for categories without label', () => {
191
+ const categories = [{ label: '', deletable: true }];
192
+ const { getByText } = renderComponent({ categories });
193
+
194
+ fireEvent.click(getByText('Actions'));
195
+
196
+ expect(getByText(/Delete <New Category>/i)).toBeInTheDocument();
197
+ });
198
+
199
+ it('handles deletion of multiple categories', () => {
200
+ const categories = [
201
+ { label: 'A', deletable: true },
202
+ { label: 'B', deletable: true },
203
+ { label: 'C', deletable: true },
204
+ ];
205
+ const { getByText } = renderComponent({ categories });
206
+
207
+ fireEvent.click(getByText('Actions'));
208
+
209
+ expect(getByText(/Delete <A>/i)).toBeInTheDocument();
210
+ expect(getByText(/Delete <B>/i)).toBeInTheDocument();
211
+ expect(getByText(/Delete <C>/i)).toBeInTheDocument();
212
+ });
213
+ });
214
+
215
+ describe('componentWillUnmount', () => {
216
+ it('cleans up state on unmount', () => {
217
+ const { unmount, getByText } = renderComponent();
218
+
219
+ fireEvent.click(getByText('Actions'));
220
+ unmount();
221
+
222
+ // Should not throw error
223
+ expect(true).toBe(true);
224
+ });
225
+ });
226
+
227
+ describe('edge cases', () => {
228
+ it('handles empty categories array', () => {
229
+ const { container } = renderComponent({ categories: [] });
230
+ expect(container).toBeInTheDocument();
231
+ });
232
+
233
+ it('handles undefined language', () => {
234
+ const { container } = renderComponent({ language: undefined });
235
+ expect(container).toBeInTheDocument();
236
+ });
237
+
238
+ it('handles mixed deletable and correctness states', () => {
239
+ const categories = [
240
+ { label: 'A', deletable: true },
241
+ { label: 'B', deletable: false },
242
+ { label: 'C', deletable: true, correctness: { value: 'correct' } },
243
+ { label: 'D', deletable: true },
244
+ ];
245
+ const { getByText } = renderComponent({ categories });
246
+
247
+ fireEvent.click(getByText('Actions'));
248
+
249
+ expect(getByText(/Delete <A>/i)).toBeInTheDocument();
250
+ expect(getByText(/Delete <D>/i)).toBeInTheDocument();
251
+ });
252
+
253
+ it('handles categories with special characters in labels', () => {
254
+ const categories = [{ label: '<>&"\'', deletable: true }];
255
+ const { getByText } = renderComponent({ categories });
256
+
257
+ fireEvent.click(getByText('Actions'));
258
+
259
+ expect(getByText(/Delete </i)).toBeInTheDocument();
260
+ });
261
+
262
+ it('handles very long category labels', () => {
263
+ const categories = [{ label: 'A'.repeat(100), deletable: true }];
264
+ const { container } = renderComponent({ categories });
265
+ expect(container).toBeInTheDocument();
266
+ });
267
+ });
268
+
269
+ describe('popover positioning', () => {
270
+ it('has correct anchor origin', () => {
271
+ const { getByText, queryByText } = renderComponent();
272
+
273
+ expect(queryByText(/\+ Add Category/i)).not.toBeInTheDocument();
274
+
275
+ fireEvent.click(getByText('Actions'));
276
+
277
+ expect(getByText(/\+ Add Category/i)).toBeInTheDocument();
278
+ });
279
+ });
280
+ });