search

drakerian / loopback-connector-elastic-search

Strongloop Loopback connector for Elasticsearch
MIT License
33 stars 86 forks source link

Test does not seems to run properly #9

Open smartmouse opened 8 years ago

smartmouse commented 8 years ago

When we do npm test. There are npm ERR! Test failed. See above for more details.

I can see that there are lots of tests defined in the test folder. But in the package.json,
"scripts": { "test": "test" }

We should change it to mocha.

loopback-connector-elastic-search@0.0.2-a test /Users/ivyho/loopback-connector-elastic-search mocha

(node) child_process: options.customFds option is deprecated. Use options.stdio instead.

When apply filter criteria ✓ should validate index and type filters ✓ should validate filter criteria

When make id document ✓ should validate id created with Index filter name

When convert elastic result to loopback model ✓ should run convert function ✓ should validate matched fields ✓ should validate data types

When set elastic client options ✓ should return number for defaultSize option ✓ should return string type for searchIndex and searchType ✓ should return a valid URL for elastic client

9 passing (19ms)

pulkitsinghal commented 8 years ago

@smartmouse - may i suggest you use a slightly more active/up-2-date/current/official~ish fork instead?

Here: https://github.com/strongloop-community/loopback-connector-elastic-search

smartmouse commented 8 years ago

sure. Thank you for the pointer. There are some failure in the new tests, though. Not sure if there are any setup required.

https://gist.github.com/smartmouse/36e6e122d1f3d82d5fc5

pulkitsinghal commented 8 years ago

@smartmouse - yes to test you need to point at an ES server, please refer to the readme in the repo mentioned above, it should talk about this and if it doesn't then open an issue describing what's missing or confusing in that repo's readme

Also use the gitter.im chat room of that repo to get help quicker