Support server-url to specify api server

[sdf611097]

lots of bugs in new API-SERVER, so test failed

object __type check is missing
role name check is missing. Role names must be specified. code 135
file/push implementation is changed
users get will get additional ACL info
user test failed by recovable session is required

newServerTest.sh need start local ParseServer.

You can start it from
https://github.com/sdf611097/parse-server-example.git
sh kaiseki.sh

[shiki]

Thanks for this. I will look into this properly before merging.

[arikw]

+1

[virmelcruz]

I also need this, 👍 +1

[sdf611097]

I modify test to fit original/new parse server as best as I can.
But there still have 1 fail(same as previous) in origin ParseServer.
3 fail in new ParseServer(2 file 1 push)

https://github.com/sdf611097/parse-server-example.git
I've change db to local for test.
Original one is corrupt so that user.test.js is failed.

U can run cmd below in parse-server-example to start local env for parse server.
sh kaiseki.sh

After that, execute below cmd in kaiseki to run the test with this env
sh newServerTest.sh

but there are still 3 failure

1. file can upload simple text as a file:

  Uncaught AssertionError: expected 'text/plain; charset=utf-8' to equal 'text/plain'
  + expected - actual

  -text/plain; charset=utf-8
  +text/plain

This is little different from original, doesn't matter.

2. file can delete a file (using API masterkey):

  Uncaught AssertionError: expected 404 to equal 403
  + expected - actual

  -404
  +403

This test is also failed in the origin test, but response is different.
ParseFile implementation is changed.
I haven't deal with it.

3. push can broadcast a notification:

this one should fail with 'To push to ios devices, you must first configure a valid certificate.'

but I'm not familiar with push, maybe is due to my local env is empty, the target is empty so that pass.

[sdf611097]

@shiki will this pr be accepted, merged and npm publish a new kaiseki version?

[shiki]

Why didn't you just put the /1 as part of the serverUrl?

[sdf611097]

Open-source ParseServer can specified mount path. Parse.com use /1, and the example use /parse. So the serverUrl is combination of host-server(:port) and its mount path. "api.parse.com/1" and "localhost:1337/parse"

[shiki]

But isn't that just the same as giving a server url with the "/1" or the "/parse" though?

new Kaiseki(.... "api.parse.com/1")

or

new Kaiseki(... "localhost:1337/parse")

[shiki]

I am. I'm having trouble to actually get this to work with the default parse server. I think we should have something that does not require a customized parse-server repository.

I'll push this in a different branch for now and work on that branch.

Do you think we should just have revocable sessions turned on by default and not let that get configured? That's probably gonna make things less complicated.

[sdf611097]

Revocable sessions turn on by default is ok. I think it should be configurable, or it will not compatible with legacy-sessions. Someone are trying to let parse-server support legacy token.

Maybe I separate the commits? Configurable first, then support switch parse-server url?

I think we should have something that does not require a customized parse-server repository.

I do not catch this sentence.
You are trying https://github.com/ParsePlatform/parse-server ? or https://github.com/ParsePlatform/parse-server-example ?

Sorry for my poor English.

[shiki]

Maybe I separate the commits? Configurable first, then support switch parse-server url?

Don't worry about it. I pushed this to the parse-server branch for now.

I am trying to get this working using the default config from https://github.com/ParsePlatform/parse-server-example.

I'll merge the parse-server branch after making the tests working. I'm also gonna remove the remaining unnecessary code for supporting the old Parse API.

[sdf611097]

If you have any problem, plz let me know.

[shiki]

This has been merged to master and pushed to NPM. I dropped the support for the old Parse to make things easier.

[sdf611097]

Thanks for your merge and publish. ^^