[all-commits] [llvm/llvm-project] efb0e9: [LLDB] Update breakpoint-command.test to use strin...

cmtice via All-commits all-commits at lists.llvm.org
Sun Oct 22 15:43:08 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: efb0e9c0bda62ba4b1c5421baeb140016ff53c84
      https://github.com/llvm/llvm-project/commit/efb0e9c0bda62ba4b1c5421baeb140016ff53c84
  Author: cmtice <cmtice at google.com>
  Date:   2023-10-22 (Sun, 22 Oct 2023)

  Changed paths:
    M lldb/test/Shell/Breakpoint/breakpoint-command.test

  Log Message:
  -----------
  [LLDB] Update breakpoint-command.test to use string instead of number. (#69796)

lldb/test/Shell/Breakpoint/breakpoint-command.test adds a python
command, to be executed when a breakpoint hits, that writes out a
number. It then runs, hits the breakpoint and checks that the number is
present exactly once.

The problem is that on some systems the test can be run in a filepath
that happens to contain the number (e.g. auto-generated directory
names). The number is then detected multiple times and the test fails.

This patch fixes the issue by using a string instead, particularly a
string with spaces, which is very unlikely to be auto-generated by any
system.




More information about the All-commits mailing list