Skip to content

test: add sameSite=none test case#14

Merged
fengmk2 merged 3 commits intomasterfrom
add-samesite-none-test
Feb 1, 2020
Merged

test: add sameSite=none test case#14
fengmk2 merged 3 commits intomasterfrom
add-samesite-none-test

Conversation

@fengmk2
Copy link
Copy Markdown
Member

@fengmk2 fengmk2 commented Feb 1, 2020

Checklist
  • npm test passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines
Affected core subsystem(s)
Description of change

@fengmk2 fengmk2 requested review from atian25 and dead-horse February 1, 2020 12:13
@fengmk2
Copy link
Copy Markdown
Member Author

fengmk2 commented Feb 1, 2020

deps on koajs/session#193

@codecov
Copy link
Copy Markdown

codecov bot commented Feb 1, 2020

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #14   +/-   ##
=======================================
  Coverage   96.42%   96.42%           
=======================================
  Files           4        4           
  Lines          28       28           
=======================================
  Hits           27       27           
  Misses          1        1
Impacted Files Coverage Δ
config/config.default.js 100% <ø> (ø) ⬆️

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 c9865a7...76bb2de. Read the comment docs.

@fengmk2 fengmk2 merged commit 75c8ee6 into master Feb 1, 2020
@fengmk2 fengmk2 deleted the add-samesite-none-test branch February 1, 2020 12:25
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