[all-commits] [llvm/llvm-project] c890b7: [lldb][Telemetry] Fix unit test compile failure wi...

Raul Tambre via All-commits all-commits at lists.llvm.org
Fri Apr 18 04:48:51 PDT 2025


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: c890b7376fc916c1818dc3c211a080601973aa0b
      https://github.com/llvm/llvm-project/commit/c890b7376fc916c1818dc3c211a080601973aa0b
  Author: Raul Tambre <raul at tambre.ee>
  Date:   2025-04-18 (Fri, 18 Apr 2025)

  Changed paths:
    M lldb/unittests/Core/TelemetryTest.cpp

  Log Message:
  -----------
  [lldb][Telemetry] Fix unit test compile failure with LLVM_ENABLE_TELEMETRY=0 (#136115)

It needs to be `TEST_F` to access `received_entries`.
Disabling also works based on the test not the fixture name.

Build failure:
```
lldb/unittests/Core/TelemetryTest.cpp:110:17: error: use of undeclared identifier 'received_entries'
  110 |   ASSERT_EQ(1U, received_entries.size());
      |                 ^
lldb/unittests/Core/TelemetryTest.cpp:112:61: error: use of undeclared identifier 'received_entries'
  112 |             llvm::dyn_cast<lldb_private::FakeTelemetryInfo>(received_entries[0])
      |                                                             ^
```

Fixes: 159b872b37363511a359c800bcc9230bb09f2457



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


More information about the All-commits mailing list