Skip to content

Account test#112

Open
ademuanthony wants to merge 19 commits intoplanetdecred:masterfrom
ademuanthony:account-test
Open

Account test#112
ademuanthony wants to merge 19 commits intoplanetdecred:masterfrom
ademuanthony:account-test

Conversation

@ademuanthony
Copy link
Contributor

@ademuanthony ademuanthony commented Feb 28, 2020

This PR add tests for account operations in the accounts.go file.

The PR use github.com/onsi/ginkgo and github.com/onsi/gomega as introduced by @RogueElement in his on going PR #96.

image

@ademuanthony ademuanthony changed the title Account test [WIP] Account test Feb 28, 2020
@ademuanthony ademuanthony changed the title [WIP] Account test Account test Feb 28, 2020
Copy link
Contributor

@oluwandabira oluwandabira left a comment

Choose a reason for hiding this comment

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

The Describe's don't need to have the full method signature, we can't overload methods in go so just the name would suffice.

Copy link
Contributor

@oluwandabira oluwandabira left a comment

Choose a reason for hiding this comment

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

git ignore the test rootDir

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.

3 participants