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

convert tests to Test::More #35

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

convert tests to Test::More #35

wants to merge 5 commits into from

Commits on May 16, 2022

  1. MANIFEST is not executable

    @haarg
    haarg committed May 16, 2022
    Configuration menu
    Copy the full SHA
    5781e58 View commit details
    Browse the repository at this point in the history

Commits on May 27, 2022

  1. add Test::More as a test prereq 

    Test::More is already used by GetOSName.t, so it should be listed as a
prereq.
    @haarg
    haarg committed May 27, 2022
    Configuration menu
    Copy the full SHA
    74c6aed View commit details
    Browse the repository at this point in the history

  2. convert most tests to Test::More

    @haarg
    haarg committed May 27, 2022
    Configuration menu
    Copy the full SHA
    e25cdb0 View commit details
    Browse the repository at this point in the history

  3. convert GetCurrentThreadId test to use Test::More 

    The GetCurrentThreadId test forks, then continues testing in the child
rather than the parent. This conflicts with the standard behavior of
Test::More, which does extra checks at process end but only for the
parent process. Use Test::Builder's ->no_ending setting to prevent that
extra handling. We have a test plan, so everything should work without
that extra handling.
    @haarg
    haarg committed May 27, 2022
    Configuration menu
    Copy the full SHA
    6bee1c9 View commit details
    Browse the repository at this point in the history

  4. drop Test.pm prereq

    @haarg
    haarg committed May 27, 2022
    Configuration menu
    Copy the full SHA
    abc9ec8 View commit details
    Browse the repository at this point in the history