Skip to content
This repository has been archived by the owner on Feb 24, 2024. It is now read-only.

Add nominal new app integration test #1219

Merged
merged 2 commits into from
Aug 7, 2018

Conversation

stanislas-m
Copy link
Member

@stanislas-m stanislas-m commented Aug 7, 2018

Here is what a nominal test case for buffalo new coke looks like.

@codecov
Copy link

codecov bot commented Aug 7, 2018

Codecov Report

Merging #1219 into development will increase coverage by 0.2%.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff               @@
##           development    #1219     +/-   ##
==============================================
+ Coverage        55.16%   55.36%   +0.2%     
==============================================
  Files               75       75             
  Lines             3865     3865             
==============================================
+ Hits              2132     2140      +8     
+ Misses            1603     1594      -9     
- Partials           130      131      +1
Impacted Files Coverage Δ
buffalo/cmd/new.go 58.15% <0%> (+5.67%) ⬆️

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 46c0eed...2dbaefc. Read the comment docs.

@stanislas-m stanislas-m requested a review from markbates August 7, 2018 13:00
@markbates
Copy link
Member

I can’t wait until we switch everything to genny. It’s crazy easy to test!

@markbates markbates merged commit b001c7e into development Aug 7, 2018
@markbates markbates modified the milestones: 0.12.0, v0.13.0 Aug 7, 2018
@stanislas-m stanislas-m deleted the task/poc-nominal-new-app-test branch August 7, 2018 20:44
@stanislas-m
Copy link
Member Author

@markbates In integration tests we attack the CLI interface just at the same level as the current Docker tests. genny will help, but not for this one. :)

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

Successfully merging this pull request may close these issues.

3 participants