Skip to content

internal: Remove Enzyme from visx #1893

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 17 commits into from
Dec 18, 2024
Prev Previous commit
Next Next commit
Clean for PR
  • Loading branch information
Charles Covey-Brandt committed Dec 17, 2024
commit 5fd8a3d13e112dec0b9ccb5aa9e7628747ea3e26
11 changes: 0 additions & 11 deletions packages/visx-annotation/test/CircleSubject.test.tsx
Original file line number Diff line number Diff line change
@@ -1,13 +1,3 @@
/**
* LLM-GENERATED REFACTOR
*
* This file was migrated from Enzyme to RTL using generative AI.
* To make the migration as clean as possible, the LLM was instructed to
* use testing patterns similar to Enzyme.
*
* If you are making changes to this file, please consider refactoring
* to more idiomatic RTL (and then removing this banner!).
*/
import React from 'react';
import { render } from '@testing-library/react';
import '@testing-library/jest-dom';
Expand All @@ -31,4 +21,3 @@ describe('<CircleSubject />', () => {
expect(circle).toHaveAttribute('cy', '10');
});
});
// MIGRATION STATUS: {"eslint":"pending","jest":{"passed":2,"failed":0,"total":2,"skipped":0,"successRate":100},"tsc":"pending","enyzme":"converted"}
11 changes: 0 additions & 11 deletions packages/visx-annotation/test/Connector.test.tsx
Original file line number Diff line number Diff line change
@@ -1,13 +1,3 @@
/**
* LLM-GENERATED REFACTOR
*
* This file was migrated from Enzyme to RTL using generative AI.
* To make the migration as clean as possible, the LLM was instructed to
* use testing patterns similar to Enzyme.
*
* If you are making changes to this file, please consider refactoring
* to more idiomatic RTL (and then removing this banner!).
*/
import React from 'react';
import { render } from '@testing-library/react';
import '@testing-library/jest-dom';
Expand All @@ -27,4 +17,3 @@ describe('<Connector />', () => {
expect(container.querySelector('path')).toBeInTheDocument();
});
});
// MIGRATION STATUS: {"eslint":"pending","jest":{"passed":2,"failed":0,"total":2,"skipped":0,"successRate":100},"tsc":"pending","enyzme":"converted"}
11 changes: 0 additions & 11 deletions packages/visx-annotation/test/EditableAnnotation.test.tsx
Original file line number Diff line number Diff line change
@@ -1,13 +1,3 @@
/**
* LLM-GENERATED REFACTOR
*
* This file was migrated from Enzyme to RTL using generative AI.
* To make the migration as clean as possible, the LLM was instructed to
* use testing patterns similar to Enzyme.
*
* If you are making changes to this file, please consider refactoring
* to more idiomatic RTL (and then removing this banner!).
*/
import React from 'react';
import { render } from '@testing-library/react';
import '@testing-library/jest-dom';
Expand Down Expand Up @@ -83,4 +73,3 @@ describe('<EditableAnnotation />', () => {
expect(labelHandle).toHaveAttribute('cy', '60'); // y + dy
});
});
// MIGRATION STATUS: {"eslint":"pending","jest":{"passed":6,"failed":0,"total":6,"skipped":0,"successRate":100},"tsc":"pending","enyzme":"converted"}
11 changes: 0 additions & 11 deletions packages/visx-annotation/test/Label.test.tsx
Original file line number Diff line number Diff line change
@@ -1,13 +1,3 @@
/**
* LLM-GENERATED REFACTOR
*
* This file was migrated from Enzyme to RTL using generative AI.
* To make the migration as clean as possible, the LLM was instructed to
* use testing patterns similar to Enzyme.
*
* If you are making changes to this file, please consider refactoring
* to more idiomatic RTL (and then removing this banner!).
*/
import React from 'react';
import { render } from '@testing-library/react';
import '@testing-library/jest-dom';
Expand Down Expand Up @@ -64,4 +54,3 @@ describe('<Label />', () => {
expect(line).toBeInTheDocument();
});
});
// MIGRATION STATUS: {"eslint":"pending","jest":{"passed":5,"failed":0,"total":5,"skipped":0,"successRate":100},"tsc":"pending","enyzme":"converted"}
11 changes: 0 additions & 11 deletions packages/visx-annotation/test/LineSubject.test.tsx
Original file line number Diff line number Diff line change
@@ -1,13 +1,3 @@
/**
* LLM-GENERATED REFACTOR
*
* This file was migrated from Enzyme to RTL using generative AI.
* To make the migration as clean as possible, the LLM was instructed to
* use testing patterns similar to Enzyme.
*
* If you are making changes to this file, please consider refactoring
* to more idiomatic RTL (and then removing this banner!).
*/
import React from 'react';
import { render } from '@testing-library/react';
import '@testing-library/jest-dom';
Expand All @@ -28,4 +18,3 @@ describe('<LineSubject />', () => {
expect(lineElement).toBeInTheDocument();
});
});
// MIGRATION STATUS: {"eslint":"pending","jest":{"passed":2,"failed":0,"total":2,"skipped":0,"successRate":100},"tsc":"pending","enyzme":"converted"}
11 changes: 0 additions & 11 deletions packages/visx-axis/test/Axis.test.tsx
Original file line number Diff line number Diff line change
@@ -1,13 +1,3 @@
/**
* LLM-GENERATED REFACTOR
*
* This file was migrated from Enzyme to RTL using generative AI.
* To make the migration as clean as possible, the LLM was instructed to
* use testing patterns similar to Enzyme.
*
* If you are making changes to this file, please consider refactoring
* to more idiomatic RTL (and then removing this banner!).
*/
import React from 'react';
import { render } from '@testing-library/react';
import '@testing-library/jest-dom';
Expand Down Expand Up @@ -278,4 +268,3 @@ describe('<Axis />', () => {
expect(fakeRef.current).toBe(axisElement);
});
});
// MIGRATION STATUS: {"eslint":"pending","jest":{"passed":16,"failed":0,"total":16,"skipped":0,"successRate":100},"tsc":"pending","enyzme":"converted"}
11 changes: 0 additions & 11 deletions packages/visx-axis/test/AxisBottom.test.tsx
Original file line number Diff line number Diff line change
@@ -1,13 +1,3 @@
/**
* LLM-GENERATED REFACTOR
*
* This file was migrated from Enzyme to RTL using generative AI.
* To make the migration as clean as possible, the LLM was instructed to
* use testing patterns similar to Enzyme.
*
* If you are making changes to this file, please consider refactoring
* to more idiomatic RTL (and then removing this banner!).
*/
import React from 'react';
import { render } from '@testing-library/react';
import '@testing-library/jest-dom';
Expand Down Expand Up @@ -100,4 +90,3 @@ describe('<AxisBottom />', () => {
});
});
});
// MIGRATION STATUS: {"eslint":"pending","jest":{"passed":9,"failed":0,"total":9,"skipped":0,"successRate":100},"tsc":"pending","enyzme":"converted"}
11 changes: 0 additions & 11 deletions packages/visx-axis/test/AxisLeft.test.tsx
Original file line number Diff line number Diff line change
@@ -1,13 +1,3 @@
/**
* LLM-GENERATED REFACTOR
*
* This file was migrated from Enzyme to RTL using generative AI.
* To make the migration as clean as possible, the LLM was instructed to
* use testing patterns similar to Enzyme.
*
* If you are making changes to this file, please consider refactoring
* to more idiomatic RTL (and then removing this banner!).
*/
import React from 'react';
import { render } from '@testing-library/react';
import '@testing-library/jest-dom';
Expand Down Expand Up @@ -107,4 +97,3 @@ describe('<AxisLeft />', () => {
expect(elements[0]).toBeInTheDocument();
});
});
// MIGRATION STATUS: {"eslint":"pass","jest":{"passed":9,"failed":0,"total":9,"skipped":0,"successRate":100},"tsc":"pending","enyzme":"converted"}
11 changes: 0 additions & 11 deletions packages/visx-axis/test/AxisRight.test.tsx
Original file line number Diff line number Diff line change
@@ -1,13 +1,3 @@
/**
* LLM-GENERATED REFACTOR
*
* This file was migrated from Enzyme to RTL using generative AI.
* To make the migration as clean as possible, the LLM was instructed to
* use testing patterns similar to Enzyme.
*
* If you are making changes to this file, please consider refactoring
* to more idiomatic RTL (and then removing this banner!).
*/
import React from 'react';
import { render } from '@testing-library/react';
import '@testing-library/jest-dom';
Expand Down Expand Up @@ -87,4 +77,3 @@ describe('<AxisRight />', () => {
expect(labelElement).toHaveTextContent(label);
});
});
// MIGRATION STATUS: {"eslint":"pass","jest":{"passed":6,"failed":0,"total":6,"skipped":0,"successRate":100},"tsc":"pending","enyzme":"converted"}
11 changes: 0 additions & 11 deletions packages/visx-axis/test/AxisTop.test.tsx
Original file line number Diff line number Diff line change
@@ -1,13 +1,3 @@
/**
* LLM-GENERATED REFACTOR
*
* This file was migrated from Enzyme to RTL using generative AI.
* To make the migration as clean as possible, the LLM was instructed to
* use testing patterns similar to Enzyme.
*
* If you are making changes to this file, please consider refactoring
* to more idiomatic RTL (and then removing this banner!).
*/
import React from 'react';
import { render, waitFor } from '@testing-library/react';
import '@testing-library/jest-dom';
Expand Down Expand Up @@ -126,4 +116,3 @@ describe('<AxisTop />', () => {
});
});
});
// MIGRATION STATUS: {"eslint":"pass","jest":{"passed":8,"failed":0,"total":8,"skipped":0,"successRate":100},"tsc":"pending","enyzme":"converted"}
11 changes: 0 additions & 11 deletions packages/visx-axis/test/scales.test.tsx
Original file line number Diff line number Diff line change
@@ -1,13 +1,3 @@
/**
* LLM-GENERATED REFACTOR
*
* This file was migrated from Enzyme to RTL using generative AI.
* To make the migration as clean as possible, the LLM was instructed to
* use testing patterns similar to Enzyme.
*
* If you are making changes to this file, please consider refactoring
* to more idiomatic RTL (and then removing this banner!).
*/
import React from 'react';
import { render } from '@testing-library/react';
import '@testing-library/jest-dom';
Expand Down Expand Up @@ -217,4 +207,3 @@ describe('Axis scales', () => {
expect(getByText('test axis')).toBeInTheDocument();
});
});
// MIGRATION STATUS: {"eslint":"pending","jest":{"passed":12,"failed":0,"total":12,"skipped":0,"successRate":100},"tsc":"pending","enyzme":"converted"}
11 changes: 0 additions & 11 deletions packages/visx-chord/test/Chord.test.tsx
Original file line number Diff line number Diff line change
@@ -1,13 +1,3 @@
/**
* LLM-GENERATED REFACTOR
*
* This file was migrated from Enzyme to RTL using generative AI.
* To make the migration as clean as possible, the LLM was instructed to
* use testing patterns similar to Enzyme.
*
* If you are making changes to this file, please consider refactoring
* to more idiomatic RTL (and then removing this banner!).
*/
import React from 'react';
import { render } from '@testing-library/react';
import { Chord } from '../src';
Expand All @@ -32,4 +22,3 @@ describe('<Chord />', () => {
expect(args.chords).toBeDefined();
});
});
// MIGRATION STATUS: {"eslint":"pending","jest":{"passed":2,"failed":0,"total":2,"skipped":0,"successRate":100},"tsc":"pending","enyzme":"converted"}
11 changes: 0 additions & 11 deletions packages/visx-chord/test/Ribbon.test.tsx
Original file line number Diff line number Diff line change
@@ -1,13 +1,3 @@
/**
* LLM-GENERATED REFACTOR
*
* This file was migrated from Enzyme to RTL using generative AI.
* To make the migration as clean as possible, the LLM was instructed to
* use testing patterns similar to Enzyme.
*
* If you are making changes to this file, please consider refactoring
* to more idiomatic RTL (and then removing this banner!).
*/
import React from 'react';
import { render } from '@testing-library/react';
import { chord as d3Chord } from 'd3-chord';
Expand Down Expand Up @@ -35,4 +25,3 @@ describe('<Ribbon />', () => {
expect(args.path).toBeDefined();
});
});
// MIGRATION STATUS: {"eslint":"pending","jest":{"passed":2,"failed":0,"total":2,"skipped":0,"successRate":100},"tsc":"pending","enyzme":"converted"}
11 changes: 0 additions & 11 deletions packages/visx-clip-path/test/ClipPaths.test.tsx
Original file line number Diff line number Diff line change
@@ -1,13 +1,3 @@
/**
* LLM-GENERATED REFACTOR
*
* This file was migrated from Enzyme to RTL using generative AI.
* To make the migration as clean as possible, the LLM was instructed to
* use testing patterns similar to Enzyme.
*
* If you are making changes to this file, please consider refactoring
* to more idiomatic RTL (and then removing this banner!).
*/
import React from 'react';
import { render } from '@testing-library/react';
import '@testing-library/jest-dom';
Expand Down Expand Up @@ -102,4 +92,3 @@ describe('<CircleClipPath />', () => {
expect(circle).toHaveAttribute('cy', '200');
});
});
// MIGRATION STATUS: {"eslint":"pending","jest":{"passed":10,"failed":0,"total":10,"skipped":0,"successRate":100},"tsc":"pending","enyzme":"converted"}
11 changes: 0 additions & 11 deletions packages/visx-delaunay/test/Polygon.test.tsx
Original file line number Diff line number Diff line change
@@ -1,13 +1,3 @@
/**
* LLM-GENERATED REFACTOR
*
* This file was migrated from Enzyme to RTL using generative AI.
* To make the migration as clean as possible, the LLM was instructed to
* use testing patterns similar to Enzyme.
*
* If you are making changes to this file, please consider refactoring
* to more idiomatic RTL (and then removing this banner!).
*/
import React from 'react';
import { render } from '@testing-library/react';
import '@testing-library/jest-dom'; // Added this import
Expand Down Expand Up @@ -57,4 +47,3 @@ describe('<Polygon />', () => {
expect(path?.getAttribute('fill')).toBe('orange');
});
});
// MIGRATION STATUS: {"eslint":"pending","jest":{"passed":5,"failed":0,"total":5,"skipped":0,"successRate":100},"tsc":"pending","enyzme":"converted"}
11 changes: 0 additions & 11 deletions packages/visx-geo/test/Projection.test.tsx
Original file line number Diff line number Diff line change
@@ -1,13 +1,3 @@
/**
* LLM-GENERATED REFACTOR
*
* This file was migrated from Enzyme to RTL using generative AI.
* To make the migration as clean as possible, the LLM was instructed to
* use testing patterns similar to Enzyme.
*
* If you are making changes to this file, please consider refactoring
* to more idiomatic RTL (and then removing this banner!).
*/
import React from 'react';
import { render } from '@testing-library/react';
import '@testing-library/jest-dom';
Expand Down Expand Up @@ -103,4 +93,3 @@ describe('<Projection />', () => {
expect(centroid.mock.calls[0][1]).toEqual(expect.any(Object));
});
});
// MIGRATION STATUS: {"eslint":"pending","jest":{"passed":6,"failed":0,"total":6,"skipped":0,"successRate":100},"tsc":"pending","enyzme":"converted"}
11 changes: 0 additions & 11 deletions packages/visx-glyph/test/Circle.test.tsx
Original file line number Diff line number Diff line change
@@ -1,13 +1,3 @@
/**
* LLM-GENERATED REFACTOR
*
* This file was migrated from Enzyme to RTL using generative AI.
* To make the migration as clean as possible, the LLM was instructed to
* use testing patterns similar to Enzyme.
*
* If you are making changes to this file, please consider refactoring
* to more idiomatic RTL (and then removing this banner!).
*/
import React from 'react';
import { render } from '@testing-library/react';
import '@testing-library/jest-dom';
Expand Down Expand Up @@ -70,4 +60,3 @@ describe('<GlyphCircle />', () => {
expect(args.path.size()()).toBe(42);
});
});
// MIGRATION STATUS: {"eslint":"pending","jest":{"passed":7,"failed":0,"total":7,"skipped":0,"successRate":100},"tsc":"pending","enyzme":"converted"}
11 changes: 0 additions & 11 deletions packages/visx-glyph/test/Cross.test.tsx
Original file line number Diff line number Diff line change
@@ -1,13 +1,3 @@
/**
* LLM-GENERATED REFACTOR
*
* This file was migrated from Enzyme to RTL using generative AI.
* To make the migration as clean as possible, the LLM was instructed to
* use testing patterns similar to Enzyme.
*
* If you are making changes to this file, please consider refactoring
* to more idiomatic RTL (and then removing this banner!).
*/
import React from 'react';
import { render } from '@testing-library/react';
import '@testing-library/jest-dom';
Expand Down Expand Up @@ -67,4 +57,3 @@ describe('<GlyphCross />', () => {
expect(childrenFn.mock.calls[0][0].path.size()()).toBe(42);
});
});
// MIGRATION STATUS: {"eslint":"pending","jest":{"passed":6,"failed":0,"total":6,"skipped":0,"successRate":100},"tsc":"pending","enyzme":"converted"}
11 changes: 0 additions & 11 deletions packages/visx-glyph/test/Diamond.test.tsx
Original file line number Diff line number Diff line change
@@ -1,13 +1,3 @@
/**
* LLM-GENERATED REFACTOR
*
* This file was migrated from Enzyme to RTL using generative AI.
* To make the migration as clean as possible, the LLM was instructed to
* use testing patterns similar to Enzyme.
*
* If you are making changes to this file, please consider refactoring
* to more idiomatic RTL (and then removing this banner!).
*/
import React from 'react';
import { render } from '@testing-library/react';
import '@testing-library/jest-dom';
Expand Down Expand Up @@ -88,4 +78,3 @@ describe('<GlyphDiamond />', () => {
expect(args.path.size()()).toBe(42);
});
});
// MIGRATION STATUS: {"eslint":"pending","jest":{"passed":7,"failed":0,"total":7,"skipped":0,"successRate":100},"tsc":"pending","enyzme":"converted"}
11 changes: 0 additions & 11 deletions packages/visx-glyph/test/Glyph.test.tsx
Original file line number Diff line number Diff line change
@@ -1,13 +1,3 @@
/**
* LLM-GENERATED REFACTOR
*
* This file was migrated from Enzyme to RTL using generative AI.
* To make the migration as clean as possible, the LLM was instructed to
* use testing patterns similar to Enzyme.
*
* If you are making changes to this file, please consider refactoring
* to more idiomatic RTL (and then removing this banner!).
*/
import React from 'react';
import { render } from '@testing-library/react';
import '@testing-library/jest-dom';
Expand Down Expand Up @@ -47,4 +37,3 @@ describe('<Glyph />', () => {
expect(glyph).toHaveAttribute('transform', 'translate(2, 2)');
});
});
// MIGRATION STATUS: {"eslint":"pending","jest":{"passed":4,"failed":0,"total":4,"skipped":0,"successRate":100},"tsc":"pending","enyzme":"converted"}
11 changes: 0 additions & 11 deletions packages/visx-glyph/test/Square.test.tsx
Original file line number Diff line number Diff line change
@@ -1,13 +1,3 @@
/**
* LLM-GENERATED REFACTOR
*
* This file was migrated from Enzyme to RTL using generative AI.
* To make the migration as clean as possible, the LLM was instructed to
* use testing patterns similar to Enzyme.
*
* If you are making changes to this file, please consider refactoring
* to more idiomatic RTL (and then removing this banner!).
*/
import React from 'react';
import { render } from '@testing-library/react';
import '@testing-library/jest-dom';
Expand Down Expand Up @@ -78,4 +68,3 @@ describe('<GlyphSquare />', () => {
expect(args.path.size()()).toBe(42);
});
});
// MIGRATION STATUS: {"eslint":"pending","jest":{"passed":7,"failed":0,"total":7,"skipped":0,"successRate":100},"tsc":"pending","enyzme":"converted"}
Loading