Skip to content

Conversation

megemini
Copy link
Contributor

@megemini megemini commented Jul 5, 2024

PR Category

User Experience

PR Types

Improvements

Description

  • 引入 _typing 模块的单元测试
  • 修改 _typing.basic.py 以用于测试

注:

  • 此测试模块,不单是测试 _typing 模块,是整个 type hint 单独的测试用例

  • 此处只添加了部分测试用例,可后续完善
    后续可以在

    • test/_typing 中添加主测试用例,如 test_<module>.py
    • test/_typing/<module>/pass 中添加通过的测试用例
    • test/_typing/<module>/fail 中添加失败的用例(用 # type: ignore[xxx] 可跳过的视为失败用例)
    • test/_typing/<module>reveal 中添加推导的用例

    这里区分 module 的主要原因是,如果都放在一个测试文件中,timeout 可能会很大 ~

关联 PR #63597

p.s. 首次添加测试模块,可能需要先调试一下 ~

@SigureMo

Copy link

paddle-bot bot commented Jul 5, 2024

你的PR提交成功,感谢你对开源项目的贡献!
请关注后续CI自动化测试结果,详情请参考Paddle-CI手册
Your PR has been submitted. Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

@SigureMo
Copy link
Member

SigureMo commented Jul 5, 2024

我记得原定计划是不引入单元测试的?

@megemini
Copy link
Contributor Author

megemini commented Jul 5, 2024

我记得原定计划是不引入单元测试的?

嗯 ~ 之前计划只引入 _typing 模块的测试 ~ 当时一开始加进去了,后来又删掉了 (#63604 当时是把单测删掉的) ~

这里先把层级结构搭建好,如果有需要的话往里面加就行 ~

主要是,没有测试总不放心 🫠🫠🫠

有啥其他顾虑?

@SigureMo
Copy link
Member

SigureMo commented Jul 5, 2024

#63604 说的也是不再额外添加单测,RFC 也是,为什么突然又加了呢?

加的话按照什么原则加呢?哪些需要加哪些不需要加?

维护成本又要怎样考虑呢?将来谁来维护呢?

@megemini megemini closed this Jul 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants