rupprecht wrote:

This cleanup isn't strictly required to proceed w/ unittest2->unittest; we 
already have a wrapper in decorators.py that we could update, from:

```py
def expectedFailure(func):
    return unittest2.expectedFailure(func)
```

To:
```py
def expectedFailure(func):
    if callable(func):
        return unittest.expectedFailure(func)
    return unittest.expectedFailure
```

IMHO, better to just start dropping the reason field to be more uniform; one 
should not have to decipher whether `@expectedFailure` is referring to the 
version in `unittest` or `decorators`.

https://github.com/llvm/llvm-project/pull/73028
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to