akkartik/sokoban.love - Change TSV3KPBRNEPLBTIHZFUCRV53VLQXNVOPCZ5L2RB7FH4RUYQJ6PZQC

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

TSV3KPBRNEPLBTIHZFUCRV53VLQXNVOPCZ5L2RB7FH4RUYQJ6PZQC

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