From 24aeedd92bace3a604be16a87a1b7b5313239df7 Mon Sep 17 00:00:00 2001
From: Phillip Johnsen <johphi@gmail.com>
Date: Thu, 7 Jul 2016 00:20:45 +0200
Subject: [PATCH] labels: prevent deps from getting "lib / src" label

---
 lib/node-labels.js       |  7 ++++++-
 test/node-labels.test.js | 15 +++++++++++++++
 2 files changed, 21 insertions(+), 1 deletion(-)

diff --git a/lib/node-labels.js b/lib/node-labels.js
index 1c95bacb..b8714504 100644
--- a/lib/node-labels.js
+++ b/lib/node-labels.js
@@ -104,7 +104,7 @@ function matchSubSystemsByRegex (rxLabelsMap, filepathsChanged, limitLib) {
 
     for (var i = 0; i < mappedSubSystems.length; ++i) {
       const mappedSubSystem = mappedSubSystems[i]
-      if (limitLib && jsSubsystemList.includes(mappedSubSystem)) {
+      if (limitLib && hasJsSubsystemChanges(filepathsChanged, mappedSubSystem)) {
         if (jsLabelCount.length >= 4) {
           for (const jsLabel of jsLabelCount) {
             delete map[jsLabel]
@@ -126,6 +126,11 @@ function matchSubSystemsByRegex (rxLabelsMap, filepathsChanged, limitLib) {
   return Object.keys(labelsMap)
 }
 
+function hasJsSubsystemChanges (filepathsChanged, mappedSubSystem) {
+  const hasLibChanges = filepathsChanged.some((filepath) => filepath.startsWith('lib/'))
+  return hasLibChanges && jsSubsystemList.includes(mappedSubSystem)
+}
+
 function mappedSubSystemsForFile (labelsMap, filepath) {
   for (const [regex, label] of labelsMap) {
     const matches = regex.exec(filepath)
diff --git a/test/node-labels.test.js b/test/node-labels.test.js
index 967b8ed2..3674b1ca 100644
--- a/test/node-labels.test.js
+++ b/test/node-labels.test.js
@@ -157,6 +157,21 @@ tap.test('label: "lib / src" when 5 or more JS sub-systems have been changed', (
   t.end()
 })
 
+// https://github.com/nodejs/node/pull/7488 wrongfully labelled with "lib / src"
+tap.test('label: not "lib / src" when only deps have been changed', (t) => {
+  const labels = nodeLabels.resolveLabels([
+    'deps/v8/test/cctest/interpreter/bytecode_expectations/ArrayLiterals.golden',
+    'deps/v8/test/cctest/interpreter/bytecode_expectations/ArrayLiteralsWide.golden',
+    'deps/v8/test/cctest/interpreter/bytecode_expectations/AssignmentsInBinaryExpression.golden',
+    'deps/v8/test/cctest/interpreter/bytecode_expectations/BasicBlockToBoolean.golden',
+    'deps/v8/test/cctest/interpreter/bytecode_expectations/BasicLoops.golden'
+  ])
+
+  t.same(labels, ['v8'])
+
+  t.end()
+})
+
 tap.test('label: "JS sub-systems when less than 5 sub-systems have changed', (t) => {
   const labels = nodeLabels.resolveLabels([
     'lib/assert.js',