Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

libtest/cmd.go: potentially unnecessary step in documentation #170

Closed
orlangure opened this issue Jan 8, 2021 · 4 comments
Closed

libtest/cmd.go: potentially unnecessary step in documentation #170

orlangure opened this issue Jan 8, 2021 · 4 comments

Comments

@orlangure
Copy link

Hi there!

I'm the maintainer of Gnomock, and I'm reaching out because I noticed you started using it in libtest/cmd.go just yesterday. Thanks for trying it out!

I also noticed a comment above main function in libtest/cmd.go stating that the following command needs to be executed before running it:

docker run --rm -p 23042:23042 -v /var/run/docker.sock:/var/run/docker.sock -v /Users/vikram/src/graphjin/libtest:/Users/vikram/src/graphjin/libtest orlangure/gnomock

I'm wondering what would be the reason to do so?

From what I'm seeing, you don't have to run orlangure/gnomock container. You could just use gnomock.Start() with whatever mysql/postgres preset you configured. orlangure/gnomock container is only required when your code is not written in Go: in such cases you could use Gnomock over HTTP from other languages, like python.

Can you try running the command without any prior setup?

@dosco
Copy link
Owner

dosco commented Jan 8, 2021

Firstly, I want to just say thank you for Gnomock. I love the project and I'm working on moving all my tests to use it. I re-read your project README, I must have misunderstood. The section was titled Using Gnomock server and I just assumed it needed me to run a separate server I somehow didn't expect a Go library to internal launch and manage databases (I love that it does). Thanks for reaching out I'll fix this and drop you a note once I have all my tests running against multi databases using your library.

Do you have thoughts on adding Yugabyte and Cockroach both of which are Postgres compatible databases with fast growing adoption?

@orlangure
Copy link
Author

Do you have thoughts on adding Yugabyte and Cockroach both of which are Postgres compatible databases with fast growing adoption?

I opened new issues for CockroachDB (orlangure/gnomock#92) and Yugabyte (orlangure/gnomock#93), but I don't have any experience with any of them. It would be great if you could help to define preset requirements. Let's continue the discussion there 😼

@dosco
Copy link
Owner

dosco commented Jan 12, 2021

Committed my first set of examples / integration tests using gnomock. It was what I wanted to do over a year go thanks for this project its very useful. Also it worked perfectly when the tests ran within a Github action. 345fe8d

@orlangure
Copy link
Author

Great to hear, and thank you!
I have a draft for cockroachdb ready, so if you still need these two presets, please go ahead and let me know in the linked issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants