From 5e479198dcf35fb918920ed47923492cfc51d9b4 Mon Sep 17 00:00:00 2001
From: Tim Jacomb <timjacomb1+github@gmail.com>
Date: Wed, 16 Feb 2022 20:44:10 +0000
Subject: [PATCH] Remove unneeded injected test

---
 .../jvnet/hudson/test/JellyTestSuiteBuilder.java    | 13 -------------
 1 file changed, 13 deletions(-)

diff --git a/src/main/java/org/jvnet/hudson/test/JellyTestSuiteBuilder.java b/src/main/java/org/jvnet/hudson/test/JellyTestSuiteBuilder.java
index 4e3ef493a..4cd9b351a 100644
--- a/src/main/java/org/jvnet/hudson/test/JellyTestSuiteBuilder.java
+++ b/src/main/java/org/jvnet/hudson/test/JellyTestSuiteBuilder.java
@@ -101,7 +101,6 @@ private static class JellyCheck extends TestCase {
         protected void runTest() throws Exception {
             jct.createContext().compileScript(jelly);
             Document dom = new SAXReader().read(jelly);
-            checkLabelFor(dom);
             if (requirePI) {
                 ProcessingInstruction pi = dom.processingInstruction("jelly");
                 if (pi==null || !pi.getText().contains("escape-by-default"))
@@ -111,18 +110,6 @@ protected void runTest() throws Exception {
             // TODO: what else can we check statically? use of taglibs?
         }
 
-        /**
-         * Makes sure that &lt;label for=...> is not used inside config.jelly nor global.jelly
-         */
-        private void checkLabelFor(Document dom) {
-            if (isConfigJelly() || isGlobalJelly()) {
-                if (!dom.selectNodes("//label[@for]").isEmpty())
-                    throw new AssertionError("<label for=...> shouldn't be used because it doesn't work " +
-                            "when the configuration item is repeated. Use <label class=\"attach-previous\"> " +
-                            "to have your label attach to the previous DOM node instead.\nurl="+jelly);
-            }
-        }
-
         private boolean isConfigJelly() {
             return jelly.toString().endsWith("/config.jelly");
         }