Changeset 18491 in josm for trunk/test/unit/org

Timestamp:

2022-06-13T23:40:17+02:00 (3 years ago)

Author:

taylor.smock

Message:

Fix #20823: Reject uploads that do not follow either comment policy or source policy (patch by ljdelight)

This fixes an issue where a the length of the comment or source was checked first,
and a warning instead of a rejection was generated when the upload policy would
have otherwise rejected the comment or source.

This also performs the validation check first, prior to the user being able to
make changes in the upload dialog UI, so that the initial dialog state matches
the upload policy.

Location:

trunk/test/unit/org/openstreetmap/josm

Files:

• gui/io/UploadTextComponentValidatorTest.java (modified) (2 diffs)
• testutils/annotations/BasicPreferences.java (modified) (1 diff)

trunk/test/unit/org/openstreetmap/josm/gui/io/UploadTextComponentValidatorTest.java

@@ -5 +5 @@
 import static org.hamcrest.MatcherAssert.assertThat;
 
+import java.util.Arrays;
+import java.util.function.BiFunction;
+import java.util.stream.Stream;
+
 import javax.swing.JLabel;
 import javax.swing.JTextField;
 
 import org.junit.jupiter.api.Test;
+import org.junit.jupiter.params.ParameterizedTest;
+import org.junit.jupiter.params.provider.Arguments;
+import org.junit.jupiter.params.provider.MethodSource;
+import org.openstreetmap.josm.spi.preferences.Config;
 import org.openstreetmap.josm.testutils.annotations.BasicPreferences;
 
@@ -40 +48 @@
         assertThat(feedback.getText(), containsString("Thank you for providing the data source"));
     }
+
+    static Stream<Arguments> testUploadWithMandatoryTerm() {
+        return Stream.of(Arguments.of("upload.comment.mandatory-terms", "Thank you for providing a changeset comment",
+                    (BiFunction<JTextField, JLabel, ? extends UploadTextComponentValidator>)
+                            UploadTextComponentValidator.UploadCommentValidator::new),
+                Arguments.of("upload.source.mandatory-terms", "Thank you for providing the data source",
+                    (BiFunction<JTextField, JLabel, ? extends UploadTextComponentValidator>)
+                            UploadTextComponentValidator.UploadSourceValidator::new)
+        );
+    }
+
+    /**
+     * Unit test of {@link UploadTextComponentValidator.UploadCommentValidator} and
+     * {@link UploadTextComponentValidator.UploadSourceValidator} with mandatory terms
+     */
+    @BasicPreferences
+    @ParameterizedTest
+    @MethodSource
+    void testUploadWithMandatoryTerm(String confPref, String expectedText,
+            BiFunction<JTextField, JLabel, ? extends UploadTextComponentValidator> validatorSupplier) {
+        Config.getPref().putList(confPref, Arrays.asList("myrequired", "xyz"));
+        JTextField textField = new JTextField("");
+        JLabel feedback = new JLabel();
+
+        validatorSupplier.apply(textField, feedback);
+
+        // A too-short string should fail validation
+        textField.setText("");
+        assertThat(feedback.getText(), containsString("The following required terms are missing: [myrequired, xyz]"));
+
+        // A long enough string without the mandatory terms should claim that the required terms are missing
+        textField.setText("a string long enough but missing the mandatory term");
+        assertThat(feedback.getText(), containsString("The following required terms are missing: [myrequired, xyz]"));
+
+        // A valid string should pass
+        textField.setText("a string long enough with the mandatory term #myrequired #xyz");
+        assertThat(feedback.getText(), containsString(expectedText));
+    }
 }

trunk/test/unit/org/openstreetmap/josm/testutils/annotations/BasicPreferences.java

@@ -57 +57 @@
             // Disable saving on put, just to avoid overwriting pref files
             pref.enableSaveOnPut(false);
+            pref.resetToDefault();
             Config.setPreferencesInstance(pref);
             Config.setBaseDirectoriesProvider(JosmBaseDirectories.getInstance());