Skip to content

Quiet some noisy loggers for lettuce acceptance tests - #3289

Merged
jzoldak merged 1 commit into
masterfrom
zoldak/quiet-acceptance-test-logs
Apr 10, 2014
Merged

Quiet some noisy loggers for lettuce acceptance tests#3289
jzoldak merged 1 commit into
masterfrom
zoldak/quiet-acceptance-test-logs

Conversation

@jzoldak

@jzoldak jzoldak commented Apr 9, 2014

Copy link
Copy Markdown
Contributor

This will suppress some of the log messages that are clogging up the logs and confusing people who are troubleshooting actual errors:

  • Cannot determine primary key of logged in user
  • Cannot find corresponding link for name: TOS
  • Cannot find corresponding link for name: PRIVACY

@auraz

auraz commented Apr 10, 2014

Copy link
Copy Markdown
Contributor

👍
Do you think it will be good idea to have common_acceptance.py to avoid duplication in cmd and lms acceptance.py files?

@jzoldak

jzoldak commented Apr 10, 2014

Copy link
Copy Markdown
Contributor Author

Yeah @auraz it does look like the files could stand some refactoring. I'll merge this for now and put that thought on the backlog.

jzoldak pushed a commit that referenced this pull request Apr 10, 2014
Quiet some noisy loggers for lettuce acceptance tests
@jzoldak
jzoldak merged commit 2c500a2 into master Apr 10, 2014
@jzoldak
jzoldak deleted the zoldak/quiet-acceptance-test-logs branch April 10, 2014 13:29
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