search

karenc / db-migrator

GNU Affero General Public License v3.0
4 stars 5 forks source link

handle empty migration list case for wide listing #55

Open reedstrm opened 6 years ago

reedstrm commented 6 years ago

Hit this traceback recently:

Traceback (most recent call last):
  File "/home/reedstrm/src/rewrite/bin/dbmigrator", line 11, in <module>
    load_entry_point('db-migrator', 'console_scripts', 'dbmigrator')()
  File "/home/reedstrm/src/rewrite/db-migrator/dbmigrator/cli.py", line 116, in main
    return args['cmmd'](**args)
  File "/home/reedstrm/src/rewrite/db-migrator/dbmigrator/utils.py", line 155, in wrapper
    return func(cursor, *args, **kwargs)
  File "/home/reedstrm/src/rewrite/db-migrator/dbmigrator/commands/list.py", line 29, in cli_command
    name_width = max([len(name) for _, name, _ in migrations])
ValueError: max() arg is an empty sequence

This code solves it.

coveralls commented 6 years ago

Coverage Status

Coverage decreased (-0.4%) to 97.366% when pulling e98128169063855af3edf1028a3c389fe0b07a5e on Connexions:wide-empty-list into 7a8e0e2f513539638a0371b609e2e89816ba0d36 on karenc:master.