Skip to content

add books package for example#568

Merged
blgm merged 1 commit into
onsi:masterfrom
jianzhangbjz:books
Jun 10, 2020
Merged

add books package for example#568
blgm merged 1 commit into
onsi:masterfrom
jianzhangbjz:books

Conversation

@jianzhangbjz

Copy link
Copy Markdown
Contributor

When I following this adding-specs-to-a-suite to practice the Ginkgo. I found no books package as a sample. So, I add it, but I haven't add all the functions for it. I draft this PR for discussing it. Comments are welcome!

mac:books jianzhang$ pwd
/Users/jianzhang/goproject/src/github.com/onsi/ginkgo/example/books
mac:books jianzhang$ ginkgo 
Running Suite: Books Suite
==========================
Random Seed: 1554195323
Will run 2 of 2 specs

••
Ran 2 of 2 Specs in 0.000 seconds
SUCCESS! -- 2 Passed | 0 Failed | 0 Pending | 0 Skipped
PASS

Ginkgo ran 1 suite in 2.302845029s
Test Suite Passed

@blgm

blgm commented Jun 10, 2020

Copy link
Copy Markdown
Collaborator

Hi @jianzhangbjz. Sorry that it's taken so long to review this. I agree that having an examples is always helpful, and hopefully your work will help other people to learn Ginkgo.

@blgm blgm merged commit fc0e44e into onsi:master Jun 10, 2020

@blgm blgm left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good

@jianzhangbjz

Copy link
Copy Markdown
Contributor Author

@blgm sorry for the late reply, yes, I agreed, thank you very much!

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.

2 participants