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.

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

Add forgotten classes related to the legacy InlineExpectationsTestclass #13261

Merged
merged 1 commit into from May 24, 2023

Conversation

jketema
Copy link
Contributor

@jketema jketema commented May 23, 2023

Without these some Python tests are broken. I verified locally that this fixes them.

Unfortunately, this requires renaming the related classes in the parameterised modules, otherwise once a test is rewritten import MakeTest<Test> will cause naming conflicts.

@jketema jketema requested a review from yoff May 23, 2023 14:50
@jketema jketema marked this pull request as draft May 23, 2023 14:56
@jketema jketema marked this pull request as ready for review May 23, 2023 15:41
exists(ValidExpectation expectation |
not exists(ActualResult actualResult | expectation.matchesActualResult(actualResult)) and
exists(ValidTestExpectation expectation |
not exists(ActualTestResult actualResult | expectation.matchesActualResult(actualResult)) and

Check warning

Code scanning / CodeQL

Omittable 'exists' variable Warning

This exists variable can be omitted by using a don't-care expression
in this argument
.
Copy link
Contributor

@michaelnebel michaelnebel left a comment

Choose a reason for hiding this comment

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

LGTM!

@jketema jketema merged commit eea972b into github:main May 24, 2023
10 checks passed
@jketema jketema deleted the inline-legacy-classes branch May 24, 2023 08:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants