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

fix oauth tests #406

Closed
wants to merge 1 commit into from
Closed

fix oauth tests #406

wants to merge 1 commit into from

Conversation

colin-axner
Copy link
Contributor

@colin-axner colin-axner commented Oct 26, 2020

Fixes the first two tests in #401 .

I noticed the test was failing [here] so I updated the app config to be set to Open Registration.

Then the test began panicing because the templates weren't loaded so I added InitTemplates to load them. The test case is already labeled success so I assume the written code should be OK.

go test
ERROR: 2020/10/26 13:34:29 oauth.go:148: viewOauthInit error: pretend unable to write state error
2020/10/26 13:34:29 Loading templates...
2020/10/26 13:34:29 Loading pages...
2020/10/26 13:34:29 Loading user pages...
2020/10/26 13:34:29 Loading testdata/config.ini configuration...
--- FAIL: TestUpdatesRoundTrip (1.09s)
    --- FAIL: TestUpdatesRoundTrip/Release_URL (0.00s)
        updates_test.go:36: Malformed Release URL: https://blog.writefreely.org/version-
FAIL
exit status 1
FAIL	github.com/writeas/writefreely	1.169s

updated output. The first 5 lines are just logs from running the tests. Is there a way to disable the logs from being outputted in the console?


  • I have signed the CLA

Fix oauth test by updating app configuration to open registration and loading templates.
@cjeller1592
Copy link
Contributor

I got the same output from running tests on this branch — thanks @colin-axner! Wonder if @thebaer has any input on how to disable the logs.

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants