Update test.spec.js

[mrmarcsmith]

added tests for additional methods of passing S3 Overrides.

[mrmarcsmith]

Hi All,
I have fixed this conflict but it needs approval from someone with write access

[flovilmart]

@mrmarcsmith thanks for extending the coverage! Let's wait for the CI to finish before merging those.

[mrmarcsmith]

Hi @flovilmart ,
On my end it says "This Branch has a conflict that must be resolved". I updated my commit to correct the conflict but the merge needs approval before the CI can kick in. here is the link to review: https://github.com/parse-server-modules/parse-server-s3-adapter/pull/39/conflicts

[flovilmart]

Seems that your commit didn't fix the conflicts but now it's ok

[mrmarcsmith]

@flovilmart this test checks pull request #37 which hasn't been approved yet. I was asked by @acinader to write this test before he would approve it.

[acinader]

ah, I see. so, in your local box you should rebase (or merge) patch-2 onto patch-1 and then push it to git hub. that way the two commits will show up in the same pr and we can see the test passing.

k? If you're not sure how to do, I could do for you....

[mrmarcsmith]

this issue was moved to PR #40

[flovilmart]

Closing as moved to #40