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

test_urllib2 convert doctests to unittest #61848

Closed
orsenthil opened this issue Apr 6, 2013 · 5 comments
Closed

test_urllib2 convert doctests to unittest #61848

orsenthil opened this issue Apr 6, 2013 · 5 comments
Assignees
Labels
type-bug An unexpected behavior, bug, or error

Comments

@orsenthil
Copy link
Member

BPO 17648
Nosy @orsenthil, @ezio-melotti, @merwok, @bitdancer
Files
  • remove_doctests.patch
  • Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.

    Show more details

    GitHub fields:

    assignee = 'https://github.com/orsenthil'
    closed_at = <Date 2013-04-09.13:03:30.862>
    created_at = <Date 2013-04-06.23:45:14.821>
    labels = ['type-bug']
    title = 'test_urllib2 convert doctests to unittest'
    updated_at = <Date 2013-04-09.13:03:45.004>
    user = 'https://github.com/orsenthil'

    bugs.python.org fields:

    activity = <Date 2013-04-09.13:03:45.004>
    actor = 'orsenthil'
    assignee = 'orsenthil'
    closed = True
    closed_date = <Date 2013-04-09.13:03:30.862>
    closer = 'orsenthil'
    components = []
    creation = <Date 2013-04-06.23:45:14.821>
    creator = 'orsenthil'
    dependencies = []
    files = ['29700']
    hgrepos = []
    issue_num = 17648
    keywords = ['patch']
    message_count = 5.0
    messages = ['186171', '186375', '186376', '186395', '186396']
    nosy_count = 5.0
    nosy_names = ['orsenthil', 'ezio.melotti', 'eric.araujo', 'r.david.murray', 'python-dev']
    pr_nums = []
    priority = 'normal'
    resolution = 'fixed'
    stage = 'resolved'
    status = 'closed'
    superseder = None
    type = 'behavior'
    url = 'https://bugs.python.org/issue17648'
    versions = ['Python 3.3', 'Python 3.4']

    @orsenthil
    Copy link
    Member Author

    Sometime back during an IRC conversation we realized that converting test_urllib2 doctests to proper unittest may help in various ways.
    a) Improve coverage report (?) Know what is covered and not.
    b) Helps expand it further when new features are added.

    Here is patch which removes the old doctests and converts them to unittest.

    Reviews will be helpful.

    @orsenthil orsenthil self-assigned this Apr 6, 2013
    @python-dev
    Copy link
    Mannequin

    python-dev mannequin commented Apr 9, 2013

    New changeset eb632aafff57 by Senthil Kumaran in branch 'default':
    bpo-17648 - Clean up test_urllib2.py. Converted doctests to unittest for
    http://hg.python.org/cpython/rev/eb632aafff57

    @orsenthil
    Copy link
    Member Author

    I have pushed the changes since they are restricted to test files. If there are review comments, I shall do make the changes.

    I shall consider to to make 3.3 tests close to default branch ones so that fixes can be backported. (Leaving the bug open for that).

    @orsenthil orsenthil added the type-bug An unexpected behavior, bug, or error label Apr 9, 2013
    @python-dev
    Copy link
    Mannequin

    python-dev mannequin commented Apr 9, 2013

    New changeset 5c9b36969abc by Senthil Kumaran in branch '3.3':
    bpo-17648 - convert test_urllib2.py doctests to unittests
    http://hg.python.org/cpython/rev/5c9b36969abc

    @orsenthil
    Copy link
    Member Author

    Changed in active branches (3.3 and default).

    @ezio-melotti ezio-melotti transferred this issue from another repository Apr 10, 2022
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Labels
    type-bug An unexpected behavior, bug, or error
    Projects
    None yet
    Development

    No branches or pull requests

    1 participant