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

add first unit tests for gno.land website #567

Merged
merged 6 commits into from
Mar 8, 2023

Conversation

grepsuzette
Copy link
Contributor

@grepsuzette grepsuzette commented Mar 7, 2023

This adds unit tests for the gno.land website, and also for PR #565 (website working with deep packages)

For now I don't have gnot on the testnet.
Someone must run an addpkg of examples/gno.land/r/demo/deep/very/deep/render.gno
before the r/demo/deep/very/deep/* tests pass. Next time the testnet is reset it should be added automatically.

Then can test with go test ./gnoland/website,
and all tests should pass.

@grepsuzette grepsuzette requested a review from a team as a code owner March 7, 2023 05:21
@grepsuzette
Copy link
Contributor Author

Will repost (accidentally included commits from yesterday), sorry

@grepsuzette grepsuzette closed this Mar 7, 2023
@grepsuzette
Copy link
Contributor Author

Nah should be good, I am not sure what to do in this situation (previous commits that went inside this PR). Think they should get merged automatically.

@grepsuzette grepsuzette reopened this Mar 7, 2023
@grepsuzette grepsuzette changed the title add test for PR #565 (website working with deep packages) add first unit tests for gno.land website Mar 7, 2023
gnoland/website/main_test.go Outdated Show resolved Hide resolved
Copy link
Member

@moul moul left a comment

Choose a reason for hiding this comment

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

I like it. Simple and efficient. Thanks.

@moul moul merged commit fbdbfa8 into gnolang:master Mar 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

2 participants