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

test: Failing test case for constraint notEqualTo null #8835

Open
wants to merge 11 commits into
base: alpha
Choose a base branch
from

Conversation

RahulLanjewar93
Copy link

@RahulLanjewar93 RahulLanjewar93 commented Dec 14, 2023

failing Test case for issue #8834
adding this so pr is not closed. closes #8834

image

Copy link

I will reformat the title to use the proper commit message syntax.

@parse-github-assistant parse-github-assistant bot changed the title test: failing test case for constraint notEqualTo null test: Failing test case for constraint notEqualTo null Dec 14, 2023
Copy link

parse-github-assistant bot commented Dec 14, 2023

Thanks for opening this pull request!

  • ❌ Please link an issue that describes the reason for this pull request, otherwise your pull request will be closed. Make sure to write it as Closes: #123 in the PR description, so I can recognize it.

Copy link
Member

@mtrezza mtrezza left a comment

Choose a reason for hiding this comment

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

Please revert the changes in package-lock.json

Copy link
Member

@mtrezza mtrezza left a comment

Choose a reason for hiding this comment

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

This test does not expect anything, please add a check to the test

@RahulLanjewar93
Copy link
Author

RahulLanjewar93 commented Dec 16, 2023

This test does not expect anything, please add a check to the test

Actually, the server itself crashes since we are not handling the error thrown inside matchesKeyConstraints.
Which is why i have not added any expect

@RahulLanjewar93 RahulLanjewar93 force-pushed the test-live-query-failing-for-notEqualTo-null branch from e8137a6 to cced89c Compare December 16, 2023 07:01
spec/ParseLiveQuery.spec.js Outdated Show resolved Hide resolved
@RahulLanjewar93 RahulLanjewar93 changed the title test: Failing test case for constraint notEqualTo null test: failing test case for constraint notEqualTo null Dec 16, 2023
Copy link

I will reformat the title to use the proper commit message syntax.

@parse-github-assistant parse-github-assistant bot changed the title test: failing test case for constraint notEqualTo null test: Failing test case for constraint notEqualTo null Dec 16, 2023
Signed-off-by: Manuel <5673677+mtrezza@users.noreply.github.com>
Signed-off-by: Manuel <5673677+mtrezza@users.noreply.github.com>
Signed-off-by: Manuel <5673677+mtrezza@users.noreply.github.com>
Signed-off-by: Manuel <5673677+mtrezza@users.noreply.github.com>
Signed-off-by: Manuel <5673677+mtrezza@users.noreply.github.com>
Signed-off-by: Manuel <5673677+mtrezza@users.noreply.github.com>
Signed-off-by: Manuel <5673677+mtrezza@users.noreply.github.com>
@mtrezza
Copy link
Member

mtrezza commented Dec 16, 2023

I believe the test looks good now; do you have any suggestion for a fix?

@RahulLanjewar93
Copy link
Author

I believe the test looks good now; do you have any suggestion for a fix?

if (compareTo.__type) {

this line should have optional chaining in my opinion

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.

[Live Query] matchesKeyConstraints throwing error when constraints is {$ne : null}
2 participants