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

Hook call mismatch warnings #42

Merged
merged 5 commits into from
Feb 25, 2017
Merged

Hook call mismatch warnings #42

merged 5 commits into from
Feb 25, 2017

Commits on Feb 18, 2017

  1. Future warn about hookspec vs. call mis-matches 

    Warn when either a hook call doesn't match a hookspec.

Additionally,
- Extend `varnames()` to return the list of both the arg and
  kwarg names for a function
- Rename `_MultiCall.kwargs` to `caller_kwargs` to be more explicit
- Store hookspec kwargs on `_HookRelay` and `HookImpl` instances

Relates to pytest-dev#15
    Tyler Goodlet committed Feb 18, 2017
    Configuration menu
    Copy the full SHA
    11955b7 View commit details
    Browse the repository at this point in the history

  2. Port tests to match new varnames() return value

    Tyler Goodlet committed Feb 18, 2017
    Configuration menu
    Copy the full SHA
    e27338e View commit details
    Browse the repository at this point in the history

  3. Handle py2.6 and better docs 

    - don't use "Positional" in warning message
- support python 2.6 sets
- don't include __multicall__ in args comparison
- document `varnames()` new pair return value
    Tyler Goodlet committed Feb 18, 2017
    Configuration menu
    Copy the full SHA
    96b6494 View commit details
    Browse the repository at this point in the history

  4. Add a call vs. spec mismatch warning test

    Tyler Goodlet committed Feb 18, 2017
    Configuration menu
    Copy the full SHA
    4ab6b1f View commit details
    Browse the repository at this point in the history

  5. Drop "Positional" adjective

    Tyler Goodlet committed Feb 18, 2017
    Configuration menu
    Copy the full SHA
    e5cfb31 View commit details
    Browse the repository at this point in the history