[all-commits] [llvm/llvm-project] 1f28da: [lldb][test] Combine libstdc++ and libc++ std::str...

[Michael Buch via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 1f28da60d5dfa51b1e623fdd168fbb3ea85f6ac9
      https://github.com/llvm/llvm-project/commit/1f28da60d5dfa51b1e623fdd168fbb3ea85f6ac9
  Author: Michael Buch <michaelbuch12 at gmail.com>
  Date:   2025-07-08 (Tue, 08 Jul 2025)

  Changed paths:
    A lldb/test/API/functionalities/data-formatter/data-formatter-stl/generic/string/Makefile
    A lldb/test/API/functionalities/data-formatter/data-formatter-stl/generic/string/TestDataFormatterStdString.py
    A lldb/test/API/functionalities/data-formatter/data-formatter-stl/generic/string/main.cpp
    R lldb/test/API/functionalities/data-formatter/data-formatter-stl/libcxx/string/Makefile
    R lldb/test/API/functionalities/data-formatter/data-formatter-stl/libcxx/string/TestDataFormatterLibcxxString.py
    R lldb/test/API/functionalities/data-formatter/data-formatter-stl/libcxx/string/main.cpp
    R lldb/test/API/functionalities/data-formatter/data-formatter-stl/libstdcpp/string/Makefile
    R lldb/test/API/functionalities/data-formatter/data-formatter-stl/libstdcpp/string/TestDataFormatterStdString.py
    R lldb/test/API/functionalities/data-formatter/data-formatter-stl/libstdcpp/string/main.cpp

  Log Message:
  -----------
  [lldb][test] Combine libstdc++ and libc++ std::string tests into generic test (#147355)

This combines the libc++ and libstdc++ test cases. The main difference
was that the libstdcpp tests had additional tests for
references/pointers to std::string. I moved those over.

The libstdc++ formatters don't support `std::u16string`/`std::u32string`
yet, so I extracted those into XFAILed test cases.

There were also two test assertions that failed for libstdc++:
1. libstdc++ doesn't obey the capped/uncapped summary options
2. When a summary isn't available for a std::string, libc++ would print
"Summary Unavailable", whereas libstdc++ just prints "((null))". This
may be better suited for the STL-specific subdirectories, but left it
here for now.

I put those in separate XFAILed test-cases.

Split out from https://github.com/llvm/llvm-project/pull/146740



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications