search

eugenezadorin / airtable-php

Simple PHP client for Airtable API
20 stars 7 forks source link

Is the link to test database in the README correct? #8

Closed defunctl closed 1 year ago

defunctl commented 1 year ago

Hi!

I wanted to provide some pull requests for this library.

I copied the readonly database to my own account after forking this repo and 11 of the 83 tests failed.

Is there something wrong with the database or are the tests also failing on your end?

image

Please let me know.

eugenezadorin commented 1 year ago

Hello! I've got the same problem with a cloned database.

Tests failed due to the fact that I did not explicitly specify the order of records. For some reason, in my own test database, the records came in the expected order and the tests didn't fail.

In the cloned database, the order of the records was different. Now I specify sorting explicitly, so everything should work.

Thank you for participating!

defunctl commented 1 year ago

All passing in my fork now, thanks!