Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tests(apps): test for apps:info, apps:destroy, and apps:transfer #233

Closed
wants to merge 7 commits into from

Conversation

ultimateboy
Copy link
Member

No description provided.

@deis-bot
Copy link

@bacongobbler, @Joshua-Anderson and @mboersma are potential reviewers of this pull request based on my analysis of git blame information. Thanks @ultimateboy!

@codecov-io
Copy link

codecov-io commented Sep 12, 2016

Current coverage is 27.65% (diff: 100%)

Merging #233 into master will increase coverage by 1.52%

@@             master       #233   diff @@
==========================================
  Files            56         57     +1   
  Lines          3861       3905    +44   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits           1009       1080    +71   
+ Misses         2719       2668    -51   
- Partials        133        157    +24   

Powered by Codecov. Last update a330059...4e032d9

ultimateboy and others added 5 commits September 12, 2016 14:20
* tests(builds): add test for 'deis builds:list --limit'

* tests(certs): add test for 'deis certs:list --limit'

* tests(domains): add test for 'deis domains:list --limit'

* tests(releases): add test for 'deis releases:list --limit'

* tests(users): add test for 'deis users:list --limit'

* tests(perms): add tests for 'deis perms:list' with and without '--limit'

* fix(tests): correct test user limit

* tests(keys): add test for 'deis keys:list --limit'

* fix(tests): correct test for keys list.

* tests(apps): add tests for 'apps:list' and 'apps:list --limit'
@ultimateboy
Copy link
Member Author

Had some crazy conflicts which weren't worth trying to cleanly merge. Closing this PR in favor of a cleaner one.

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.

None yet

4 participants