don't report symbol parse failures in lint test

cmd/scip/lint.go

@@ -240,14 +240,19 @@ func (st *symbolTable) addOccurrence(path string, occ *scip.Occurrence) error {
 	return nil
 }
 
+// SkipLintSymbolParseTest is only set to true in the tests to allow using more succinct symbols
+var SkipLintSymbolParseTest = false
+
 func lintSymbolString(symbol string, context string) error {
 	if symbol == "" {
 		return emptyStringError{what: "symbol", context: context}
 	}
 	sym, err := scip.ParseSymbol(symbol)
 	if err != nil {
-		// TODO: This should be linted (but it makes all the tests fail)
-		return nil
+		if SkipLintSymbolParseTest {
+			return nil
+		}
+		return err
 	}
 	formatted := scip.VerboseSymbolFormatter.FormatSymbol(sym)
 	if symbol != formatted {

cmd/scip/lint_test.go

@@ -96,6 +96,7 @@ type testCase struct {
 }
 
 func TestErrors(t *testing.T) {
+	SkipLintSymbolParseTest = true
 	var testCases []testCase
 
 	testCases = []testCase{