revert 3a8478131a
Some checks failed
akbasic CI Build / sanitizers (push) Failing after 6m58s
akbasic CI Build / cmake_build (push) Failing after 19m38s
akbasic CI Build / akgl_build (push) Failing after 12m29s
akbasic CI Build / mutation_test (push) Failing after 4m30s
akbasic CI Build / coverage (push) Failing after 20m51s
Some checks failed
akbasic CI Build / sanitizers (push) Failing after 6m58s
akbasic CI Build / cmake_build (push) Failing after 19m38s
akbasic CI Build / akgl_build (push) Failing after 12m29s
akbasic CI Build / mutation_test (push) Failing after 4m30s
akbasic CI Build / coverage (push) Failing after 20m51s
revert Register line-limit fixtures as expected failures This issue is not something this branch should solve
This commit is contained in:
@@ -609,11 +609,6 @@ file(GLOB_RECURSE AKBASIC_GOLDEN_CASES
|
|||||||
"${CMAKE_CURRENT_SOURCE_DIR}/tests/reference/*.bas"
|
"${CMAKE_CURRENT_SOURCE_DIR}/tests/reference/*.bas"
|
||||||
)
|
)
|
||||||
|
|
||||||
set(AKBASIC_LINE_LIMIT_GOLDEN_CASES
|
|
||||||
language/flowcontrol/nestedforloopwaitingforcommand.bas
|
|
||||||
language/functions/mod.bas
|
|
||||||
)
|
|
||||||
|
|
||||||
foreach(_case IN LISTS AKBASIC_GOLDEN_CASES)
|
foreach(_case IN LISTS AKBASIC_GOLDEN_CASES)
|
||||||
string(REGEX REPLACE "^tests/" "" _name "${_case}")
|
string(REGEX REPLACE "^tests/" "" _name "${_case}")
|
||||||
string(REGEX REPLACE "\\.bas$" "" _name "${_name}")
|
string(REGEX REPLACE "\\.bas$" "" _name "${_name}")
|
||||||
@@ -625,12 +620,6 @@ foreach(_case IN LISTS AKBASIC_GOLDEN_CASES)
|
|||||||
-DCASE=${CMAKE_CURRENT_SOURCE_DIR}/tests/reference/${_case}
|
-DCASE=${CMAKE_CURRENT_SOURCE_DIR}/tests/reference/${_case}
|
||||||
-P ${CMAKE_CURRENT_SOURCE_DIR}/tests/golden.cmake
|
-P ${CMAKE_CURRENT_SOURCE_DIR}/tests/golden.cmake
|
||||||
)
|
)
|
||||||
# The reference corpus is immutable, and these two source lines are longer
|
|
||||||
# than the 80-column input contract. Keep the cases visible in CTest while
|
|
||||||
# issue #32 remains open rather than editing the upstream-derived fixtures.
|
|
||||||
if(_case IN_LIST AKBASIC_LINE_LIMIT_GOLDEN_CASES)
|
|
||||||
_set_tests_properties(golden_${_name} PROPERTIES WILL_FAIL TRUE)
|
|
||||||
endif()
|
|
||||||
endforeach()
|
endforeach()
|
||||||
|
|
||||||
if(AKBASIC_GOLDEN_CASES)
|
if(AKBASIC_GOLDEN_CASES)
|
||||||
@@ -669,23 +658,6 @@ file(GLOB_RECURSE AKBASIC_LOCAL_CASES
|
|||||||
"${CMAKE_CURRENT_SOURCE_DIR}/tests/language/*.bas"
|
"${CMAKE_CURRENT_SOURCE_DIR}/tests/language/*.bas"
|
||||||
)
|
)
|
||||||
|
|
||||||
set(AKBASIC_LINE_LIMIT_LOCAL_CASES
|
|
||||||
language/arrays_in_parameter_lists.bas
|
|
||||||
language/audio/filter_refused.bas
|
|
||||||
language/audio/no_device.bas
|
|
||||||
language/functions/recursion.bas
|
|
||||||
language/graphics/argument_errors.bas
|
|
||||||
language/graphics/no_device.bas
|
|
||||||
language/housekeeping/verbs.bas
|
|
||||||
language/numeric/octal_literal.bas
|
|
||||||
language/numeric/truth_value_arithmetic.bas
|
|
||||||
language/statements/multiple_per_line.bas
|
|
||||||
language/structures/missing_field.bas
|
|
||||||
language/structures/parameters.bas
|
|
||||||
language/structures/records.bas
|
|
||||||
language/structures/reserved_names.bas
|
|
||||||
)
|
|
||||||
|
|
||||||
#
|
#
|
||||||
# One family of local cases is driver-specific and has to be. A case named
|
# One family of local cases is driver-specific and has to be. A case named
|
||||||
# no_device.bas asserts that a verb needing a device refuses politely when it was
|
# no_device.bas asserts that a verb needing a device refuses politely when it was
|
||||||
@@ -712,12 +684,6 @@ foreach(_case IN LISTS AKBASIC_LOCAL_CASES)
|
|||||||
-P ${CMAKE_CURRENT_SOURCE_DIR}/tests/golden.cmake
|
-P ${CMAKE_CURRENT_SOURCE_DIR}/tests/golden.cmake
|
||||||
)
|
)
|
||||||
list(APPEND AKBASIC_LOCAL_NAMES local_${_name})
|
list(APPEND AKBASIC_LOCAL_NAMES local_${_name})
|
||||||
# These fixtures exercise behavior unrelated to the line-length contract,
|
|
||||||
# but their existing source lines exceed the current 80-column limit. Keep
|
|
||||||
# them registered and visible until issue #32 is resolved.
|
|
||||||
if(_case IN_LIST AKBASIC_LINE_LIMIT_LOCAL_CASES)
|
|
||||||
_set_tests_properties(local_${_name} PROPERTIES WILL_FAIL TRUE)
|
|
||||||
endif()
|
|
||||||
endforeach()
|
endforeach()
|
||||||
|
|
||||||
if(AKBASIC_LOCAL_NAMES)
|
if(AKBASIC_LOCAL_NAMES)
|
||||||
|
|||||||
Reference in New Issue
Block a user