-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
remove deprecated stuff in unittest module #17156
Conversation
ringabout
commented
Feb 23, 2021
•
edited
Loading
edited
- use runnableExamples
- remove one callsite
- add two regression tests
after removing callsite, CI fails in in case: https://github.com/LemonBoy/jstin with after reduction, here's a repro: import unittest
type MyFoo = object
var obj = MyFoo()
let check = 1
check(obj == obj)
and here's a different issue with a similar test case: when true:
import unittest
block:
let check = 123
var a = 1
var b = 1
check(a == b)
maybe those are fixable. Another aspect is whether we actually should deprecate regardless, if we can avoid notecan you add those 2 minimized examples to tunittest to avoid future regressions? |
* remove unnecessary when statement * remove outdated codes * remove deprecated stuff in testament * fix
* remove unnecessary when statement * remove outdated codes * remove deprecated stuff in testament * fix