blob: 9735f77369c84803f0c5c4593713c9056e81a746 [file] [log] [blame]
Sean Daguea3d2ab72013-01-12 08:43:49 -05001#!/usr/bin/env python
Jay Pipes257d3f82012-07-08 23:01:31 -04002
ZhiQiang Fan39f97222013-09-20 04:49:44 +08003# Copyright 2012 OpenStack Foundation
Jay Pipes257d3f82012-07-08 23:01:31 -04004# All Rights Reserved.
5#
6# Licensed under the Apache License, Version 2.0 (the "License"); you may
7# not use this file except in compliance with the License. You may obtain
8# a copy of the License at
9#
10# http://www.apache.org/licenses/LICENSE-2.0
11#
12# Unless required by applicable law or agreed to in writing, software
13# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
14# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
15# License for the specific language governing permissions and limitations
16# under the License.
17
18"""
19Track test skips via launchpadlib API and raise alerts if a bug
20is fixed but a skip is still in the Tempest test code
21"""
22
Jay Pipes257d3f82012-07-08 23:01:31 -040023import os
24import re
25
26from launchpadlib import launchpad
zhuzeyuf5501332017-01-23 18:27:18 +080027from oslo_log import log as logging
Jay Pipes257d3f82012-07-08 23:01:31 -040028
29BASEDIR = os.path.abspath(os.path.join(os.path.dirname(__file__), '..'))
Giampaolo Lauria07f51e62013-05-23 16:08:07 -040030TESTDIR = os.path.join(BASEDIR, 'tempest')
Jay Pipes257d3f82012-07-08 23:01:31 -040031LPCACHEDIR = os.path.expanduser('~/.launchpadlib/cache')
32
zhuzeyuf5501332017-01-23 18:27:18 +080033LOG = logging.getLogger(__name__)
34
Jay Pipes257d3f82012-07-08 23:01:31 -040035
36def info(msg, *args, **kwargs):
zhuzeyuf5501332017-01-23 18:27:18 +080037 LOG.info(msg, *args, **kwargs)
Jay Pipes257d3f82012-07-08 23:01:31 -040038
39
40def debug(msg, *args, **kwargs):
zhuzeyuf5501332017-01-23 18:27:18 +080041 LOG.debug(msg, *args, **kwargs)
Jay Pipes257d3f82012-07-08 23:01:31 -040042
43
44def find_skips(start=TESTDIR):
Ken'ichi Ohmichi7616d192015-11-17 13:12:55 +000045 """Find skipped tests
46
Jay Pipes257d3f82012-07-08 23:01:31 -040047 Returns a list of tuples (method, bug) that represent
48 test methods that have been decorated to skip because of
49 a particular bug.
50 """
Matthew Treinishbae2a992013-10-16 18:28:10 -040051 results = {}
Jay Pipes257d3f82012-07-08 23:01:31 -040052 debug("Searching in %s", start)
53 for root, _dirs, files in os.walk(start):
54 for name in files:
55 if name.startswith('test_') and name.endswith('py'):
56 path = os.path.join(root, name)
57 debug("Searching in %s", path)
Matthew Treinishbae2a992013-10-16 18:28:10 -040058 temp_result = find_skips_in_file(path)
59 for method_name, bug_no in temp_result:
60 if results.get(bug_no):
61 result_dict = results.get(bug_no)
62 if result_dict.get(name):
63 result_dict[name].append(method_name)
64 else:
65 result_dict[name] = [method_name]
66 results[bug_no] = result_dict
67 else:
68 results[bug_no] = {name: [method_name]}
Jay Pipes257d3f82012-07-08 23:01:31 -040069 return results
70
71
72def find_skips_in_file(path):
Ken'ichi Ohmichi7616d192015-11-17 13:12:55 +000073 """Return the skip tuples in a test file"""
Giulio Fidente83181a92013-10-01 06:02:24 +020074 BUG_RE = re.compile(r'\s*@.*skip_because\(bug=[\'"](\d+)[\'"]')
75 DEF_RE = re.compile(r'\s*def (\w+)\(')
Jay Pipes257d3f82012-07-08 23:01:31 -040076 bug_found = False
77 results = []
zhang.leia4b1cef2016-03-01 10:50:01 +080078 with open(path, 'rb') as content:
79 lines = content.readlines()
80 for x, line in enumerate(lines):
81 if not bug_found:
82 res = BUG_RE.match(line)
83 if res:
84 bug_no = int(res.group(1))
85 debug("Found bug skip %s on line %d", bug_no, x + 1)
86 bug_found = True
87 else:
88 res = DEF_RE.match(line)
89 if res:
90 method = res.group(1)
91 debug("Found test method %s skips for bug %d",
92 method, bug_no)
93 results.append((method, bug_no))
94 bug_found = False
Jay Pipes257d3f82012-07-08 23:01:31 -040095 return results
96
97
Matthew Treinishbae2a992013-10-16 18:28:10 -040098def get_results(result_dict):
99 results = []
Joe H. Rahmea72f2c62016-07-11 16:28:19 +0200100 for bug_no in result_dict:
Matthew Treinishbae2a992013-10-16 18:28:10 -0400101 for method in result_dict[bug_no]:
102 results.append((method, bug_no))
103 return results
104
105
Jay Pipes257d3f82012-07-08 23:01:31 -0400106if __name__ == '__main__':
Jay Pipes257d3f82012-07-08 23:01:31 -0400107 results = find_skips()
Matthew Treinishbae2a992013-10-16 18:28:10 -0400108 unique_bugs = sorted(set([bug for (method, bug) in get_results(results)]))
Jay Pipes257d3f82012-07-08 23:01:31 -0400109 unskips = []
Matthew Treinishd2a4c082013-03-11 15:13:42 -0400110 duplicates = []
Jay Pipes257d3f82012-07-08 23:01:31 -0400111 info("Total bug skips found: %d", len(results))
112 info("Total unique bugs causing skips: %d", len(unique_bugs))
Jay Pipesa6aa5f22012-07-24 19:40:29 -0400113 lp = launchpad.Launchpad.login_anonymously('grabbing bugs',
114 'production',
115 LPCACHEDIR)
Jay Pipes257d3f82012-07-08 23:01:31 -0400116 for bug_no in unique_bugs:
117 bug = lp.bugs[bug_no]
Matthew Treinishd2a4c082013-03-11 15:13:42 -0400118 duplicate = bug.duplicate_of_link
119 if duplicate is not None:
120 dup_id = duplicate.split('/')[-1]
121 duplicates.append((bug_no, dup_id))
Jay Pipes257d3f82012-07-08 23:01:31 -0400122 for task in bug.bug_tasks:
Jay Pipesa6aa5f22012-07-24 19:40:29 -0400123 info("Bug #%7s (%12s - %12s)", bug_no,
124 task.importance, task.status)
Jay Pipes257d3f82012-07-08 23:01:31 -0400125 if task.status in ('Fix Released', 'Fix Committed'):
126 unskips.append(bug_no)
127
Matthew Treinishd2a4c082013-03-11 15:13:42 -0400128 for bug_id, dup_id in duplicates:
129 if bug_id not in unskips:
130 dup_bug = lp.bugs[dup_id]
131 for task in dup_bug.bug_tasks:
132 info("Bug #%7s is a duplicate of Bug#%7s (%12s - %12s)",
133 bug_id, dup_id, task.importance, task.status)
134 if task.status in ('Fix Released', 'Fix Committed'):
135 unskips.append(bug_id)
136
137 unskips = sorted(set(unskips))
Jay Pipes257d3f82012-07-08 23:01:31 -0400138 if unskips:
Dirk Mueller1db5db22013-06-23 20:21:32 +0200139 print("The following bugs have been fixed and the corresponding skips")
140 print("should be removed from the test cases:")
141 print()
Jay Pipes257d3f82012-07-08 23:01:31 -0400142 for bug in unskips:
Matthew Treinishbae2a992013-10-16 18:28:10 -0400143 message = " %7s in " % bug
144 locations = ["%s" % x for x in results[bug].keys()]
145 message += " and ".join(locations)
146 print(message)