akkartik/wardley.love - Change HMZSOZQCBEQBHYDEZ57U4LAITX4XHFMASJDJOB2WZHYAD53QEKCAC

tweak format of test failures

Put the name of the test first. File and line number are less important, and also our heuristics sometimes get them wrong.

Created by Kartik K. Agaram on January 22, 2023

HMZSOZQCBEQBHYDEZ57U4LAITX4XHFMASJDJOB2WZHYAD53QEKCAC

Dependencies

In channels

main

Change contents

Replacement in app.lua at line 75 [4.2]

B:BD[3.61] → [3.61:131]

    error('There were test failures:\n\n'..table.concat(Test_errors))

[3.61]

[3.131]

    error('There were test failures:\n\n'..table.concat(Test_errors, '\n'))

Replacement in app.lua at line 453 [4.2]

B:BD[5.60966] → [5.60966:61148]

  local file_and_line_number = stack_trace:gsub('stack traceback:\n', ''):gsub(': .*', '')
  local full_error = file_and_line_number..':'..test_name..' -- '..err_without_line_number

[5.60966]

[5.61148]

  local file_and_line_number = stack_trace:gsub('stack traceback:\n', ''):gsub(': .*', '') --[[strip newline and tab]] :sub(3)
  local full_error = test_name..' -- '..file_and_line_number..' '..err_without_line_number