fix: Saving object with a pointer field set to undefined throws error for Postgres
#9147
Conversation
|
I will reformat the title to use the proper commit message syntax. |
parse-github-assistant
bot
changed the title
Thanks for opening this pull request! |
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>
spec/PostgresStorageAdapter.spec.js
Outdated
| @@ -570,6 +570,35 @@ describe_only_db('postgres')('PostgresStorageAdapter', () => { | |||
| await adapter.deleteIdempotencyFunction(); | |||
| await client.none(qs); | |||
| }); | |||
|
|
|||
| it('saves object with a pointer field set to undefined', async () => { | |||
There was a problem hiding this comment.
This test should be moved to where the existing pointer tests are, as it's not a postgres specific test, even if the issue occurs only with postgres.
Signed-off-by: Manuel <5673677+mtrezza@users.noreply.github.com>
mtrezza
changed the title
mtrezza
changed the title
undefined throws error for Postgres
|
Test passes for Postgres and MongoDB. Is this issue related to your custom code? |
I just found out how to test with postgres locally so I'm gonna try setting that up properly first. My issue might be because I didn't configure postgres for parse correctly. |
|
@andrewalc sure, let us know how it goes, and whether the related issue can be closed. |
posted an update in the issue here, might try again if i find some time but it seems like the case i have refuses to fail in this repo |
Pull Request
Issue
Closes: #9146
Approach
Work in progress, attempting to write a failing test case. Need to see if it fails for Postgres in CI (afaik i can only test on mongo locally. It passes for mongo but shouldn't for postgres)
Tasks