Skip to content
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: format of printing errs in e2e test #777

Closed
wants to merge 1 commit into from

Conversation

Uncle-Justice
Copy link
Contributor

Ⅰ. Describe what this PR did

  1. Content-Type为form-data类型时,通常会在后面跟一个boudary的数据,在header中实际应为(举例):"ContentType": "multipart/form-data; boundary=----WebKitFormBoundaryAnydWsQ1ajKuGoCd",为此body类型补充了一些检查逻辑
  2. CompareRequest&Response中的一些报错输出存在一些bug,比如对于body数据结构的类型断言失败时,不应打印body内容,因为类型未知。

Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?

Ⅳ. Describe how to verify it

运行所有e2e测试

Ⅴ. Special notes for reviews

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (a71ecf4) 38.08% compared to head (f8afea6) 38.08%.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #777   +/-   ##
=======================================
  Coverage   38.08%   38.08%           
=======================================
  Files          61       61           
  Lines       10323    10323           
=======================================
  Hits         3932     3932           
  Misses       6091     6091           
  Partials      300      300           

@Uncle-Justice
Copy link
Contributor Author

duplicate with #778

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants