9799e48991dac848a8e3081e493199630e7e3112,allennlp/commands/test_install.py,,_run_test,#Any#,63

Before Change

def _run_test(args: argparse.Namespace):
    initial_working_dir = os.getcwd()
    module_parent = _get_module_root().parent
    logger.info("Changing directory to %s", module_parent)
    os.chdir(module_parent)
    test_dir = os.path.join(module_parent, "allennlp")
    logger.info("Running tests at %s", test_dir)

    if args.k:
        pytest_k = ["-k", args.k]
        pytest_m = ["-m", "not java"]
        if args.run_all:
            logger.warning("the argument "-k" overwrites "--run-all".")
    elif args.run_all:
        pytest_k = []
        pytest_m = []
    else:
        pytest_k = ["-k", "not sniff_test"]
        pytest_m = ["-m", "not java"]

    exit_code = pytest.main([test_dir, "--color=no"] + pytest_k + pytest_m)

    // Change back to original working directory after running tests
    os.chdir(initial_working_dir)
    sys.exit(exit_code)

After Change

def _run_test(args: argparse.Namespace):
    module_parent = _get_module_root().parent
    logger.info("Changing directory to %s", module_parent)
    with pushd(module_parent):
        test_dir = os.path.join(module_parent, "allennlp")
        logger.info("Running tests at %s", test_dir)

        if args.k:
            pytest_k = ["-k", args.k]
            pytest_m = ["-m", "not java"]
            if args.run_all:
                logger.warning("the argument "-k" overwrites "--run-all".")
        elif args.run_all:
            pytest_k = []
            pytest_m = []
        else:
            pytest_k = ["-k", "not sniff_test"]
            pytest_m = ["-m", "not java"]

        exit_code = pytest.main([test_dir, "--color=no"] + pytest_k + pytest_m)
        sys.exit(exit_code)

Instances