Clean authData of null values on _User update

[yuzeh]

Note: I haven't been able to test this because I'm still trying to figure out how to get my project to depend on my fork of parse-server.

---

Adds a step to the RestWrite#execute chain: it cleans the response
authData object of null values.

For example, this:

{"authData": {"anonymous": null}, "updatedAt", ...}

will be transformed to this:

{"updatedAt", ...}

And this:

{"authData": {"anonymous": null, "twitter": ...}, "updatedAt", ...}

will be transformed to this:

{"authData": {"twitter": ...}, "updatedAt", ...}

Fixing this issue will fix anonymous user upgrades from the Android SDK.

[codecov-io]

Current coverage is 93.00%

Merging #1199 into master will increase coverage by +0.03% as of b6201b6

@@            master   #1199   diff @@
======================================
  Files           84      84       
  Stmts         5265    5275    +10
  Branches       956     960     +4
  Methods          0       0       
======================================
+ Hit           4895    4906    +11
  Partial         11      11       
+ Missed         359     358     -1

Review entire Coverage Diff as of b6201b6

Powered by Codecov. Updated on successful CI builds.

[flovilmart]

@yuzeh please add unit tests to make sure the fix is correct and doing what's expected the cleaning is currently tested here https://github.com/ParsePlatform/parse-server/blob/master/spec/OAuth.spec.js#L263

[yuzeh]

Hi @flovilmart, I've added a test case that exercises this functionality (if I undo the code in src/, the test will fail). I'm not sure if this is the best place to put it, though. Please advise!

[facebook-github-bot]

@yuzeh updated the pull request.

[flovilmart]

Can you rebase / squash all those commits?

[yuzeh]

@flovilmart done.

BTW, I noticed I currently can't use my fork of parse-server as a dependency in NPM. Is there an easy way to fix this?

[flovilmart]

There is still an extra commit....

[flovilmart]

And to answer your question, there is an 'easy' way through git submodules and npm link

[j-koenig]

I'm still having a problem with a user having authData set to {"facebook": null} running parse-server 2.2.10