[all-commits] [llvm/llvm-project] 7cdd53: [lldb][Test] Add C++ tests for DumpValueObjectOpti...

[David Spickett via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 7cdd53dacdfd9eef3d46ecddde872c71ec4292a1
      https://github.com/llvm/llvm-project/commit/7cdd53dacdfd9eef3d46ecddde872c71ec4292a1
  Author: David Spickett <david.spickett at linaro.org>
  Date:   2024-05-28 (Tue, 28 May 2024)

  Changed paths:
    M lldb/unittests/CMakeLists.txt
    A lldb/unittests/ValueObject/CMakeLists.txt
    A lldb/unittests/ValueObject/DumpValueObjectOptionsTests.cpp

  Log Message:
  -----------
  [lldb][Test] Add C++ tests for DumpValueObjectOptions and enums (#93158)

DumpValueObjectOptions can only be created and modified from C++. This
means it's currently only testable from Python by calling some command
that happens to use one, and even so, you can't pick which options get
chosen.

So we have decent coverage for the major options that way, but I want to
add more niche options that will be harder to test from Python (register
field options).

So this change adds some "unit tests", though it's stretching the
definition to the point it's more "test written in C++". So we can test
future options in isolation.

Since I want to add options specific to enums, that's all it covers.
There is a test class that sets up the type system so it will be easy to
test other types in future (e.g. structs, which register fields also
use).



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