Project

General

Profile

Actions

Bug #21590

closed

Hammer should end with error when unknown output adapter is selected

Added by Tomáš Strachota over 6 years ago. Updated over 4 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Hammer core
Target version:
-
Fixed in Releases:
Found in Releases:
In Kanboard:

Description

Hammer defaults to 'base' output adapter if it can't find adapter specified in --output:

hammer --output=unknown user list                                                                                                                                                         
Id:            3
Login:         admin
Name:          Admin User
Email:         root@example.com
Admin:         yes
Last login:    2017/10/20 11:21:39
Authorized by: Internal

Id:            52
Login:         ipa_bob
Name:          Bob Doe
Email:         ipa_bob@ipa.test
Admin:         no
Last login:    2017/07/13 12:36:02
Authorized by: FreeIPA ldap

This is quite confusing behavior, especially when user passes 'yml' instead of the only supported 'yaml'. Both outputs aren't that different at a first glance.

Expected result:
Hammer should end with error instead, informing that the adapter name is unknown and listing the available ones.


Related issues 1 (0 open1 closed)

Related to Hammer CLI - Tracker #2823: Tracker for usability issuesClosed

Actions
Actions #1

Updated by Tomáš Strachota over 6 years ago

  • Category set to Hammer core
Actions #2

Updated by Tomáš Strachota over 6 years ago

Actions #3

Updated by yifat makias over 4 years ago

  • Status changed from New to Assigned
  • Assignee set to yifat makias
Actions #4

Updated by The Foreman Bot over 4 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/hammer-cli-foreman/pull/445 added
Actions #5

Updated by The Foreman Bot over 4 years ago

  • Pull request https://github.com/theforeman/hammer-cli-foreman/pull/446 added
Actions #6

Updated by The Foreman Bot over 4 years ago

  • Pull request https://github.com/theforeman/hammer-cli/pull/315 added
Actions #7

Updated by The Foreman Bot over 4 years ago

  • Fixed in Releases hammer-cli-0.19.0 added
Actions #8

Updated by yifat makias over 4 years ago

  • Status changed from Ready For Testing to Closed
Actions

Also available in: Atom PDF