apis-is / apis

Making data readily available to anyone interested
https://apis.is
MIT License
162 stars 64 forks source link

Aircraft endpoint CircleCI test errors #481

Closed valentinoli closed 5 years ago

valentinoli commented 5 years ago

Hey all. I've been trying to get this pull request to pass the integration tests without any luck... Keep getting the following errors even though the endpoint works perfectly locally. Any ideas what might be causing the refusal of samgongustofa.is to respond? Both errors are due to this.

  1) aircraft
       correct-fields
         should return an array of objects containing correct fields:
     Uncaught AssertionError: Does not contain a 'results' field
      at assertResults (lib/test_helpers.js:7:3)
      at Request._callback (lib/test_helpers.js:59:5)
      at Request.self.callback (node_modules/request/request.js:186:22)
      at Request.<anonymous> (node_modules/request/request.js:1163:10)
      at IncomingMessage.<anonymous> (node_modules/request/request.js:1085:12)
      at endReadableNT (_stream_readable.js:975:12)
      at _combinedTickCallback (internal/process/next_tick.js:80:11)
      at process._tickDomainCallback (internal/process/next_tick.js:128:9)

  2) aircraft
       aircraft-not-found
         should return a 404 when an aircraft is not found:

      Uncaught AssertionError: 'www.samgongustofa.is refuses to respond or give back data' === 'No aircraft found with the query loftur'
      + expected - actual

      -www.samgongustofa.is refuses to respond or give back data
      +No aircraft found with the query loftur
MiniGod commented 5 years ago

Please keep conversation in the pull request #480.

valentinoli commented 5 years ago

Will do next time 👍