Project

General

Profile

Support #695

Remove cruft from test files

Added by John Abbott almost 9 years ago. Updated almost 9 years ago.

Status:
New
Priority:
Normal
Assignee:
-
Category:
Tidying
Target version:
Start date:
08 May 2015
Due date:
% Done:

0%

Estimated time:
Spent time:

Description

Several test files contain commented out code; this should either be removed or made active (or perhaps add a clear comment as to why the code is commented out).


Related issues

Related to CoCoALib - Support #694: In test files: when is assert.H needed?Closed2015-05-08

Related to CoCoALib - Support #696: test-OrderedRing: activate or eliminate?Closed2015-05-08

History

#1 Updated by John Abbott almost 9 years ago

Commented out code could make sense in the following cases:
  • the test is valid, but the relevant CoCoALib code is NYI
  • the test is valid but too slow for general use (maybe a simpler test can be found to replace it?)

We should try to make tests fairly easily comprehensible (to "experts").

Also available in: Atom PDF