search

graphiti-api / graphiti-rails

MIT License
53 stars 28 forks source link

Move more tests here #75

Open wagenet opened 2 years ago

wagenet commented 2 years ago

Right now, most tests live in the main graphiti repo. We should move the graphiti-rails ones here.

richmolj commented 2 years ago

I agree conceptually, but particularly for integration specs I'm not sure. Seems like we could get into a situation where an update to graphiti codebase breaks the overall behavior users expect, but we don't know about it because the PR ran specs for graphiti, not graphiti+graphiti-rails

wagenet commented 2 years ago

@richmolj we’d probably want some on both sides ideally.