)]}'
{"id":"openstack%2Fpython-openstackclient~277824","triplet_id":"openstack%2Fpython-openstackclient~master~I1afe4852069d25d562a9448ec2bf2cff58955052","project":"openstack/python-openstackclient","branch":"master","topic":"bug/1477199","hashtags":[],"change_id":"I1afe4852069d25d562a9448ec2bf2cff58955052","subject":"Fix DisplayCommandBase comments for cliff ShowOne subclass tests","status":"MERGED","created":"2016-02-09 13:16:45.000000000","updated":"2016-02-10 09:07:27.000000000","submitted":"2016-02-10 09:07:27.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":2,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"88a980ed3a9d1ba4659d743707bef2685521a5ff","_number":277824,"virtual_id_number":277824,"owner":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},{"value":0,"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"value":2,"date":"2016-02-10 09:07:27.000000000","post_submit":true,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":1941,"name":"Lin Hua Cheng","email":"os.lcheng@gmail.com","username":"lin-hua-cheng"},{"value":0,"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},{"value":0,"_account_id":17211,"name":"Brandon Palm","email":"bapalm@us.ibm.com","username":"bapalm"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"all":[{"value":0,"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},{"value":2,"date":"2016-02-10 08:14:42.000000000","_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":1941,"name":"Lin Hua Cheng","email":"os.lcheng@gmail.com","username":"lin-hua-cheng"},{"value":0,"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},{"value":0,"_account_id":17211,"name":"Brandon Palm","email":"bapalm@us.ibm.com","username":"bapalm"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"all":[{"value":0,"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},{"value":1,"date":"2016-02-10 08:14:42.000000000","_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":1941,"name":"Lin Hua Cheng","email":"os.lcheng@gmail.com","username":"lin-hua-cheng"},{"value":0,"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},{"value":0,"_account_id":17211,"name":"Brandon Palm","email":"bapalm@us.ibm.com","username":"bapalm"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"_account_id":1941,"name":"Lin Hua Cheng","email":"os.lcheng@gmail.com","username":"lin-hua-cheng"},{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},{"_account_id":17211,"name":"Brandon Palm","email":"bapalm@us.ibm.com","username":"bapalm"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2016-02-09 13:22:22.000000000","updated_by":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"reviewer":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"state":"REVIEWER"},{"updated":"2016-02-09 13:22:26.000000000","updated_by":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"reviewer":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"state":"REVIEWER"},{"updated":"2016-02-09 13:22:29.000000000","updated_by":{"_account_id":1941,"name":"Lin Hua Cheng","email":"os.lcheng@gmail.com","username":"lin-hua-cheng"},"reviewer":{"_account_id":1941,"name":"Lin Hua Cheng","email":"os.lcheng@gmail.com","username":"lin-hua-cheng"},"state":"REVIEWER"},{"updated":"2016-02-09 13:22:38.000000000","updated_by":{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},"reviewer":{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},"state":"REVIEWER"},{"updated":"2016-02-09 16:25:57.000000000","updated_by":{"_account_id":17211,"name":"Brandon Palm","email":"bapalm@us.ibm.com","username":"bapalm"},"reviewer":{"_account_id":17211,"name":"Brandon Palm","email":"bapalm@us.ibm.com","username":"bapalm"},"state":"REVIEWER"},{"updated":"2016-02-10 08:14:42.000000000","updated_by":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"reviewer":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"state":"REVIEWER"},{"updated":"2016-02-10 09:07:27.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"5f13d10c07bb0387c8edd805feb0dfa2eb30b9ea","author":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"date":"2016-02-09 13:16:45.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"2c311f6350efbca717f97e5091adab27f5a58989","author":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"date":"2016-02-09 13:20:34.000000000","message":"Patch Set 1:\n\nThis series of patches fixed the comments in all tests except those in dir tests/identity. There are about 150+ bad comments in dir tests/identity. So I intend to fix them all in another patch.","accounts_in_message":[],"_revision_number":1},{"id":"90f7f9a4bf2bcb4a7ecdc35ca6966b4e924e6fea","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-02-09 14:18:43.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-openstackclient-pep8 http://logs.openstack.org/24/277824/1/check/gate-python-openstackclient-pep8/5ca6b40/ : SUCCESS in 49s\n- gate-python-openstackclient-docs http://docs-draft.openstack.org/24/277824/1/check/gate-python-openstackclient-docs/4c95d6e//doc/build/html/ : SUCCESS in 1m 53s\n- gate-python-openstackclient-python27 http://logs.openstack.org/24/277824/1/check/gate-python-openstackclient-python27/7d84ed5/ : SUCCESS in 1m 02s\n- gate-python-openstackclient-python34 http://logs.openstack.org/24/277824/1/check/gate-python-openstackclient-python34/faff882/ : SUCCESS in 2m 15s\n- gate-tempest-dsvm-neutron-src-python-openstackclient http://logs.openstack.org/24/277824/1/check/gate-tempest-dsvm-neutron-src-python-openstackclient/7dd7d4d/ : SUCCESS in 56m 05s\n- check-osc-plugins http://logs.openstack.org/24/277824/1/check/check-osc-plugins/4dc4421/ : SUCCESS in 40s (non-voting)\n- gate-osc-dsvm-functional http://logs.openstack.org/24/277824/1/check/gate-osc-dsvm-functional/db47871/ : SUCCESS in 25m 36s","accounts_in_message":[],"_revision_number":1},{"id":"a725064d2f482f6aecf9002239e364ba26f6024c","author":{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},"date":"2016-02-09 16:14:02.000000000","message":"Patch Set 1: Code-Review+1\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"f13d182e987f275b4ba6b081b3b0d643373cfae3","author":{"_account_id":17211,"name":"Brandon Palm","email":"bapalm@us.ibm.com","username":"bapalm"},"date":"2016-02-09 16:25:57.000000000","message":"Patch Set 1: Code-Review+1","accounts_in_message":[],"_revision_number":1},{"id":"2d8a3e5fa8bc5f4dd21cbf5a9fd5d543a24b3d29","author":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"date":"2016-02-10 03:27:14.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"f2e47daab3243341530d893b7af541338f557c26","author":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"date":"2016-02-10 03:40:27.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"e466b1f8fa2a3784aafb3188cbc1505b3541dcae","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-02-10 04:44:29.000000000","message":"Patch Set 2: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- gate-python-openstackclient-pep8 http://logs.openstack.org/24/277824/2/check/gate-python-openstackclient-pep8/219fd5e/ : FAILURE in 46s\n- gate-python-openstackclient-docs http://docs-draft.openstack.org/24/277824/2/check/gate-python-openstackclient-docs/1f504a1//doc/build/html/ : SUCCESS in 1m 56s\n- gate-python-openstackclient-python27 http://logs.openstack.org/24/277824/2/check/gate-python-openstackclient-python27/ff8542e/ : SUCCESS in 2m 04s\n- gate-python-openstackclient-python34 http://logs.openstack.org/24/277824/2/check/gate-python-openstackclient-python34/dff2504/ : SUCCESS in 2m 11s\n- gate-tempest-dsvm-neutron-src-python-openstackclient http://logs.openstack.org/24/277824/2/check/gate-tempest-dsvm-neutron-src-python-openstackclient/57cbe8f/ : SUCCESS in 1h 01m 09s\n- check-osc-plugins http://logs.openstack.org/24/277824/2/check/check-osc-plugins/edc1a69/ : SUCCESS in 1m 18s (non-voting)\n- gate-osc-dsvm-functional http://logs.openstack.org/24/277824/2/check/gate-osc-dsvm-functional/b66a3ff/ : SUCCESS in 21m 15s","accounts_in_message":[],"_revision_number":2},{"id":"fa9b818478e7bfc11bd6a6c29d9749a6e3d917f9","author":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"date":"2016-02-10 05:32:22.000000000","message":"Uploaded patch set 3: Patch Set 2 was rebased.","accounts_in_message":[],"_revision_number":3},{"id":"15013d0a964b5d4d15dca707769202df2191cf00","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-02-10 06:27:39.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-openstackclient-pep8 http://logs.openstack.org/24/277824/3/check/gate-python-openstackclient-pep8/a9e1fca/ : SUCCESS in 36s\n- gate-python-openstackclient-docs http://docs-draft.openstack.org/24/277824/3/check/gate-python-openstackclient-docs/8ff7829//doc/build/html/ : SUCCESS in 1m 39s\n- gate-python-openstackclient-python27 http://logs.openstack.org/24/277824/3/check/gate-python-openstackclient-python27/07cfd18/ : SUCCESS in 59s\n- gate-python-openstackclient-python34 http://logs.openstack.org/24/277824/3/check/gate-python-openstackclient-python34/780a4fe/ : SUCCESS in 1m 44s\n- gate-tempest-dsvm-neutron-src-python-openstackclient http://logs.openstack.org/24/277824/3/check/gate-tempest-dsvm-neutron-src-python-openstackclient/1f2a9ff/ : SUCCESS in 47m 58s\n- check-osc-plugins http://logs.openstack.org/24/277824/3/check/check-osc-plugins/af78614/ : SUCCESS in 46s (non-voting)\n- gate-osc-dsvm-functional http://logs.openstack.org/24/277824/3/check/gate-osc-dsvm-functional/c9ab6f9/ : SUCCESS in 18m 49s","accounts_in_message":[],"_revision_number":3},{"id":"39a3f0d4cc7c725ecc98d7b55dc9f89bcd599c4a","author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"date":"2016-02-10 08:14:42.000000000","message":"Patch Set 3: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":3},{"id":"36dc414be5606932de9c2516017a7d171a1cb047","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-02-10 08:15:04.000000000","message":"Patch Set 3: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":3},{"id":"788180c73445ada962e6502226850fc1d9852523","date":"2016-02-10 09:07:27.000000000","message":"Change has been successfully merged into the git repository by Jenkins","accounts_in_message":[],"_revision_number":3},{"id":"2ecae549722c78cff5ae82ed26bf7864ace49a72","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-02-10 09:07:27.000000000","message":"Patch Set 3: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-python-openstackclient-docs http://docs-draft.openstack.org/24/277824/3/gate/gate-python-openstackclient-docs/9037bf0//doc/build/html/ : SUCCESS in 2m 12s\n- gate-python-openstackclient-pep8 http://logs.openstack.org/24/277824/3/gate/gate-python-openstackclient-pep8/375ad2d/ : SUCCESS in 54s\n- gate-python-openstackclient-python27 http://logs.openstack.org/24/277824/3/gate/gate-python-openstackclient-python27/1246b63/ : SUCCESS in 1m 58s\n- gate-python-openstackclient-python34 http://logs.openstack.org/24/277824/3/gate/gate-python-openstackclient-python34/29469a8/ : SUCCESS in 2m 15s\n- gate-tempest-dsvm-neutron-src-python-openstackclient http://logs.openstack.org/24/277824/3/gate/gate-tempest-dsvm-neutron-src-python-openstackclient/f1f8f8f/ : SUCCESS in 46m 45s\n- gate-osc-dsvm-functional http://logs.openstack.org/24/277824/3/gate/gate-osc-dsvm-functional/30d5e8a/ : SUCCESS in 18m 42s","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"35833f7bd82ac5d7cad4b5fba0e498d8936e6f48","revisions":{"b3d07234c4f583fba1d705bc1a7537f38c47f316":{"kind":"REWORK","_number":1,"created":"2016-02-09 13:16:45.000000000","uploader":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"ref":"refs/changes/24/277824/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-openstackclient","ref":"refs/changes/24/277824/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/24/277824/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/24/277824/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/24/277824/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-openstackclient refs/changes/24/277824/1"}}},"commit":{"parents":[{"commit":"487142cca00e50cf66c2b780709895fb9316ffb0","subject":"Fix DisplayCommandBase comments for cliff Lister subclass tests","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/487142cca00e50cf66c2b780709895fb9316ffb0"}]}],"author":{"name":"Tang Chen","email":"chen.tang@easystack.cn","date":"2016-02-09 13:04:49.000000000","tz":480},"committer":{"name":"Tang Chen","email":"chen.tang@easystack.cn","date":"2016-02-09 13:04:49.000000000","tz":480},"subject":"Fix DisplayCommandBase comments for cliff ShowOne subclass tests","message":"Fix DisplayCommandBase comments for cliff ShowOne subclass tests\n\nAs bug #1477199 describes, the wrong comment below is all over the\nunit test code of OSC.\n\n    # DisplayCommandBase.take_action() returns two tuples\n\nThere is no such class named DisplayCommandBase in OSC. It is in cliff.\n\nAll OSC command classes inherit from the base classes in cliff,\nclass Command, class Lister and class ShowOne. It is like this:\n\nObject\n|--\u003e Command\n     |--\u003e DisplayCommandBase\n          |--\u003e Lister\n          |--\u003e ShowOne\n\ntake_action() is an abstract method of class Command, and generally is\noverwritten by subclasses.\n* Command.take_action() returns nothing.\n* Lister.take_action() returns a tuple which contains a tuple of columns\n  and a generator used to generate the data.\n* ShowOne.take_action() returns an iterator which contains a tuple of\n  columns and a tuple of data.\n\nSo, this problem should be fixed in 3 steps:\n1. Remove all DisplayCommandBase comments for tests of classes inheriting\n   from class Command in cliff as it returns nothing.\n2. Fix all DisplayCommandBase comments for tests of classes inheriting\n   from class Lister in cliff. Lister.take_action() returns a tuple and\n   a generator.\n3. Fix all DisplayCommandBase comments for tests of classes inheriting\n   from class ShowOne in cliff. ShowOne.take_action() returns two tuples.\n\nThis patch finishes step 3 in all but identity tests. There are too many\nsuch comments in identity tests. So fix them all in another patch.\n\nChange-Id: I1afe4852069d25d562a9448ec2bf2cff58955052\nPartial-bug: #1477199\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/b3d07234c4f583fba1d705bc1a7537f38c47f316"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/b3d07234c4f583fba1d705bc1a7537f38c47f316"}]},"branch":"refs/heads/master"},"ebc909cc7ab239334b99609d81e5e97268761bca":{"kind":"REWORK","_number":2,"created":"2016-02-10 03:40:27.000000000","uploader":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"ref":"refs/changes/24/277824/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-openstackclient","ref":"refs/changes/24/277824/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/24/277824/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/24/277824/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/24/277824/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-openstackclient refs/changes/24/277824/2"}}},"commit":{"parents":[{"commit":"45f50c9393ba2c018dd6b06499c7253221d27a53","subject":"Fix DisplayCommandBase comments for cliff Lister subclass tests","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/45f50c9393ba2c018dd6b06499c7253221d27a53"}]}],"author":{"name":"Tang Chen","email":"chen.tang@easystack.cn","date":"2016-02-09 13:04:49.000000000","tz":480},"committer":{"name":"Tang Chen","email":"chen.tang@easystack.cn","date":"2016-02-10 03:33:44.000000000","tz":480},"subject":"Fix DisplayCommandBase comments for cliff ShowOne subclass tests","message":"Fix DisplayCommandBase comments for cliff ShowOne subclass tests\n\nAs bug #1477199 describes, the wrong comment below is all over the\nunit test code of OSC.\n\n    # DisplayCommandBase.take_action() returns two tuples\n\nThere is no such class named DisplayCommandBase in OSC. It is in cliff.\n\nAll OSC command classes inherit from the base classes in cliff,\nclass Command, class Lister and class ShowOne. It is like this:\n\nObject\n|--\u003e Command\n     |--\u003e DisplayCommandBase\n          |--\u003e Lister\n          |--\u003e ShowOne\n\ntake_action() is an abstract method of class Command, and generally is\noverwritten by subclasses.\n* Command.take_action() returns nothing.\n* Lister.take_action() returns a tuple which contains a tuple of columns\n  and a generator used to generate the data.\n* ShowOne.take_action() returns an iterator which contains a tuple of\n  columns and a tuple of data.\n\nSo, this problem should be fixed in 3 steps:\n1. Remove all DisplayCommandBase comments for tests of classes inheriting\n   from class Command in cliff as it returns nothing.\n2. Fix all DisplayCommandBase comments for tests of classes inheriting\n   from class Lister in cliff. Lister.take_action() returns a tuple and\n   a generator.\n3. Fix all DisplayCommandBase comments for tests of classes inheriting\n   from class ShowOne in cliff. ShowOne.take_action() returns two tuples.\n\nThis patch finishes step 3 in all but identity tests. There are too many\nsuch comments in identity tests. So fix them all in another patch.\n\nChange-Id: I1afe4852069d25d562a9448ec2bf2cff58955052\nPartial-bug: #1477199\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/ebc909cc7ab239334b99609d81e5e97268761bca"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/ebc909cc7ab239334b99609d81e5e97268761bca"}]},"branch":"refs/heads/master"},"35833f7bd82ac5d7cad4b5fba0e498d8936e6f48":{"kind":"TRIVIAL_REBASE","_number":3,"created":"2016-02-10 05:32:22.000000000","uploader":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"ref":"refs/changes/24/277824/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-openstackclient","ref":"refs/changes/24/277824/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/24/277824/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/24/277824/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/24/277824/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-openstackclient refs/changes/24/277824/3"}}},"commit":{"parents":[{"commit":"e69b88ef525445325843c129e1a113a92ed98e3a","subject":"Fix DisplayCommandBase comments for cliff Lister subclass tests","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/e69b88ef525445325843c129e1a113a92ed98e3a"}]}],"author":{"name":"Tang Chen","email":"chen.tang@easystack.cn","date":"2016-02-09 13:04:49.000000000","tz":480},"committer":{"name":"Tang Chen","email":"chen.tang@easystack.cn","date":"2016-02-10 05:31:38.000000000","tz":480},"subject":"Fix DisplayCommandBase comments for cliff ShowOne subclass tests","message":"Fix DisplayCommandBase comments for cliff ShowOne subclass tests\n\nAs bug #1477199 describes, the wrong comment below is all over the\nunit test code of OSC.\n\n    # DisplayCommandBase.take_action() returns two tuples\n\nThere is no such class named DisplayCommandBase in OSC. It is in cliff.\n\nAll OSC command classes inherit from the base classes in cliff,\nclass Command, class Lister and class ShowOne. It is like this:\n\nObject\n|--\u003e Command\n     |--\u003e DisplayCommandBase\n          |--\u003e Lister\n          |--\u003e ShowOne\n\ntake_action() is an abstract method of class Command, and generally is\noverwritten by subclasses.\n* Command.take_action() returns nothing.\n* Lister.take_action() returns a tuple which contains a tuple of columns\n  and a generator used to generate the data.\n* ShowOne.take_action() returns an iterator which contains a tuple of\n  columns and a tuple of data.\n\nSo, this problem should be fixed in 3 steps:\n1. Remove all DisplayCommandBase comments for tests of classes inheriting\n   from class Command in cliff as it returns nothing.\n2. Fix all DisplayCommandBase comments for tests of classes inheriting\n   from class Lister in cliff. Lister.take_action() returns a tuple and\n   a generator.\n3. Fix all DisplayCommandBase comments for tests of classes inheriting\n   from class ShowOne in cliff. ShowOne.take_action() returns two tuples.\n\nThis patch finishes step 3 in all but identity tests. There are too many\nsuch comments in identity tests. So fix them all in another patch.\n\nChange-Id: I1afe4852069d25d562a9448ec2bf2cff58955052\nPartial-bug: #1477199\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/35833f7bd82ac5d7cad4b5fba0e498d8936e6f48"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/35833f7bd82ac5d7cad4b5fba0e498d8936e6f48"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}