Skip to content

Avoid snapshot in operationDetailsTable/index.test.js #2659

Open
@yurishkuro

Description

packages/jaeger-ui/src/components/Monitor/ServicesView/operationDetailsTable/index.test.js

The automated upgrade of AntD in #2658 failed because the snapshot needed to be regenerated, for meaningless changes like this:

-  className="ant-col ant-col-24 css-dev-only-do-not-override-cdzvx5"
+  className="ant-col ant-col-24 css-dev-only-do-not-override-1yacf91"

Our tests should not be depending on implementation details of the 3rd party library. The goal is to rewrite this test using react testing library practices.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions