search

multiprocessio / datastation

App to easily query, script, and visualize data from every database, file, and API.
https://datastation.multiprocess.io
Other
2.9k stars 112 forks source link

Add Neo4j support #219

Closed Posrabi closed 2 years ago

Posrabi commented 2 years ago
eatonphil commented 2 years ago

Really nice work overall! Were you able to test this out inside of the DataStation UI?

Posrabi commented 2 years ago

Really nice work overall! Were you able to test this out inside of the DataStation UI?

Yeah, everything work as expected

eatonphil commented 2 years ago

You should also add a test to https://github.com/multiprocessio/datastation/blob/main/desktop/panel/database.test.js#L680. Feel free to copy the Prometheus tests and just change the parts to Neo4j.

You can test this with 

yarn test-local desktop/panel/database.test.js -t 'neo4j'

Line 694 is the query panel.content = 'up'. So you'll replace up with your own query.

Second step: after that you'll also have to add steps to https://github.com/multiprocessio/datastation/blob/main/scripts/ci/prepare_linux_integration_test_setup_only.sh to turn on a neo4j docker container and load it with some data for this test. Look at how influx or prometheus are set up there.

But you can skip that second step while you're working on the test locally.