Update the labels

Author: jgreffe

src/main/java/org/jenkinsci/plugins/workflow/log/FileLogStorageFactory.java

@@ -31,7 +31,7 @@ public static final class DescriptorImpl extends LogStorageFactoryDescriptor<Fil
         @NonNull
         @Override
         public String getDisplayName() {
-            return "File Log Storage Factory";
+            return "Standard file logger";
         }
     }
 }

src/main/java/org/jenkinsci/plugins/workflow/log/tee/TeeLogStorageFactory.java

@@ -27,13 +27,13 @@ public class TeeLogStorageFactory implements LogStorageFactory {
     @DataBoundConstructor
     public TeeLogStorageFactory(LogStorageFactory primary, LogStorageFactory secondary) {
         if (primary == null) {
-            throw new IllegalArgumentException("Primary LogStorageFactory cannot be null");
+            throw new IllegalArgumentException("Primary Pipeline logger cannot be null");
         }
         if (secondary == null) {
-            throw new IllegalArgumentException("Secondary LogStorageFactory cannot be null");
+            throw new IllegalArgumentException("Secondary Pipeline logger cannot be null");
         }
         if (primary.getClass() == secondary.getClass()) {
-            throw new IllegalArgumentException("Primary and secondary LogStorageFactory must be distinct, but both were " + primary.getClass());
+            throw new IllegalArgumentException("Primary and secondary Pipeline loggers must be distinct, but both were " + primary.getClass());
         }
         this.primary = primary;
         this.secondary = secondary;
@@ -54,13 +54,13 @@ public LogStorage forBuild(@NonNull FlowExecutionOwner b) {
         var primaryLogStorage = this.primary.forBuild(b);
         if (primaryLogStorage == null) {
             return new BrokenLogStorage(new IllegalArgumentException(String.format(
-                    "The primary TeeLogStorageFactory of type %s returned null",
+                    "The primary Pipeline logger of type %s returned null",
                     primary.getClass().getName())));
         }
         var secondaryLogStorage = this.secondary.forBuild(b);
         if (secondaryLogStorage == null) {
             return new BrokenLogStorage(new IllegalArgumentException(String.format(
-                    "The secondary TeeLogStorageFactory of type %s returned null",
+                    "The secondary Pipeline logger of type %s returned null",
                     primary.getClass().getName())));
         }
         return new TeeLogStorage(primaryLogStorage, secondaryLogStorage);
@@ -72,7 +72,7 @@ public static final class DescriptorImpl extends LogStorageFactoryDescriptor<Tee
         @NonNull
         @Override
         public String getDisplayName() {
-            return "Tee Log Storage Factory";
+            return "Multiple loggers";
         }
 
         @Override

src/main/resources/org/jenkinsci/plugins/workflow/log/configuration/PipelineLoggingGlobalConfiguration/config.jelly

@@ -1,11 +1,11 @@
 <?jelly escape-by-default='true'?>
 <j:jelly xmlns:j="jelly:core" xmlns:f="/lib/form">
-  <f:section title="${%Pipeline Logging}">
+  <f:section title="${%Pipeline logging}">
     <f:block>
       ${%description(descriptor.getDefaultFactoryDescriptor().getDisplayName(), descriptor.getDefaultFactoryPlugin())}
     </f:block>
     <f:dropdownDescriptorSelector
-        field="factory" title="${%Logging Factory}"
+        field="factory" title="${%Pipeline logger}"
         descriptors="${descriptor.getLogStorageFactoryDescriptors()}"
     />
   </f:section>

src/main/resources/org/jenkinsci/plugins/workflow/log/configuration/PipelineLoggingGlobalConfiguration/config.properties

@@ -1 +1 @@
-description=Define the logging factory to use for pipeline logging. Without any explicit configuration, the default logging factory will be "{0}" (from {1}).
+description=Specify the Pipeline logger. The default logger is "{0}" (from {1}).

src/main/resources/org/jenkinsci/plugins/workflow/log/tee/TeeLogStorageFactory/config.jelly

@@ -1,6 +1,6 @@
 <?jelly escape-by-default='true'?>
 <j:jelly xmlns:j="jelly:core" xmlns:f="/lib/form">
-  <f:section title="${%Tee Log Storage Factory}">
+  <f:section title="${%Multiple loggers}">
     <f:block>
       ${%description}
     </f:block>

src/main/resources/org/jenkinsci/plugins/workflow/log/tee/TeeLogStorageFactory/config.properties

@@ -1 +1 @@
-description=Controls the Tee Log Storage factories order. Primary allows read and writes. Secondary allows writes only.
+description=Specify the multiple loggers order.

src/main/resources/org/jenkinsci/plugins/workflow/log/tee/TeeLogStorageFactory/help-primary.html

@@ -0,0 +1,3 @@
+<p>
+  <b>Primary</b> allows read and writes.
+</p>

src/main/resources/org/jenkinsci/plugins/workflow/log/tee/TeeLogStorageFactory/help-secondary.html

@@ -0,0 +1,3 @@
+<p>
+  <b>Secondary</b> allows writes only.
+</p>

src/test/java/org/jenkinsci/plugins/workflow/log/configuration/PipelineLoggingGlobalConfigurationJCasCTest.java

@@ -77,7 +77,7 @@ public void custom_default_factory_ui() throws Throwable {
                 form.getFirstByXPath("//*[@id='pipeline-logging']/../descendant::select/option[@selected]/text()");
         assertThat(selectedText, nullValue());
         var description = form.getFirstByXPath("//*[@id='pipeline-logging']/../descendant::div[@colspan]/text()");
-        assertThat(description.toString(), containsString("My Custom Log"));
+        assertThat(description.toString(), containsString("My custom log"));
         checkNoPipelineLoggingCasCConfiguration();
     }
 
@@ -162,7 +162,7 @@ public static final class DescriptorImpl extends LogStorageFactoryDescriptor<Log
             @NonNull
             @Override
             public String getDisplayName() {
-                return "My Custom Log";
+                return "My custom log";
             }
 
             @Override

src/test/java/org/jenkinsci/plugins/workflow/log/configuration/PipelineLoggingGlobalConfigurationTest.java

@@ -133,7 +133,7 @@ public static final class DescriptorImpl extends LogStorageFactoryDescriptor<Log
             @NonNull
             @Override
             public String getDisplayName() {
-                return "My Custom Log";
+                return "My custom log";
             }
 
             @Override