parse-community / parse-php-sdk

The PHP SDK for Parse Platform
https://parseplatform.org/
Other
811 stars 346 forks source link

Add Indexes via Schema #357

Closed dplewis closed 6 years ago

dplewis commented 7 years ago

Pending: https://github.com/parse-community/parse-server/pull/4240

montymxb commented 7 years ago

Cool! Same deal here as with the aggregate queries. Once the server side functionality is in I'll go over this.

dplewis commented 6 years ago

@montymxb I added documentation and fixed the tests. The test weren't working locally at first until I updated my version of node from 6.10 to 9.2. I know the latest version of parse-server requires a minimum node version 6.11.4. Can you have a look into travis?

montymxb commented 6 years ago

Sure thing @dplewis , I'll see what's going on here...

montymxb commented 6 years ago

Took a look locally, seems to be just an outdated version of parse-server in cache. Dumped it and rerunning now. I did see one failure which looks to be consistent and is beyond this PR, probably due to a change in the server. We'll see if it reproduces when these run through.

dplewis commented 6 years ago

Are you talking about the testBadSchemaDelete or testCheckBadServer tests? I got those failing when running locally.

montymxb commented 6 years ago

Nope, schema tests look good actually. This is what I saw locally.

1) Parse\Test\ParseFileTest::testParseFileTypes
Failed asserting that two strings are equal.
--- Expected
+++ Actual
@@ @@
-'application/octet-stream'
+'null'

Curious if it's just a local issue or if it will repeat up here. If so I'll need to take a further look.

dplewis commented 6 years ago

@montymxb I got that as well

montymxb commented 6 years ago

Looks like there's still further issues starting up the test server. I'll have to look at this tomorrow when I have some time.

montymxb commented 6 years ago

@dplewis master is good now, you can merge against it when you have a moment.

codecov[bot] commented 6 years ago

Codecov Report

Merging #357 into master will increase coverage by <.01%. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #357      +/-   ##
==========================================
+ Coverage   99.01%   99.02%   +<.01%     
==========================================
  Files          38       38              
  Lines        3562     3579      +17     
==========================================
+ Hits         3527     3544      +17     
  Misses         35       35
Impacted Files Coverage Δ
src/Parse/ParseSchema.php 97.3% <100%> (+0.22%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update b8fdecc...c3531e7. Read the comment docs.

dplewis commented 6 years ago

@montymxb thanks for fixing the tests

montymxb commented 6 years ago

Sweet we're ready for 1.4.0. I'll craft a release later tonight.