Skip to content

test function name#1395

Merged
tiangolo merged 1 commit intofastapi:masterfrom
chenl:patch-2
Jun 13, 2020
Merged

test function name#1395
tiangolo merged 1 commit intofastapi:masterfrom
chenl:patch-2

Conversation

@chenl
Copy link
Contributor

@chenl chenl commented May 10, 2020

We are testing that creating an item that already exists should fail, not that creating an item with a token that already exists should fail.

test_create_existing_token -> test_create_existing_item
@codecov
Copy link

codecov bot commented May 10, 2020

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff            @@
##            master     #1395   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          226       226           
  Lines         6785      6785           
=========================================
  Hits          6785      6785           

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 4e77737...d55de97. Read the comment docs.

@tiangolo tiangolo merged commit fe15620 into fastapi:master Jun 13, 2020
@tiangolo
Copy link
Member

Ah! Good point 🤓 🔍

Thanks!

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