diff --git a/src/gn/target.cc b/src/gn/target.cc
index c6cef20..a11a014 100644
--- a/src/gn/target.cc
+++ b/src/gn/target.cc
@@ -1222,14 +1222,6 @@
     }
   }
 
-  // Verify no validations have "testonly" set.
-  for (const auto& pair : validations_) {
-    if (pair.ptr->testonly()) {
-      *err = MakeTestOnlyError(this, pair.ptr);
-      return false;
-    }
-  }
-
   // Verify no configs have "testonly" set.
   for (ConfigValuesIterator iter(this); !iter.done(); iter.Next()) {
     if (const Config* config = iter.GetCurrentConfig()) {
diff --git a/src/gn/target_unittest.cc b/src/gn/target_unittest.cc
index 81c93ef..7ed4055 100644
--- a/src/gn/target_unittest.cc
+++ b/src/gn/target_unittest.cc
@@ -372,6 +372,12 @@
   product.set_testonly(false);
   product.private_deps().push_back(LabelTargetPair(&testlib));
   ASSERT_FALSE(product.OnResolved(&err));
+
+  // "package" is a non-test with a testonly validation, this is OK.
+  TestTarget package(setup, "//app:package", Target::EXECUTABLE);
+  package.set_testonly(false);
+  package.validations().push_back(LabelTargetPair(&test));
+  ASSERT_TRUE(package.OnResolved(&err));
 }
 
 // Configs can be testonly too.