eggjs / egg-security

Security plugin for egg, force performance too.

MIT License

238 stars 43 forks source link

test: use expectLog to assert log #53

Closed fengmk2 closed 5 years ago

fengmk2 commented 5 years ago

make sure log assert is stable on windows too

dead-horse commented 5 years ago

还是不稳定。。

fengmk2 commented 5 years ago

看起来是 decodeURIComponent 没有挂，我打印日志调试一下。

codecov[bot] commented 5 years ago

Codecov Report

Merging #53 into master will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #53   +/-   ##
=======================================
  Coverage   96.13%   96.13%           
=======================================
  Files          30       30           
  Lines         492      492           
=======================================
  Hits          473      473           
  Misses         19       19

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update f7445d0...6fbefa8. Read the comment docs.

codecov[bot] commented 5 years ago

Codecov Report

Merging #53 into master will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #53   +/-   ##
=======================================
  Coverage   96.13%   96.13%           
=======================================
  Files          30       30           
  Lines         492      492           
=======================================
  Hits          473      473           
  Misses         19       19

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update f7445d0...6fbefa8. Read the comment docs.