"""Methods related to test expectations/expectation files."""
from __future__ import print_function
import collections
import copy
import datetime
import logging
import os
import re
import subprocess
import sys
from typing import Dict, FrozenSet, Iterable, List, Optional, Set, Tuple, Union
import six
from typ import expectations_parser
from unexpected_passes_common import data_types
from unexpected_passes_common import result_output
FINDER_DISABLE_COMMENT_BASE = 'finder:disable'
FINDER_ENABLE_COMMENT_BASE = 'finder:enable'
FINDER_COMMENT_SUFFIX_GENERAL = '-general'
FINDER_COMMENT_SUFFIX_STALE = '-stale'
FINDER_COMMENT_SUFFIX_UNUSED = '-unused'
FINDER_COMMENT_SUFFIX_NARROWING = '-narrowing'
FINDER_GROUP_COMMENT_START = 'finder:group-start'
FINDER_GROUP_COMMENT_END = 'finder:group-end'
ALL_FINDER_START_ANNOTATION_BASES = frozenset([
FINDER_DISABLE_COMMENT_BASE,
FINDER_GROUP_COMMENT_START,
])
ALL_FINDER_END_ANNOTATION_BASES = frozenset([
FINDER_ENABLE_COMMENT_BASE,
FINDER_GROUP_COMMENT_END,
])
ALL_FINDER_DISABLE_SUFFIXES = frozenset([
FINDER_COMMENT_SUFFIX_GENERAL,
FINDER_COMMENT_SUFFIX_STALE,
FINDER_COMMENT_SUFFIX_UNUSED,
FINDER_COMMENT_SUFFIX_NARROWING,
])
FINDER_DISABLE_COMMENT_GENERAL = (FINDER_DISABLE_COMMENT_BASE +
FINDER_COMMENT_SUFFIX_GENERAL)
FINDER_DISABLE_COMMENT_STALE = (FINDER_DISABLE_COMMENT_BASE +
FINDER_COMMENT_SUFFIX_STALE)
FINDER_DISABLE_COMMENT_UNUSED = (FINDER_DISABLE_COMMENT_BASE +
FINDER_COMMENT_SUFFIX_UNUSED)
FINDER_DISABLE_COMMENT_NARROWING = (FINDER_DISABLE_COMMENT_BASE +
FINDER_COMMENT_SUFFIX_NARROWING)
FINDER_ENABLE_COMMENT_GENERAL = (FINDER_ENABLE_COMMENT_BASE +
FINDER_COMMENT_SUFFIX_GENERAL)
FINDER_ENABLE_COMMENT_STALE = (FINDER_ENABLE_COMMENT_BASE +
FINDER_COMMENT_SUFFIX_STALE)
FINDER_ENABLE_COMMENT_UNUSED = (FINDER_ENABLE_COMMENT_BASE +
FINDER_COMMENT_SUFFIX_UNUSED)
FINDER_ENABLE_COMMENT_NARROWING = (FINDER_ENABLE_COMMENT_BASE +
FINDER_COMMENT_SUFFIX_NARROWING)
FINDER_DISABLE_COMMENTS = frozenset([
FINDER_DISABLE_COMMENT_GENERAL,
FINDER_DISABLE_COMMENT_STALE,
FINDER_DISABLE_COMMENT_UNUSED,
FINDER_DISABLE_COMMENT_NARROWING,
])
FINDER_ENABLE_COMMENTS = frozenset([
FINDER_ENABLE_COMMENT_GENERAL,
FINDER_ENABLE_COMMENT_STALE,
FINDER_ENABLE_COMMENT_UNUSED,
FINDER_ENABLE_COMMENT_NARROWING,
])
FINDER_ENABLE_DISABLE_PAIRS = frozenset([
(FINDER_DISABLE_COMMENT_GENERAL, FINDER_ENABLE_COMMENT_GENERAL),
(FINDER_DISABLE_COMMENT_STALE, FINDER_ENABLE_COMMENT_STALE),
(FINDER_DISABLE_COMMENT_UNUSED, FINDER_ENABLE_COMMENT_UNUSED),
(FINDER_DISABLE_COMMENT_NARROWING, FINDER_ENABLE_COMMENT_NARROWING),
])
FINDER_GROUP_COMMENTS = frozenset([
FINDER_GROUP_COMMENT_START,
FINDER_GROUP_COMMENT_END,
])
ALL_FINDER_COMMENTS = frozenset(FINDER_DISABLE_COMMENTS
| FINDER_ENABLE_COMMENTS
| FINDER_GROUP_COMMENTS)
GIT_BLAME_REGEX = re.compile(
r'^[\w\s]+\(.+(?P<date>\d\d\d\d-\d\d-\d\d)[^\)]+\)(?P<content>.*)$',
re.DOTALL)
TAG_GROUP_REGEX = re.compile(r'# tags: \[([^\]]*)\]', re.MULTILINE | re.DOTALL)
ANNOTATION_COMMENT_START_PATTERN = r' *# '
ANNOTATION_OPTIONAL_TRAILING_TEXT_PATTERN = r'[^\n]*\n'
ANNOTATION_REQUIRED_TRAILING_TEXT_PATTERN = r'[^\n]+\n'
BLANK_OR_COMMENT_LINES_PATTERN = r'(?:\s*| *#[^\n]*\n)*'
ALL_STALE_COMMENT_REGEXES = set()
for start_comment, end_comment in FINDER_ENABLE_DISABLE_PAIRS:
ALL_STALE_COMMENT_REGEXES.add(
re.compile(
ANNOTATION_COMMENT_START_PATTERN + start_comment +
ANNOTATION_OPTIONAL_TRAILING_TEXT_PATTERN +
BLANK_OR_COMMENT_LINES_PATTERN + ANNOTATION_COMMENT_START_PATTERN +
end_comment + r'\n', re.MULTILINE | re.DOTALL))
ALL_STALE_COMMENT_REGEXES.add(
re.compile(
ANNOTATION_COMMENT_START_PATTERN + FINDER_GROUP_COMMENT_START +
ANNOTATION_REQUIRED_TRAILING_TEXT_PATTERN +
BLANK_OR_COMMENT_LINES_PATTERN + ANNOTATION_COMMENT_START_PATTERN +
FINDER_GROUP_COMMENT_END + r'\n', re.MULTILINE | re.DOTALL))
ALL_STALE_COMMENT_REGEXES = frozenset(ALL_STALE_COMMENT_REGEXES)
_registered_instance = None
def GetInstance() -> 'Expectations':
return _registered_instance
def RegisterInstance(instance: 'Expectations') -> None:
global _registered_instance
assert _registered_instance is None
assert isinstance(instance, Expectations)
_registered_instance = instance
def ClearInstance() -> None:
global _registered_instance
_registered_instance = None
class RemovalType(object):
STALE = FINDER_COMMENT_SUFFIX_STALE
UNUSED = FINDER_COMMENT_SUFFIX_UNUSED
NARROWING = FINDER_COMMENT_SUFFIX_NARROWING
class Expectations(object):
def __init__(self):
self._cached_tag_groups = {}
def CreateTestExpectationMap(
self, expectation_files: Optional[Union[str, List[str]]],
tests: Optional[Iterable[str]],
grace_period: datetime.timedelta) -> data_types.TestExpectationMap:
"""Creates an expectation map based off a file or list of tests.
Args:
expectation_files: A filepath or list of filepaths to expectation files to
read from, or None. If a filepath is specified, |tests| must be None.
tests: An iterable of strings containing test names to check. If
specified, |expectation_file| must be None.
grace_period: A datetime.timedelta specifying how many days old an
expectation must be in order to be parsed, i.e. how many days old an
expectation must be before it is a candidate for removal/modification.
Returns:
A data_types.TestExpectationMap, although all its BuilderStepMap contents
will be empty.
"""
def AddContentToMap(content: str, ex_map: data_types.TestExpectationMap,
expectation_file_name: str) -> None:
list_parser = expectations_parser.TaggedTestListParser(content)
expectations_for_file = ex_map.setdefault(
expectation_file_name, data_types.ExpectationBuilderMap())
logging.debug('Parsed %d expectations', len(list_parser.expectations))
for e in list_parser.expectations:
if 'Skip' in e.raw_results:
continue
if e.raw_results == ['Pass']:
continue
expectation = data_types.Expectation(e.test, e.tags, e.raw_results,
e.reason)
assert expectation not in expectations_for_file
expectations_for_file[expectation] = data_types.BuilderStepMap()
logging.info('Creating test expectation map')
assert expectation_files or tests
assert not (expectation_files and tests)
expectation_map = data_types.TestExpectationMap()
if expectation_files:
if not isinstance(expectation_files, list):
expectation_files = [expectation_files]
for ef in expectation_files:
expectation_file_name = os.path.normpath(ef).replace(os.path.sep, '/')
content = self._GetNonRecentExpectationContent(expectation_file_name,
grace_period)
AddContentToMap(content, expectation_map, expectation_file_name)
else:
expectation_file_name = ''
content = '# results: [ RetryOnFailure ]\n'
for t in tests:
content += '%s [ RetryOnFailure ]\n' % t
AddContentToMap(content, expectation_map, expectation_file_name)
return expectation_map
def _GetNonRecentExpectationContent(self, expectation_file_path: str,
num_days: datetime.timedelta) -> str:
"""Gets content from |expectation_file_path| older than |num_days| days.
Args:
expectation_file_path: A string containing a filepath pointing to an
expectation file.
num_days: A datetime.timedelta containing how old an expectation in the
given expectation file must be to be included.
Returns:
The contents of the expectation file located at |expectation_file_path|
as a string with any recent expectations removed.
"""
content = ''
cmd = ['git', 'blame', '-c', expectation_file_path]
with open(os.devnull, 'w', newline='', encoding='utf-8') as devnull:
blame_output = subprocess.check_output(cmd,
stderr=devnull).decode('utf-8')
for line in blame_output.splitlines(True):
match = GIT_BLAME_REGEX.match(line)
assert match
date = match.groupdict()['date']
line_content = match.groupdict()['content']
stripped_line_content = line_content.strip()
if not stripped_line_content or stripped_line_content.startswith('#'):
content += line_content
else:
if six.PY2:
date_parts = date.split('-')
date = datetime.date(year=int(date_parts[0]),
month=int(date_parts[1]),
day=int(date_parts[2]))
else:
date = datetime.date.fromisoformat(date)
date_diff = datetime.date.today() - date
if date_diff > num_days:
content += line_content
else:
logging.debug('Omitting expectation %s because it is too new',
line_content.rstrip())
return content
def RemoveExpectationsFromFile(self,
expectations: List[data_types.Expectation],
expectation_file: str,
removal_type: str) -> Set[str]:
"""Removes lines corresponding to |expectations| from |expectation_file|.
Ignores any lines that match but are within a disable block or have an
inline disable comment.
Args:
expectations: A list of data_types.Expectations to remove.
expectation_file: A filepath pointing to an expectation file to remove
lines from.
removal_type: A RemovalType enum corresponding to the type of expectations
being removed.
Returns:
A set of strings containing URLs of bugs associated with the removed
expectations.
"""
with open(expectation_file, encoding='utf-8') as f:
input_contents = f.read()
group_to_expectations, expectation_to_group = (
self._GetExpectationGroupsFromFileContent(expectation_file,
input_contents))
disable_annotated_expectations = (
self._GetDisableAnnotatedExpectationsFromFile(expectation_file,
input_contents))
output_contents = ''
removed_urls = set()
removed_lines = set()
num_removed_lines = 0
for line_number, line in enumerate(input_contents.splitlines(True)):
stripped_line = line.strip()
if _IsCommentOrBlankLine(stripped_line):
output_contents += line
continue
current_expectation = self._CreateExpectationFromExpectationFileLine(
line, expectation_file)
if any(e for e in expectations if e == current_expectation):
disable_block_suffix, disable_block_reason = (
disable_annotated_expectations.get(current_expectation,
(None, None)))
if disable_block_suffix and _DisableSuffixIsRelevant(
disable_block_suffix, removal_type):
output_contents += line
logging.info(
'Would have removed expectation %s, but it is inside a disable '
'block or has an inline disable with reason %s', stripped_line,
disable_block_reason)
elif _ExpectationPartOfNonRemovableGroup(current_expectation,
group_to_expectations,
expectation_to_group,
expectations):
output_contents += line
logging.info(
'Would have removed expectation %s, but it is part of group "%s" '
'whose members are not all removable.', stripped_line,
expectation_to_group[current_expectation])
else:
bug = current_expectation.bug
if bug:
removed_urls |= set(bug.split())
removed_lines.add(line_number - num_removed_lines)
num_removed_lines += 1
else:
output_contents += line
header_length = len(
self._GetExpectationFileTagHeader(expectation_file).splitlines(True))
output_contents = _RemoveStaleComments(output_contents, removed_lines,
header_length)
with open(expectation_file, 'w', newline='', encoding='utf-8') as f:
f.write(output_contents)
return removed_urls
def _GetDisableAnnotatedExpectationsFromFile(
self, expectation_file: str,
content: str) -> Dict[data_types.Expectation, Tuple[str, str]]:
"""Extracts expectations which are affected by disable annotations.
Args:
expectation_file: A filepath pointing to an expectation file.
content: A string containing the contents of |expectation_file|.
Returns:
A dict mapping data_types.Expectation to (disable_suffix, disable_reason).
If an expectation is present in this dict, it is affected by a disable
annotation of some sort. |disable_suffix| is a string specifying which
type of annotation is applicable, while |disable_reason| is a string
containing the comment/reason why the disable annotation is present.
"""
in_disable_block = False
disable_block_reason = ''
disable_block_suffix = ''
disable_annotated_expectations = {}
for line in content.splitlines(True):
stripped_line = line.strip()
if _IsCommentOrBlankLine(stripped_line):
assert len([c for c in ALL_FINDER_COMMENTS if c in line]) <= 1
if _LineContainsDisableComment(line):
if in_disable_block:
raise RuntimeError(
'Invalid expectation file %s - contains a disable comment "%s" '
'that is in another disable block.' %
(expectation_file, stripped_line))
in_disable_block = True
disable_block_reason = _GetDisableReasonFromComment(line)
disable_block_suffix = _GetFinderCommentSuffix(line)
elif _LineContainsEnableComment(line):
if not in_disable_block:
raise RuntimeError(
'Invalid expectation file %s - contains an enable comment "%s" '
'that is outside of a disable block.' %
(expectation_file, stripped_line))
in_disable_block = False
continue
current_expectation = self._CreateExpectationFromExpectationFileLine(
line, expectation_file)
if in_disable_block:
disable_annotated_expectations[current_expectation] = (
disable_block_suffix, disable_block_reason)
elif _LineContainsDisableComment(line):
disable_block_reason = _GetDisableReasonFromComment(line)
disable_block_suffix = _GetFinderCommentSuffix(line)
disable_annotated_expectations[current_expectation] = (
disable_block_suffix, disable_block_reason)
return disable_annotated_expectations
def _GetExpectationGroupsFromFileContent(
self, expectation_file: str, content: str
) -> Tuple[Dict[str, Set[data_types.Expectation]], Dict[data_types.
Expectation, str]]:
"""Extracts all groups of expectations from an expectationfile.
Args:
expectation_file: A filepath pointing to an expectation file.
content: A string containing the contents of |expectation_file|.
Returns:
A tuple (group_to_expectations, expectation_to_group).
|group_to_expectations| is a dict of group names to sets of
data_type.Expectations that belong to that group. |expectation_to_group|
is the same, but mapped the other way from data_type.Expectations to group
names.
"""
group_to_expectations = collections.defaultdict(set)
expectation_to_group = {}
group_name = None
for line in content.splitlines():
stripped_line = line.strip()
if _IsCommentOrBlankLine(stripped_line):
if _LineContainsGroupStartComment(stripped_line):
if group_name:
raise RuntimeError(
'Invalid expectation file %s - contains a group comment "%s" '
'that is inside another group block.' %
(expectation_file, stripped_line))
group_name = _GetGroupNameFromCommentLine(stripped_line)
elif _LineContainsGroupEndComment(stripped_line):
if not group_name:
raise RuntimeError(
'Invalid expectation file %s - contains a group comment "%s" '
'without a group start comment.' %
(expectation_file, stripped_line))
group_name = None
elif group_name:
e = self._CreateExpectationFromExpectationFileLine(
stripped_line, expectation_file)
group_to_expectations[group_name].add(e)
expectation_to_group[e] = group_name
return group_to_expectations, expectation_to_group
def _CreateExpectationFromExpectationFileLine(self, line: str,
expectation_file: str
) -> data_types.Expectation:
"""Creates a data_types.Expectation from |line|.
Args:
line: A string containing a single line from an expectation file.
expectation_file: A filepath pointing to an expectation file |line| came
from.
Returns:
A data_types.Expectation containing the same information as |line|.
"""
header = self._GetExpectationFileTagHeader(expectation_file)
single_line_content = header + line
list_parser = expectations_parser.TaggedTestListParser(single_line_content)
assert len(list_parser.expectations) == 1
typ_expectation = list_parser.expectations[0]
return data_types.Expectation(typ_expectation.test, typ_expectation.tags,
typ_expectation.raw_results,
typ_expectation.reason)
def _GetExpectationFileTagHeader(self, expectation_file: str) -> str:
"""Gets the tag header used for expectation files.
Args:
expectation_file: A filepath pointing to an expectation file to get the
tag header from.
Returns:
A string containing an expectation file header, i.e. the comment block at
the top of the file defining possible tags and expected results.
"""
raise NotImplementedError()
def ParseTaggedTestListContent(self, content: str
) -> expectations_parser.TaggedTestListParser:
"""Helper to parse typ expectation files.
This allows subclasses to avoid adding typ to PYTHONPATH.
"""
return expectations_parser.TaggedTestListParser(content)
def FilterToKnownTags(self, tags: Iterable[str]) -> Set[str]:
"""Filters |tags| to only include tags known to expectation files.
Args:
tags: An iterable of strings containing tags.
Returns:
A set containing the elements of |tags| with any tags that are not defined
in any expectation files removed.
"""
return self._GetKnownTags() & set(tags)
def _GetKnownTags(self) -> Set[str]:
"""Gets all known/defined tags from expectation files.
Returns:
A set of strings containing all known/defined tags from expectation files.
"""
raise NotImplementedError()
def _FilterToMostSpecificTypTags(self, typ_tags: FrozenSet[str],
expectation_file: str) -> FrozenSet[str]:
"""Filters |typ_tags| to the most specific set.
Assumes that the tags in |expectation_file| are ordered from least specific
to most specific within each tag group.
Args:
typ_tags: A frozenset of strings containing the typ tags to filter.
expectations_file: A string containing a filepath pointing to the
expectation file to filter tags with.
Returns:
A frozenset containing the contents of |typ_tags| with only the most
specific tag from each group remaining.
"""
if expectation_file not in self._cached_tag_groups:
with open(expectation_file, encoding='utf-8') as infile:
contents = infile.read()
tag_groups = []
for match in TAG_GROUP_REGEX.findall(contents):
tag_groups.append(match.lower().strip().replace('#', '').split())
self._cached_tag_groups[expectation_file] = tag_groups
tag_groups = self._cached_tag_groups[expectation_file]
num_matches = 0
tags_in_same_group = collections.defaultdict(list)
for tag in typ_tags:
for index, tag_group in enumerate(tag_groups):
if tag in tag_group:
tags_in_same_group[index].append(tag)
num_matches += 1
break
if num_matches != len(typ_tags):
all_tags = set()
for group in tag_groups:
all_tags |= set(group)
raise RuntimeError('Found tags not in expectation file %s: %s' %
(expectation_file, ' '.join(set(typ_tags) - all_tags)))
filtered_tags = set()
for index, tags in tags_in_same_group.items():
if len(tags) == 1:
filtered_tags.add(tags[0])
else:
tag_group = tag_groups[index]
best_index = -1
for t in tags:
i = tag_group.index(t)
if i > best_index:
best_index = i
filtered_tags.add(tag_group[best_index])
return frozenset(filtered_tags)
def _ConsolidateKnownOverlappingTags(self, typ_tags: FrozenSet[str]
) -> FrozenSet[str]:
"""Consolidates tags that are known to overlap/cause issues.
One known example of this would be dual GPU machines that report tags for
both GPUs.
"""
return typ_tags
def NarrowSemiStaleExpectationScope(
self, stale_expectation_map: data_types.TestExpectationMap) -> Set[str]:
"""Narrows the scope of expectations in |stale_expectation_map|.
Expectations are modified such that they only apply to configurations that
need them, to the best extent possible. If scope narrowing is not possible,
e.g. the same hardware/software combination reports fully passing on one bot
but reports some failures on another bot, the expectation will not be
modified.
Args:
stale_expectation_map: A data_types.TestExpectationMap containing
semi-stale expectations.
Returns:
A set of strings containing URLs of bugs associated with the modified
expectations.
"""
modified_urls = set()
cached_disable_annotated_expectations = {}
for expectation_file, e, builder_map in (
stale_expectation_map.IterBuilderStepMaps()):
if expectation_file not in cached_disable_annotated_expectations:
with open(expectation_file, encoding='utf-8') as infile:
disable_annotated_expectations = (
self._GetDisableAnnotatedExpectationsFromFile(
expectation_file, infile.read()))
cached_disable_annotated_expectations[
expectation_file] = disable_annotated_expectations
disable_block_suffix, disable_block_reason = (
cached_disable_annotated_expectations[expectation_file].get(
e, ('', '')))
if _DisableSuffixIsRelevant(disable_block_suffix, RemovalType.NARROWING):
logging.info(
'Skipping semi-stale narrowing check for expectation %s since it '
'has a narrowing disable annotation with reason %s',
e.AsExpectationFileString(), disable_block_reason)
continue
skip_to_next_expectation = False
pass_tag_sets = set()
fail_tag_sets = set()
for builder, step, build_stats in builder_map.IterBuildStats():
if len(build_stats.tag_sets) > 1:
logging.warning(
'Step %s on builder %s produced multiple tag sets: %s. Not '
'narrowing expectation scope for expectation %s.', step, builder,
build_stats.tag_sets, e.AsExpectationFileString())
skip_to_next_expectation = True
break
if build_stats.NeverNeededExpectation(e):
pass_tag_sets |= build_stats.tag_sets
else:
fail_tag_sets |= build_stats.tag_sets
if skip_to_next_expectation:
continue
common_tags = set()
for ts in pass_tag_sets:
common_tags |= ts
break
for ts in pass_tag_sets | fail_tag_sets:
common_tags &= ts
common_tags -= e.tags
pass_tag_sets = {ts - common_tags for ts in pass_tag_sets}
fail_tag_sets = {ts - common_tags for ts in fail_tag_sets}
new_tag_sets = set()
covered_fail_tag_sets = set()
for fail_tags in fail_tag_sets:
if any(fail_tags <= pt for pt in pass_tag_sets):
logging.warning(
'Unable to determine what makes failing configs unique for %s, '
'not narrowing expectation scope.', e.AsExpectationFileString())
skip_to_next_expectation = True
break
if fail_tags in covered_fail_tag_sets:
continue
tag_set_to_add = fail_tags
for ft in fail_tag_sets:
if ft in covered_fail_tag_sets:
continue
intersection = tag_set_to_add & ft
if any(intersection <= pt for pt in pass_tag_sets):
continue
if any(intersection <= cft for cft in covered_fail_tag_sets):
continue
tag_set_to_add = intersection
new_tag_sets.add(tag_set_to_add)
covered_fail_tag_sets.update(cft for cft in fail_tag_sets
if tag_set_to_add <= cft)
if skip_to_next_expectation:
continue
new_tag_sets = {
self._ConsolidateKnownOverlappingTags(nts)
for nts in new_tag_sets
}
new_tag_sets = {
self._FilterToMostSpecificTypTags(nts, expectation_file)
for nts in new_tag_sets
}
with open(expectation_file, encoding='utf-8') as infile:
file_contents = infile.read()
line, _ = self._GetExpectationLine(e, file_contents, expectation_file)
modified_urls |= set(e.bug.split())
expectation_strs = []
for new_tags in new_tag_sets:
expectation_copy = copy.copy(e)
expectation_copy.tags = new_tags
expectation_strs.append(expectation_copy.AsExpectationFileString())
expectation_strs.sort()
replacement_lines = '\n'.join(expectation_strs)
file_contents = file_contents.replace(line, replacement_lines)
with open(expectation_file, 'w', newline='', encoding='utf-8') as outfile:
outfile.write(file_contents)
return modified_urls
def _GetExpectationLine(self, expectation: data_types.Expectation,
file_contents: str, expectation_file: str
) -> Union[Tuple[None, None], Tuple[str, int]]:
"""Gets the line and line number of |expectation| in |file_contents|.
Args:
expectation: A data_types.Expectation.
file_contents: A string containing the contents read from an expectation
file.
expectation_file: A string containing the path to the expectation file
that |file_contents| came from.
Returns:
A tuple (line, line_number). |line| is a string containing the exact line
in |file_contents| corresponding to |expectation|. |line_number| is an int
corresponding to where |line| is in |file_contents|. |line_number| may be
off if the file on disk has changed since |file_contents| was read. If a
corresponding line cannot be found, both |line| and |line_number| are
None.
"""
file_lines = file_contents.splitlines()
for line_number, line in enumerate(file_lines):
if _IsCommentOrBlankLine(line.strip()):
continue
current_expectation = self._CreateExpectationFromExpectationFileLine(
line, expectation_file)
if expectation == current_expectation:
return line, line_number + 1
return None, None
def FindOrphanedBugs(self, affected_urls: Iterable[str]) -> Set[str]:
"""Finds cases where expectations for bugs no longer exist.
Args:
affected_urls: An iterable of affected bug URLs, as returned by functions
such as RemoveExpectationsFromFile.
Returns:
A set containing a subset of |affected_urls| who no longer have any
associated expectations in any expectation files.
"""
seen_bugs = set()
expectation_files = self.GetExpectationFilepaths()
for ef in expectation_files:
with open(ef, encoding='utf-8') as infile:
contents = infile.read()
for url in affected_urls:
if url in seen_bugs:
continue
if url in contents:
seen_bugs.add(url)
return set(affected_urls) - seen_bugs
def GetExpectationFilepaths(self) -> List[str]:
"""Gets all the filepaths to expectation files of interest.
Returns:
A list of strings, each element being a filepath pointing towards an
expectation file.
"""
raise NotImplementedError()
def _LineContainsGroupStartComment(line: str) -> bool:
return FINDER_GROUP_COMMENT_START in line
def _LineContainsGroupEndComment(line: str) -> bool:
return FINDER_GROUP_COMMENT_END in line
def _LineContainsDisableComment(line: str) -> bool:
return FINDER_DISABLE_COMMENT_BASE in line
def _LineContainsEnableComment(line: str) -> bool:
return FINDER_ENABLE_COMMENT_BASE in line
def _GetGroupNameFromCommentLine(line: str) -> str:
"""Gets the group name from the finder comment on the given line."""
assert FINDER_GROUP_COMMENT_START in line
uncommented_line = line.lstrip('#').strip()
split_line = uncommented_line.split(maxsplit=1)
if len(split_line) != 2:
raise RuntimeError('Given line %s did not have a group name.' % line)
return split_line[1]
def _GetFinderCommentSuffix(line: str) -> str:
"""Gets the suffix of the finder comment on the given line.
Examples:
'foo # finder:disable' -> ''
'foo # finder:disable-stale some_reason' -> '-stale'
"""
target_str = None
if _LineContainsDisableComment(line):
target_str = FINDER_DISABLE_COMMENT_BASE
elif _LineContainsEnableComment(line):
target_str = FINDER_ENABLE_COMMENT_BASE
else:
raise RuntimeError('Given line %s did not have a finder comment.' % line)
line = line[line.find(target_str):]
line = line.split()[0]
suffix = line.replace(target_str, '')
assert suffix in ALL_FINDER_DISABLE_SUFFIXES
return suffix
def _LineContainsRelevantDisableComment(line: str, removal_type: str) -> bool:
"""Returns whether the given line contains a relevant disable comment.
Args:
line: A string containing the line to check.
removal_type: A RemovalType enum corresponding to the type of expectations
being removed.
Returns:
A bool denoting whether |line| contains a relevant disable comment given
|removal_type|.
"""
if FINDER_DISABLE_COMMENT_GENERAL in line:
return True
if FINDER_DISABLE_COMMENT_BASE + removal_type in line:
return True
return False
def _DisableSuffixIsRelevant(suffix: str, removal_type: str) -> bool:
"""Returns whether the given suffix is relevant given the removal type.
Args:
suffix: A string containing a disable comment suffix.
removal_type: A RemovalType enum corresponding to the type of expectations
being removed.
Returns:
True if suffix is relevant and its disable request should be honored.
"""
if suffix == FINDER_COMMENT_SUFFIX_GENERAL:
return True
if suffix == removal_type:
return True
return False
def _GetDisableReasonFromComment(line: str) -> str:
suffix = _GetFinderCommentSuffix(line)
return line.split(FINDER_DISABLE_COMMENT_BASE + suffix, 1)[1].strip()
def _IsCommentOrBlankLine(line: str) -> bool:
return (not line or line.startswith('#'))
def _ExpectationPartOfNonRemovableGroup(
current_expectation: data_types.Expectation,
group_to_expectations: Dict[str, Set[data_types.Expectation]],
expectation_to_group: Dict[data_types.Expectation, str],
removable_expectations: List[data_types.Expectation]):
"""Determines if the given expectation is part of a non-removable group.
This is the case if the expectation is part of a group, but not all
expectations in that group are marked as removable.
Args:
current_expectation: A data_types.Expectation that is being checked.
group_to_expectations: A dict mapping group names to sets of expectations
contained within that group.
expectation_to_group: A dict mapping an expectation to the group name it
belongs to.
removable_expectations: A list of all expectations that are removable.
"""
removable_expectations = set(removable_expectations)
group_name = expectation_to_group.get(current_expectation)
if not group_name:
return False
all_expectations_in_group = group_to_expectations[group_name]
return not (all_expectations_in_group <= removable_expectations)
def _RemoveStaleComments(content: str, removed_lines: Set[int],
header_length: int) -> str:
"""Attempts to remove stale contents from the given expectation file content.
Args:
content: A string containing the contents of an expectation file.
removed_lines: A set of ints denoting which line numbers were removed in
the process of creating |content|.
header_length: An int denoting how many lines long the tag header is.
Returns:
A copy of |content| with various stale comments removed, e.g. group blocks
if the group has been removed.
"""
comment_line_numbers_to_remove = []
split_content = content.splitlines(True)
for rl in removed_lines:
found_trailing_annotation = False
found_starting_annotation = False
if rl < len(split_content):
stripped_line = split_content[rl].strip()
if stripped_line and not stripped_line.startswith('#'):
continue
if any(annotation in stripped_line
for annotation in ALL_FINDER_END_ANNOTATION_BASES):
found_trailing_annotation = True
comment_line_number = rl - 1
while comment_line_number != header_length - 1:
stripped_line = split_content[comment_line_number].strip()
if stripped_line.startswith('#'):
if any(annotation in stripped_line
for annotation in ALL_FINDER_END_ANNOTATION_BASES):
break
if any(annotation in stripped_line
for annotation in ALL_FINDER_START_ANNOTATION_BASES):
if found_starting_annotation:
comment_line_number -= 1
continue
found_starting_annotation = True
if found_starting_annotation and not found_trailing_annotation:
break
comment_line_numbers_to_remove.append(comment_line_number)
comment_line_number -= 1
else:
break
if found_trailing_annotation and found_starting_annotation:
comment_line_numbers_to_remove.append(rl)
for i in comment_line_numbers_to_remove:
split_content[i] = ''
if comment_line_numbers_to_remove:
content = ''.join(split_content)
for regex in ALL_STALE_COMMENT_REGEXES:
for match in regex.findall(content):
content = content.replace(match, '')
return content