-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix structural parameters with POST request. #7193
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: kiy0taka <kiy0taka333@gmail.com>
Signed-off-by: kiy0taka <kiy0taka333@gmail.com>
b2ad2e1
to
122cfdb
Compare
Thanks - will review this asap... |
* What went wrong:
Execution failed for task ':zap:spotlessJavaCheck'.
> The following files had format violations:
zap/src/test/java/org/zaproxy/zap/model/StandardParameterParserUnitTest.java
@@ -19,6 +19,7 @@
·*/
package·org.zaproxy.zap.model;
+import·static·java.util.Arrays.asList;
import·static·org.hamcrest.MatcherAssert.assertThat;
import·static·org.hamcrest.Matchers.empty;
import·static·org.hamcrest.Matchers.equalTo;
@@ -26,7 +27,6 @@
import·static·org.hamcrest.Matchers.is;
import·static·org.hamcrest.Matchers.nullValue;
import·static·org.junit.jupiter.api.Assertions.assertEquals;
-import·static·java.util.Arrays.asList;
import·java.util.ArrayList;
import·java.util.List;
Run './gradlew :zap:spotlessApply' to fix these violations. |
Signed-off-by: kiy0taka <kiy0taka333@gmail.com>
Thanks @kiy0taka |
@@ -80,6 +82,53 @@ void defaultParser() { | |||
assertEquals(res2.get(3).getValue(), "g"); | |||
} | |||
|
|||
@Test | |||
void defaultTreePath() throws HttpMalformedHeaderException, URIException { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Better use parameterized tests.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also be good to add one that has both query and form parameters.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added test cases using ParameterizedTest.
842b7c5
to
c3f7af2
Compare
Signed-off-by: kiy0taka <kiy0taka333@gmail.com>
Signed-off-by: kiy0taka <kiy0taka333@gmail.com>
c3f7af2
to
15de3b7
Compare
With structural parameter
page
and following POST requests:Parameter names are used for nodes, so different values are combined into one node.
Fixed so that parameter values are used for different nodes.