Remove test package linting

Having _test packages leads to unnecessary symbol exports

lint

@@ -18,18 +18,13 @@
 set -e
 
 IGNORE_LINT_COMMENT=
-IGNORE_TEST_PACKAGES=
 IGNORE_SPELLINGS=
 while true; do
     case "$1" in
         -nocomment)
             IGNORE_LINT_COMMENT=1
             shift 1
             ;;
-        -notestpackage)
-            IGNORE_TEST_PACKAGES=1
-            shift 1
-            ;;
         -ignorespelling)
             IGNORE_SPELLINGS="$2,$IGNORE_SPELLINGS"
             shift 2
@@ -65,30 +60,6 @@ spell_check() {
     return $lint_result
 }
 
-test_mismatch() {
-    local filename="$1"
-    local package=$(grep '^package ' "$filename" | awk '{print $2}')
-    local lint_result=0
-
-    if [[ $package == "main" ]]; then
-        return # in package main, all bets are off
-    fi
-
-    if [[ $filename == *"_internal_test.go" ]]; then
-        if [[ $package == *"_test" ]]; then
-            lint_result=1
-            echo "${filename}: should not be part of a _test package"
-        fi
-    else
-        if [[ ! $package == *"_test" ]]; then
-            lint_result=1
-            echo "${filename}: should be part of a _test package"
-        fi
-    fi
-
-    return $lint_result
-}
-
 lint_go() {
     local filename="$1"
     local lint_result=0
@@ -183,12 +154,6 @@ lint() {
         tf) lint_tf "${filename}" || lint_result=1 ;;
     esac
 
-    if [ -z "$IGNORE_TEST_PACKAGES" ]; then
-        if [[ "$filename" == *"_test.go" ]]; then
-            test_mismatch "${filename}" || lint_result=1
-        fi
-    fi
-
     spell_check "${filename}" || lint_result=1
 
     return $lint_result