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

new rules: prefer-toBeNull / prefer-toBeTrue / prefer-toBeFalse #352

Open
rklec opened this issue Jun 28, 2022 · 0 comments
Open

new rules: prefer-toBeNull / prefer-toBeTrue / prefer-toBeFalse #352

rklec opened this issue Jun 28, 2022 · 0 comments

Comments

@rklec
Copy link

rklec commented Jun 28, 2022

Is your feature request related to a problem? Please describe.

Incorrect code should be:

expect(value).toBe(null);
expect(value).toBe(null, 'with an explanation');

Describe the solution you'd like

It should sugest a correction to

expect(value).toBeNull();
expect(value).toBeNull('with an explanation');

This exists since jasmine v1.3.0.

Describe alternatives you've considered

A clear and concise description of any alternative solutions or features you've considered

Additional context
The same may be done with toBeTrue()/toBeFalse().

These are all very similar to prefer-toBeUndefined and can easily be auto-fixed.

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

No branches or pull requests

1 participant