Skip to content
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.

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

Chore: Replace ts-jest mocked with jest.mocked #44630

Merged
merged 1 commit into from Jan 31, 2022
Merged

Chore: Replace ts-jest mocked with jest.mocked #44630

merged 1 commit into from Jan 31, 2022

Conversation

joshhunt
Copy link
Contributor

What this PR does / why we need it:

The mocked function from ts-jest/utils was moved into Jest core jestjs/jest#12089 (comment)

This PR removed about 1,103 lines of deprecation warnings from the tests 馃く

Which issue(s) this PR fixes:

Part of #42359

@joshhunt joshhunt added type/chore no-backport Skip backport of PR labels Jan 31, 2022
@joshhunt joshhunt requested review from mckn, a team, andresmgot, vickyyyyyyy, axelavargas and kaydelaney and removed request for a team January 31, 2022 03:21
@grafanabot grafanabot added datasource/Azure Azure Monitor Datasource area/frontend labels Jan 31, 2022
@joshhunt joshhunt added this to the 8.4.0 milestone Jan 31, 2022
@joshhunt joshhunt enabled auto-merge (squash) January 31, 2022 06:07
andresmgot
andresmgot approved these changes Jan 31, 2022
View reviewed changes
Copy link
Contributor

@andresmgot andresmgot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

馃憤

@joshhunt joshhunt merged commit d2f80c3 into main Jan 31, 2022
@joshhunt joshhunt deleted the joshhunt/test-logs-jest-mocked branch January 31, 2022 08:29
ryantxu pushed a commit that referenced this pull request Feb 1, 2022
@joshhunt @ryantxu
Chore: Replace ts-jest mocked with jest.mocked (#44630)
fff88d3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@andresmgot andresmgot andresmgot approved these changes

@mckn mckn Awaiting requested review from mckn

@vickyyyyyyy vickyyyyyyy Awaiting requested review from vickyyyyyyy vickyyyyyyy was automatically assigned from grafana/aws-datasources

@axelavargas axelavargas Awaiting requested review from axelavargas axelavargas was automatically assigned from grafana/grafana-frontend-platform

@kaydelaney kaydelaney Awaiting requested review from kaydelaney kaydelaney was automatically assigned from grafana/grafana-frontend-platform

Assignees
No one assigned
Labels
area/frontend datasource/Azure Azure Monitor Datasource no-backport Skip backport of PR type/chore
Projects
Grafana Frontend Platform
Archived in project
Milestone
8.4.0-beta1
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@joshhunt @andresmgot @grafanabot