Clang: Update ModifiedTimeChecker in SymbolIndexer

If a watched file changed we should update the ModifiedTimeChecker too.

Change-Id: Ie43f5cf5b6dd4ddb1383168a1326add21f6e3e9d
Reviewed-by: Tim Jenssen <tim.jenssen@qt.io>

src/libs/clangsupport/modifiedtimechecker.h

@@ -25,6 +25,7 @@
 
 #pragma once
 
+#include "clangpathwatcher.h"
 #include "filepathcachinginterface.h"
 #include "modifiedtimecheckerinterface.h"
 
@@ -54,7 +55,7 @@ public:
         return compareEntries(sourceEntries);
     }
 
-    void pathsChanged(const FilePathIds &filePathIds)
+    void pathsChanged(const FilePathIds &filePathIds) override
     {
         using SourceTimeStampReferences = std::vector<std::reference_wrapper<SourceTimeStamp>>;
 

src/libs/clangsupport/modifiedtimecheckerinterface.h

@@ -38,6 +38,7 @@ public:
     ModifiedTimeCheckerInterface &operator=(const ModifiedTimeCheckerInterface &) = delete;
 
     virtual bool isUpToDate(const SourceEntries &sourceEntries) const = 0;
+    virtual void pathsChanged(const FilePathIds &filePathIds) = 0;
 
 protected:
     ~ModifiedTimeCheckerInterface() = default;

src/tools/clangrefactoringbackend/source/symbolindexer.cpp

@@ -87,8 +87,8 @@ SymbolIndexer::SymbolIndexer(SymbolIndexerTaskQueueInterface &symbolIndexerTaskQ
 
 void SymbolIndexer::updateProjectParts(ProjectPartContainers &&projectParts)
 {
-        for (ProjectPartContainer &projectPart : projectParts)
-            updateProjectPart(std::move(projectPart));
+    for (ProjectPartContainer &projectPart : projectParts)
+        updateProjectPart(std::move(projectPart));
 }
 
 void SymbolIndexer::updateProjectPart(ProjectPartContainer &&projectPart)
@@ -154,6 +154,8 @@ void SymbolIndexer::pathsWithIdsChanged(const ProjectPartIds &) {}
 
 void SymbolIndexer::pathsChanged(const FilePathIds &filePathIds)
 {
+    m_modifiedTimeChecker.pathsChanged(filePathIds);
+
     FilePathIds dependentSourcePathIds = m_symbolStorage.fetchDependentSourceIds(filePathIds);
 
     std::vector<SymbolIndexerTask> symbolIndexerTask;

src/tools/clangrefactoringbackend/source/symbolindexer.h

@@ -29,8 +29,8 @@
 #include "symbolindexertaskqueueinterface.h"
 #include "symbolstorageinterface.h"
 #include "builddependenciesstorageinterface.h"
-#include "clangpathwatcher.h"
 
+#include <clangpathwatcher.h>
 #include <filecontainerv2.h>
 #include <modifiedtimecheckerinterface.h>
 #include <precompiledheaderstorageinterface.h>

tests/unit/unittest/mockmodifiedtimechecker.h

@@ -35,6 +35,7 @@ class MockSourceEntriesModifiedTimeChecker
 public:
     MOCK_CONST_METHOD1(isUpToDate,
                        bool (const ClangBackEnd::SourceEntries &sourceEntries));
+    MOCK_METHOD1(pathsChanged, void(const ClangBackEnd::FilePathIds &filePathIds));
 };
 
 class MockSourceTimeStampsModifiedTimeChecker
@@ -42,4 +43,5 @@ class MockSourceTimeStampsModifiedTimeChecker
 {
 public:
     MOCK_CONST_METHOD1(isUpToDate, bool(const ClangBackEnd::SourceTimeStamps &sourceTimeStamps));
+    MOCK_METHOD1(pathsChanged, void(const ClangBackEnd::FilePathIds &filePathIds));
 };

tests/unit/unittest/symbolindexer-test.cpp

@@ -1667,6 +1667,15 @@ TEST_F(SymbolIndexer, PathsChangedUpdatesFileStatusCache)
     ASSERT_THAT(fileStatusCache.lastModifiedTime(sourceId), 65);
 }
 
+TEST_F(SymbolIndexer, PathsChangedCallsModifiedTimeChecker)
+{
+    auto sourceId = filePathId(TESTDATA_DIR "/symbolindexer_pathChanged.cpp");
+
+    EXPECT_CALL(mockModifiedTimeChecker, pathsChanged(ElementsAre(sourceId)));
+
+    indexer.pathsChanged({sourceId});
+}
+
 TEST_F(SymbolIndexer, GetUpdatableFilePathIdsIfCompilerMacrosAreDifferent)
 {
     ON_CALL(mockProjectPartsStorage, fetchProjectPartArtefact(A<ProjectPartId>()))