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.

Sign up for GitHub

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

Jump to bottom

Test /events and /labels, especially auth #223

Merged
merged 4 commits into from
May 5, 2018
Merged

Test /events and /labels, especially auth #223

merged 4 commits into from
May 5, 2018

Commits on May 4, 2018

  1. Add more event tests 

    Starts #103.

Creates stubs for #75 (marked in `test_events.py` by a TODO).

Issues #218, #217, #216 and ,#215 should add to these tests.
    @osteele
    osteele committed May 4, 2018
    Configuration menu
    Copy the full SHA
    b7f354e View commit details
    Browse the repository at this point in the history

  2. Minor improvements to test_events.py 

    * Rename the test case.
* it's no longer necessary to set app.debug
    @osteele
    osteele committed May 4, 2018
    Configuration menu
    Copy the full SHA
    937445b View commit details
    Browse the repository at this point in the history

  3. Add test cases for /labels 

    Also in label_resources.py:
* Change a logger.warning to a logging.debug, since it's not a server issue if this happens (and was chattering at the test suite)
* Letter logger.{debug,info} do the string formatting instead of passing it formatted strings — for style and efficiency
    @osteele
    osteele committed May 4, 2018
    Configuration menu
    Copy the full SHA
    2adaea2 View commit details
    Browse the repository at this point in the history

Commits on May 5, 2018

  1. Remove unused import

    songbird175 committed May 5, 2018
    Configuration menu
    Copy the full SHA
    5eebfda View commit details
    Browse the repository at this point in the history