[fit] correct Javadoc of isInvalidCookiePair & add default value for Cookie.Builder.value (#343)

* [fit] correct Javadoc of isInvalidCookiePair to reflect invalid check

* [fit] add default value for Cookie.Builder.value

* [fit] update cookie parse test

Author: Mai-icy

framework/fit/java/fit-builtin/services/fit-http-classic/definition/src/main/java/modelengine/fit/http/Cookie.java

@@ -7,6 +7,7 @@
 package modelengine.fit.http;
 
 import modelengine.fitframework.pattern.builder.BuilderFactory;
+import modelengine.fitframework.util.StringUtils;
 
 /**
  * 表示 Http 中的 Cookie。
@@ -210,7 +211,7 @@ interface Builder {
      * @return 表示 {@link Cookie} 的构建器的 {@link Builder}。
      */
     static Builder builder() {
-        return builder(null).maxAge(-1);
+        return builder(null).value(StringUtils.EMPTY).maxAge(-1).secure(false).httpOnly(false);
     }
 
     /**

framework/fit/java/fit-builtin/services/fit-http-classic/definition/src/main/java/modelengine/fit/http/util/HttpUtils.java

@@ -229,11 +229,11 @@ private static void parseCookieAttributes(String[] parts, Cookie.Builder builder
     }
 
     /**
-     * 验证给定的 Cookie 名称和值是否合法。
+     * 判断给定的 Cookie 名称和值是否无效。
      *
      * @param name 表示 Cookie 的名称 {@link String}。
      * @param value 表示 Cookie 的值 {@link String}，允许为空但不允许为 {@code null}，可带双引号。
-     * @return 如果 name 和 value 都合法返回 {@code true}，否则返回 {@code false}。
+     * @return 如果 name 或 value 无效返回 {@code true}，否则返回 {@code false}。
      */
     public static boolean isInvalidCookiePair(String name, String value) {
         if (StringUtils.isEmpty(name) || !TOKEN_PATTERN.matcher(name).matches()) {

framework/fit/java/fit-builtin/services/fit-http-classic/definition/src/test/java/modelengine/fit/http/util/HttpUtilsTest.java

@@ -82,7 +82,7 @@ void givenValidSetCookieStringThenParseSuccessfully() {
     void givenEmptySetCookieThenReturnEmptyCookie() {
         Cookie cookie = HttpUtils.parseSetCookie("");
         assertThat(cookie.name()).isNull();
-        assertThat(cookie.value()).isNull();
+        assertThat(cookie.value()).isEmpty();
     }
 
     @Test