Make Artifact::setFileTags() a little smarter

There's no general need for wholesale removal from the artifacts map,
followed by (re-)adding. Instead, do just what is necessary.

Change-Id: I5d4aa3c456d91bde94a8c77fb85f4e28ad5742d0
Reviewed-by: Joerg Bornemann <joerg.bornemann@qt.io>

1 files changed, 16 insertions, 0 deletions

diff --git a/tests/auto/blackbox/testdata/source-artifact-changes/source-artifact-changes.qbs b/tests/auto/blackbox/testdata/source-artifact-changes/source-artifact-changes.qbs
index 52fde9ea7..de56376df 100644
--- a/tests/auto/blackbox/testdata/source-artifact-changes/source-artifact-changes.qbs
+++ b/tests/auto/blackbox/testdata/source-artifact-changes/source-artifact-changes.qbs
@@ -36,5 +36,21 @@ CppApplication {
         }
     }
 
+    Rule {
+        multiplex: true
+        inputs: "cpp"
+        requiresInputs: false
+        Artifact { filePath: "dummy2"; fileTags: "dummy" }
+        prepare: {
+            var cmd = new JavaScriptCommand();
+            cmd.silent = true;
+            cmd.sourceCode = function() {
+                console.info("cpp artifacts: "
+                             + (product.artifacts.cpp ? product.artifacts.cpp.length : 0))
+            };
+            return cmd;
+        }
+    }
+
     Depends { name: "module_with_files" }
 }