Pylint提示R1703含义解析及pytest测试脚本问题咨询

一、修复Pylint的R1703提示 #

R1703提示的核心是：你的if-else判断完全冗余，因为os.path.exists(file)本身就返回布尔值（True/False），直接返回该结果即可，无需额外的条件分支。

你修改后的代码反而画蛇添足——将布尔值与1比较完全没必要，Python中布尔值是int的子类，但直接返回原结果才是最简洁正确的写法。

正确的函数代码：

import os

def is_file_valid(file):
    """
    检查输入文件是否存在
    :param file: 文件路径
    :return: True/False
    """
    return os.path.exists(file)

二、修复pytest测试脚本报错 #

你的测试脚本存在三个关键问题：

• 测试函数的参数file_test会被pytest识别为fixture，但你未定义该fixture，导致报错。
• 脚本未调用目标函数is_file_valid，等于没测试你写的校验逻辑。
• 目标文件本身不存在，调用os.remove(file_test)会抛出文件不存在的异常。

修正后的测试脚本：

import os
from 你的模块名 import is_file_valid  # 替换为实际的函数所在模块名

def test_is_file_valid():
    file_test = 'test.txt'
    
    # 测试文件不存在的场景
    assert is_file_valid(file_test) is False, "文件不存在时应返回False"
    
    # 测试文件存在的场景
    with open(file_test, 'w') as f:
        f.write('测试内容')
    try:
        assert is_file_valid(file_test) is True, "文件存在时应返回True"
    finally:
        os.remove(file_test)  # 无论测试成败都清理临时文件

内容的提问来源于stack exchange，提问作者sleepbug