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

Adding new spec helpers file #42

Merged
merged 2 commits into from
May 14, 2021
Merged

Adding new spec helpers file #42

merged 2 commits into from
May 14, 2021

Conversation

jwoertink
Copy link
Member

Fixes #21

The file right now just requires the migrations. This is a required step in order to run specs with an app that's using Breeze.

Without this step, you'll get the following error:

❯ crystal spec


Finished in 281.44 milliseconds
0 examples, 0 failures, 0 errors, 0 pending
Unhandled exception: Breeze::BreezePipe wants to use the 'breeze_pipes' table but it is missing.

If you need to create the 'breeze_pipes' table...

  ▸ Generate a migration:

      lucky gen.migration CreateBreeze::BreezePipes

Which could be confusing for someone new to Lucky. You currently have to add require "breeze/db/migrations/**" which is fine, but we may want to add other additional helper stuff later. Now we have a file we can add additional things to.

jwoertink added 2 commits May 12, 2021 19:23
…ing possibly due to the code blocks between the list items...
@jwoertink jwoertink requested a review from matthewmcgarvey May 14, 2021 21:54
@jwoertink jwoertink merged commit 280c52a into master May 14, 2021
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

Successfully merging this pull request may close these issues.

Integrating with specs
2 participants