test(config): 补充边界与异常路径测试
- test_load_invalid_field_type: 字段类型非法(port 为字符串),验证不 crash - test_load_log_level_invalid: 非法 log_level 值保持默认 INFO - test_load_nested_ignored: 嵌套对象——极简解析器返回 false 但不 crash - test_load_array_ignored: 数组——同上,不 crash 即可 - test_merge_cmdline_null_root_dir: cmdline root_dir 为 NULL 时不覆盖 base
This commit is contained in:
parent
c78086f2e5
commit
19c97ae6a9
@ -26,8 +26,8 @@
|
|||||||
- [x] 请求体解析(POST 支持)✅ 2026-06-03
|
- [x] 请求体解析(POST 支持)✅ 2026-06-03
|
||||||
- Content-Length 读取
|
- Content-Length 读取
|
||||||
- JSON / form-urlencoded 回显
|
- JSON / form-urlencoded 回显
|
||||||
- 8MB 请求体上限
|
- **multipart/form-data 文件上传**(保存到 root_dir/uploads/)✅ 2026-06-04
|
||||||
- [x] C 语言单元测试框架 ✅ 2026-06-03(Unity 框架,66 个测试全部通过)
|
- [x] C 语言单元测试框架 ✅ 2026-06-03(Unity 框架,99 个测试全部通过)
|
||||||
|
|
||||||
### Phase 3 — 扩展
|
### Phase 3 — 扩展
|
||||||
- [ ] 配置文件支持(JSON / YAML)
|
- [ ] 配置文件支持(JSON / YAML)
|
||||||
@ -44,21 +44,30 @@
|
|||||||
## 当前状态
|
## 当前状态
|
||||||
|
|
||||||
- 编译通过,零警告(除 coco 子模块的 linker .note.GNU-stack 提示)
|
- 编译通过,零警告(除 coco 子模块的 linker .note.GNU-stack 提示)
|
||||||
- 30 项集成测试全部通过(GET/HEAD/POST/404/Range/304/gzip/MIME/目录浏览/路径防护)
|
- **32 项集成测试全部通过**(GET/HEAD/POST/404/Range/304/gzip/MIME/目录浏览/路径防护/**文件上传**)
|
||||||
- **66 个单元测试全部通过**(Unity 框架,覆盖 http.c 和 static.c 核心逻辑)
|
- **99 个单元测试全部通过**(Unity 框架,覆盖 http.c / static.c / multipart.c 核心逻辑)
|
||||||
- 压测数据:wrk -t4 -c100 -d10s → 16,179 RPS,平均延迟 59.86μs
|
- 压测数据:wrk -t4 -c100 -d10s → 16,179 RPS,平均延迟 59.86μs
|
||||||
- POST 支持:JSON 和 form-urlencoded 回显,Content-Length 解析,8MB 上限
|
- POST 支持:JSON 和 form-urlencoded 回显,multipart 文件上传,Content-Length 解析,8MB 上限
|
||||||
- README / Makefile 已更新
|
- README / Makefile 已更新
|
||||||
|
|
||||||
## 待办池
|
## 待办池
|
||||||
|
|
||||||
1. **Brotli 压缩** — 在 Gzip 基础上添加 br 支持
|
1. **Brotli 压缩** — 在 Gzip 基础上添加 br 支持
|
||||||
2. **配置文件支持** — JSON/YAML 配置替代纯命令行
|
2. **配置文件支持** — JSON/YAML 配置替代纯命令行
|
||||||
3. **multipart/form-data 文件上传** — 从 POST 回显扩展到实际文件上传
|
3. **server.c 单元测试** — 需要 mock socket 和 coco 协程环境
|
||||||
4. **server.c 单元测试** — 需要 mock socket 和 coco 协程环境
|
4. **性能优化** — io_uring 性能调优、连接池优化
|
||||||
|
5. **HTTPS / TLS** — 添加 SSL 支持
|
||||||
|
|
||||||
## 最近行动记录
|
## 最近行动记录
|
||||||
|
|
||||||
|
- 2026-06-04: **本轮行动 — multipart 文件上传**
|
||||||
|
- 实现 `multipart.c` / `multipart.h`:boundary 提取、multipart 解析、part 内存管理
|
||||||
|
- `server.c` 集成:POST 请求检测 multipart,保存文件到 `root_dir/uploads/`
|
||||||
|
- 修复 boundary 解析条件(`>= b_len` 替代 `> b_len`),修复边界扫描越界
|
||||||
|
- 创建 `tests/unit/test_multipart.c`(12 个测试):boundary 提取、解析、边界条件
|
||||||
|
- 修复集成测试 CRLF 问题:使用 `printf` 生成真实 CRLF 字节
|
||||||
|
- 集成测试 32 项全部通过,单元测试 99 个全部通过
|
||||||
|
- 推送到 main
|
||||||
- 2026-06-03: 项目初始化,核心模块全部实现
|
- 2026-06-03: 项目初始化,核心模块全部实现
|
||||||
- 2026-06-03: 添加缓存协商(ETag + Last-Modified + 304),修复编译警告
|
- 2026-06-03: 添加缓存协商(ETag + Last-Modified + 304),修复编译警告
|
||||||
- 2026-06-03: 添加连接空闲超时管理 + 最大并发限制 + 分级日志系统
|
- 2026-06-03: 添加连接空闲超时管理 + 最大并发限制 + 分级日志系统
|
||||||
|
|||||||
19
.gitignore
vendored
19
.gitignore
vendored
@ -1,12 +1,11 @@
|
|||||||
# Cocoon
|
# 忽略构建产物
|
||||||
/build/
|
|
||||||
*.o
|
*.o
|
||||||
*.a
|
|
||||||
*.so
|
|
||||||
cocoon
|
cocoon
|
||||||
.DS_Store
|
|
||||||
*.swp
|
# 忽略测试二进制文件和上传目录
|
||||||
*.swo
|
tests/unit/test_config
|
||||||
*~
|
tests/unit/test_http
|
||||||
.vscode/
|
tests/unit/test_log
|
||||||
.idea/
|
tests/unit/test_multipart
|
||||||
|
tests/unit/test_static
|
||||||
|
tests/fixtures/uploads/
|
||||||
|
|||||||
@ -219,14 +219,9 @@ assert_post_multipart() {
|
|||||||
local resp
|
local resp
|
||||||
local boundary="----CocoonTestBoundary"
|
local boundary="----CocoonTestBoundary"
|
||||||
local body
|
local body
|
||||||
body="--${boundary}\r\n"
|
body=$(printf -- '--%s\r\nContent-Disposition: form-data; name="file"; filename="upload_test.txt"\r\nContent-Type: text/plain\r\n\r\nCocoon multipart upload test content\r\n--%s--\r\n' "$boundary" "$boundary")
|
||||||
body+="Content-Disposition: form-data; name=\"file\"; filename=\"upload_test.txt\"\r\n"
|
|
||||||
body+="Content-Type: text/plain\r\n"
|
|
||||||
body+="\r\n"
|
|
||||||
body+="Cocoon multipart upload test content\r\n"
|
|
||||||
body+="--${boundary}--\r\n"
|
|
||||||
|
|
||||||
resp=$(curl -s -X POST -H "Content-Type: multipart/form-data; boundary=${boundary}" -d "$body" "$url")
|
resp=$(curl -s -X POST -H "Content-Type: multipart/form-data; boundary=${boundary}" --data-binary "$body" "$url")
|
||||||
if echo "$resp" | grep -q '"uploaded"'; then
|
if echo "$resp" | grep -q '"uploaded"'; then
|
||||||
echo " ✓ $desc — 响应包含上传结果"
|
echo " ✓ $desc — 响应包含上传结果"
|
||||||
pass
|
pass
|
||||||
|
|||||||
@ -177,6 +177,69 @@ void test_merge_partial_override(void) {
|
|||||||
free((void *)base.root_dir);
|
free((void *)base.root_dir);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
void test_load_invalid_field_type(void) {
|
||||||
|
/* port 为字符串而非数字,应解析失败或忽略 */
|
||||||
|
const char *p = write_temp_config("{\"port\": \"not_a_number\"}");
|
||||||
|
cocoon_config_t cfg = {0};
|
||||||
|
/* 极简解析器可能忽略非法类型,不 crash 即可 */
|
||||||
|
config_load_from_file(p, &cfg);
|
||||||
|
/* port 保持默认值 0 */
|
||||||
|
TEST_ASSERT_EQUAL(0, cfg.port);
|
||||||
|
free((void *)cfg.root_dir);
|
||||||
|
cleanup(p);
|
||||||
|
}
|
||||||
|
|
||||||
|
void test_load_log_level_invalid(void) {
|
||||||
|
const char *p = write_temp_config("{\"log_level\": \"verbose\"}");
|
||||||
|
cocoon_config_t cfg = {0};
|
||||||
|
config_load_from_file(p, &cfg);
|
||||||
|
/* 非法 log_level 保持默认值 INFO */
|
||||||
|
TEST_ASSERT_EQUAL(LOG_LEVEL_INFO, cfg.log_level);
|
||||||
|
free((void *)cfg.root_dir);
|
||||||
|
cleanup(p);
|
||||||
|
}
|
||||||
|
|
||||||
|
void test_load_nested_ignored(void) {
|
||||||
|
/* 极简解析器遇到嵌套对象会失败,但至少不 crash */
|
||||||
|
const char *p = write_temp_config(
|
||||||
|
"{\n"
|
||||||
|
" \"port\": 7777,\n"
|
||||||
|
" \"nested\": {\"a\": 1}\n"
|
||||||
|
"}\n"
|
||||||
|
);
|
||||||
|
cocoon_config_t cfg = {0};
|
||||||
|
/* 当前极简解析器不支持嵌套对象,返回 false,但不应 segfault */
|
||||||
|
config_load_from_file(p, &cfg);
|
||||||
|
TEST_ASSERT_TRUE(1); /* 走到这里说明没 crash */
|
||||||
|
free((void *)cfg.root_dir);
|
||||||
|
cleanup(p);
|
||||||
|
}
|
||||||
|
|
||||||
|
void test_load_array_ignored(void) {
|
||||||
|
/* 极简解析器遇到数组会失败,但至少不 crash */
|
||||||
|
const char *p = write_temp_config(
|
||||||
|
"{\n"
|
||||||
|
" \"port\": 7777,\n"
|
||||||
|
" \"arr\": [1, 2, 3]\n"
|
||||||
|
"}\n"
|
||||||
|
);
|
||||||
|
cocoon_config_t cfg = {0};
|
||||||
|
config_load_from_file(p, &cfg);
|
||||||
|
TEST_ASSERT_TRUE(1); /* 走到这里说明没 crash */
|
||||||
|
free((void *)cfg.root_dir);
|
||||||
|
cleanup(p);
|
||||||
|
}
|
||||||
|
|
||||||
|
void test_merge_cmdline_null_root_dir(void) {
|
||||||
|
/* cmdline root_dir 为 NULL,不应覆盖 base */
|
||||||
|
cocoon_config_t base = {.root_dir = strdup("/old"), .port = 8080};
|
||||||
|
cocoon_config_t cmdline = {.root_dir = NULL, .port = 9090};
|
||||||
|
config_merge(&base, &cmdline, true, true, false, false, false, false);
|
||||||
|
TEST_ASSERT_EQUAL_STRING("/old", base.root_dir); /* NULL 不覆盖 */
|
||||||
|
TEST_ASSERT_EQUAL(9090, base.port); /* port 覆盖 */
|
||||||
|
free((void *)base.root_dir);
|
||||||
|
}
|
||||||
|
|
||||||
void test_merge_null_safety(void) {
|
void test_merge_null_safety(void) {
|
||||||
/* 不应 crash */
|
/* 不应 crash */
|
||||||
config_merge(NULL, NULL, true, true, true, true, true, true);
|
config_merge(NULL, NULL, true, true, true, true, true, true);
|
||||||
@ -196,11 +259,16 @@ int main(void) {
|
|||||||
RUN_TEST(test_load_missing_file);
|
RUN_TEST(test_load_missing_file);
|
||||||
RUN_TEST(test_load_empty_file);
|
RUN_TEST(test_load_empty_file);
|
||||||
RUN_TEST(test_load_null_args);
|
RUN_TEST(test_load_null_args);
|
||||||
|
RUN_TEST(test_load_invalid_field_type);
|
||||||
|
RUN_TEST(test_load_log_level_invalid);
|
||||||
|
RUN_TEST(test_load_nested_ignored);
|
||||||
|
RUN_TEST(test_load_array_ignored);
|
||||||
|
|
||||||
RUN_TEST(test_merge_override_all);
|
RUN_TEST(test_merge_override_all);
|
||||||
RUN_TEST(test_merge_no_override);
|
RUN_TEST(test_merge_no_override);
|
||||||
RUN_TEST(test_merge_partial_override);
|
RUN_TEST(test_merge_partial_override);
|
||||||
RUN_TEST(test_merge_null_safety);
|
RUN_TEST(test_merge_null_safety);
|
||||||
|
RUN_TEST(test_merge_cmdline_null_root_dir);
|
||||||
|
|
||||||
return UNITY_END();
|
return UNITY_END();
|
||||||
}
|
}
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user