danilashtefan added inline comments.
================
Comment at:
lldb/test/API/functionalities/data-formatter/data-formatter-stl/generic/set/TestDataFormatterGenericSet.py:45
"[3] = 3",
"[6] = 6"])
----------------
wallace wrote:
> can you add additional assertions like the one you did in
> https://reviews.llvm.org/D112180 that uses ValueCheck?
I have made check method more generic and less verbose. Before it was
specifically checking the "ii" variable, that was a set<int> of size 7. With
value check approach we can now test set<string> and others.
I guess, that I can delete all of the
```
self.expect()
```
and simply call check method with the expected size. Please, let me know if I
should do it or leave it as it is. Thanks
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D112537/new/
https://reviews.llvm.org/D112537
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
