From 863de52f7ad70e12433392bdcefd87bac428894b Mon Sep 17 00:00:00 2001 From: Nikolay Krasko Date: Mon, 17 Aug 2020 19:14:23 +0300 Subject: [PATCH] Minor: better name for file walking with excludes --- .../org/jetbrains/kotlin/code/CodeConformanceTest.kt | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/compiler/tests/org/jetbrains/kotlin/code/CodeConformanceTest.kt b/compiler/tests/org/jetbrains/kotlin/code/CodeConformanceTest.kt index 9d0dfc26d98..c5f0cd25d8a 100644 --- a/compiler/tests/org/jetbrains/kotlin/code/CodeConformanceTest.kt +++ b/compiler/tests/org/jetbrains/kotlin/code/CodeConformanceTest.kt @@ -133,7 +133,7 @@ class CodeConformanceTest : TestCase() { val root = nonSourcesMatcher.root val extensions = File(root, ".bunch").readLines().map { it.split("_") }.flatten().toSet() val failBuilder = mutableListOf() - nonSourcesMatcher.walkTopDown(sourceBunchFilePattern).forEach { sourceFile -> + nonSourcesMatcher.excludeWalkTopDown(sourceBunchFilePattern).forEach { sourceFile -> val matches = Regex("BUNCH (\\w+)") .findAll(sourceFile.readText()) .map { it.groupValues[1] } @@ -208,7 +208,7 @@ class CodeConformanceTest : TestCase() { } ) - nonSourcesMatcher.walkTopDown(SOURCES_FILE_PATTERN).forEach { sourceFile -> + nonSourcesMatcher.excludeWalkTopDown(SOURCES_FILE_PATTERN).forEach { sourceFile -> val source = sourceFile.readText() for (test in tests) { if (test.filter(sourceFile, source)) test.result.add(sourceFile) @@ -234,7 +234,7 @@ class CodeConformanceTest : TestCase() { val copyrightRegex = Regex("""\bCopyright\b""") val root = COPYRIGHT_EXCLUDED_FILES_AND_DIRS_MATCHER.root - COPYRIGHT_EXCLUDED_FILES_AND_DIRS_MATCHER.walkTopDown(SOURCES_FILE_PATTERN) + COPYRIGHT_EXCLUDED_FILES_AND_DIRS_MATCHER.excludeWalkTopDown(SOURCES_FILE_PATTERN) .filter { sourceFile -> val relativePath = FileUtil.toSystemIndependentName(sourceFile.toRelativeString(root)) !knownThirdPartyCode.any { relativePath.startsWith(it) } @@ -274,7 +274,7 @@ class CodeConformanceTest : TestCase() { } } - private fun FileMatcher.walkTopDown(filePattern: Pattern): Sequence { + private fun FileMatcher.excludeWalkTopDown(filePattern: Pattern): Sequence { return root.walkTopDown() .onEnter { dir -> !matchExact(dir) // Don't enter to ignored dirs @@ -390,7 +390,7 @@ class CodeConformanceTest : TestCase() { data class RepoOccurrences(val repo: String, val files: Collection) val extensionsPattern = Pattern.compile(".+\\.(java|kt|gradle|kts|xml)(\\.\\w+)?") - val repoOccurrences: List = nonSourcesMatcher.walkTopDown(extensionsPattern) + val repoOccurrences: List = nonSourcesMatcher.excludeWalkTopDown(extensionsPattern) .flatMap { file -> val checkers = repoCheckers.filter { checker -> !checker.matcher.matchWithContains(file)