Bug 969235 - Add version check tests for server utilities
Summary: Add version check tests for server utilities
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Beaker
Classification: Retired
Component: scheduler
Version: develop
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: 24.0
Assignee: Dan Callaghan
QA Contact: tools-bugs
URL:
Whiteboard: ImplementationQuality
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-05-31 01:49 UTC by Nick Coghlan
Modified: 2018-02-06 00:41 UTC (History)
7 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2017-02-21 18:50:40 UTC
Embargoed:


Attachments (Terms of Use)

Description Nick Coghlan 2013-05-31 01:49:58 UTC
Several of the server command line tools report a hardcoded version of "0.1". Some of these tools can also be broken without causing an obvious failure in the unittest suite.

A sanity check test that ensures the output of "<command> --version" matches the expected Beaker for version for each of these scripts would ensure that:

1. At least import failures resulting from changes in other parts of the code will be detected by the test suite, even if full operation of the tool is not yet covered.

2. The tools report the correct version.

Comment 1 wangjing 2013-06-21 02:51:52 UTC
(In reply to Nick Coghlan from comment #0)
> 
> A sanity check test that ensures the output of "<command> --version" matches
> the expected Beaker for version for each of these scripts would ensure that:
> 

hi, what is the specific command for version check tests? thanks.

Comment 2 Nick Coghlan 2013-06-21 02:59:20 UTC
Documenting the server utilities is covered by bug 970403

The approximate list is given by the "console_scripts" entry in the server setup.py: http://git.beaker-project.org/cgit/beaker/tree/Server/setup.py.in?h=develop#n203

Comment 4 Dan Callaghan 2016-10-25 02:13:14 UTC
http://gerrit.beaker-project.org/5353

Comment 5 Dan Callaghan 2016-10-25 02:14:39 UTC
This picked up a few other pre-existing problems with stderr spew, so it was  definitely worthwhile.

http://gerrit.beaker-project.org/5354 beaker-repo-update: don't spew pointless warnings about RHEL repos
http://gerrit.beaker-project.org/5355 beaker-create-kickstart: fix unicode warnings
http://gerrit.beaker-project.org/5356 tests: run server commands in subprocesses wherever possible

Comment 6 Dan Callaghan 2016-11-07 06:55:57 UTC
This bug fix is included in beaker-server-24.0-0.git.194.c515848 which is currently available for download here:

https://beaker-project.org/nightlies/develop/

Comment 8 Dan Callaghan 2017-02-21 18:50:40 UTC
Beaker 24.0 has been released.


Note You need to log in before you can comment on or make changes to this bug.