search

octet-stream / form-data

Spec-compliant FormData implementation for Node.js
https://www.npmjs.com/package/formdata-node
MIT License
142 stars 17 forks source link

Correct Blob filename #20

Closed dbitting closed 4 years ago

dbitting commented 4 years ago

Operator precedence: || is evaluated before ?, so the ternary is incorrectly evaluated without additional parenthesis

codecov[bot] commented 4 years ago

Codecov Report

Merging #20 into master will not change coverage. The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #20   +/-   ##
=======================================
  Coverage   99.62%   99.62%           
=======================================
  Files          17       17           
  Lines         269      269           
  Branches       70       70           
=======================================
  Hits          268      268           
  Misses          1        1
Flag Coverage Δ
#unittests 99.62% <100.00%> (ø)
Impacted Files Coverage Δ
src/lib/FormData.mjs 100.00% <100.00%> (ø)

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 93f8e9d...9aa610a. Read the comment docs.

octet-stream commented 4 years ago

Looks good to me 👍 Thank you!

octet-stream commented 4 years ago

Published with v2.2.1