Bug #6766

hammer activation-key pagination failed

Added by Martin Bacovsky over 3 years ago. Updated over 3 years ago.

Status:Closed
Priority:Normal
Assigned To:Martin Bacovsky
Category:-
Target version:Foreman - Sprint 27
Difficulty: Bugzilla link:1109934
Found in release: Pull request:https://github.com/theforeman/hammer-cli-foreman/pull/138
Story points-
Velocity based estimate-

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1109934
Description of problem:
hammer activation-key pagination failed

Version-Release number of selected component (if applicable):

How reproducible:

Steps to Reproduce:
1. Create 10 different activation keys under ACME_Corporation

  1. hammer activation-key list --organization-label='ACME_Corporation' --page=1 --per-page=2
    ---|--------|----------------|-----------------------|-------------
    ID | NAME | CONSUMED | LIFECYCLE ENVIRONMENT | CONTENT VIEW
    ---|--------|----------------|-----------------------|-------------
    5 | actkey | 0 of Unlimited | |
    8 | ak1234 | 0 of Unlimited | | cv1
    ---|--------|----------------|-----------------------|-------------
    List next page? (Y/n): Y
    Error: can't convert Fixnum into String
  1. hammer activation-key list --organization-label='ACME_Corporation' --page='1' --per-page='2'
    ---|--------|----------------|-----------------------|-------------
    ID | NAME | CONSUMED | LIFECYCLE ENVIRONMENT | CONTENT VIEW
    ---|--------|----------------|-----------------------|-------------
    5 | actkey | 0 of Unlimited | |
    8 | ak1234 | 0 of Unlimited | | cv1
    ---|--------|----------------|-----------------------|-------------
    List next page? (Y/n): y
    Error: can't convert Fixnum into String

Actual results:
Pagination failed

Expected results:
Pagination should work as expected

Additional info:

Associated revisions

Revision 3f9c4266
Added by Martin Bačovský over 3 years ago

Fixes #6766 - Pagination is failing

Revision a050b05f
Added by mbacovsky over 3 years ago

Merge pull request #138 from mbacovsky/6766_failing_pagination

Fixes #6766 - Pagination is failing

History

#1 Updated by The Foreman Bot over 3 years ago

  • Status changed from New to Ready For Testing
  • Target version set to Sprint 26
  • Pull request https://github.com/theforeman/hammer-cli-foreman/pull/138 added

#2 Updated by Dmitri Dolguikh over 3 years ago

  • Target version changed from Sprint 26 to Sprint 27

#3 Updated by Anonymous over 3 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

Also available in: Atom PDF