test(project): Add case for dependency content changes

This test should cover a scenario with an application depending on a library.
Specifically, we're directly modifying the contents of the library
which should have effects on the application because a custom task will detect it
and modify the application's resources. The application is expected to get rebuilt.

Author: maxreichmann

packages/project/test/fixtures/application.a/task.dependency-change.js

@@ -0,0 +1,36 @@
+// This is a modified version of the compileLicenseSummary example of the UI5 CLI.
+// (https://github.com/UI5/cli/blob/b72919469d856508dd757ecf325a5fb45f15e56d/internal/documentation/docs/pages/extensibility/CustomTasks.md#example-libtaskscompilelicensesummaryjs)
+
+module.exports = async function ({dependencies, log, taskUtil, workspace, options: {projectNamespace}}) {
+	const {createResource} = taskUtil.resourceFactory;
+    const projectsVisited = new Set();
+
+    async function processProject(project) {
+		return Promise.all(taskUtil.getDependencies().map(async (projectName) => {
+            if (projectName !== "library.d") {
+				return;
+			}
+			if (projectsVisited.has(projectName)) {
+                return;
+            }
+            projectsVisited.add(projectName);
+            const project = taskUtil.getProject(projectName);
+			const newLibraryFile = await project.getReader().byGlob("**/newLibraryFile.js");
+			if (newLibraryFile.length > 0) {
+				console.log('New Library file found. We are in #4 build.');
+				// Change content of application.a:
+				const applicationResource = await workspace.byPath("/resources/id1/test.js");
+				const content = (await applicationResource.getString()) + "\n console.log('something new');";
+				await workspace.write(createResource({
+        			path: "/test.js",
+        			string: content
+    			}));
+            } else {
+				console.log(`New Library file not found. We are still in an earlier build.`);
+            }
+            return processProject(project);
+        }));
+    }
+    // Start processing dependencies of the root project
+    await processProject(taskUtil.getProject());
+};

packages/project/test/fixtures/application.a/ui5-customTask-dependency-change.yaml

@@ -0,0 +1,18 @@
+---
+specVersion: "2.3"
+type: application
+metadata:
+  name: application.a
+builder:  
+  customTasks:
+    - name: dependency-change
+      afterTask: minify
+---
+specVersion: "5.0"
+kind: extension
+type: task
+metadata:
+  name: dependency-change
+task:
+  path: task.dependency-change.js
+

packages/project/test/lib/build/ProjectBuilder.integration.js

@@ -361,6 +361,90 @@ test.serial.skip("Build application.a (multiple custom tasks 2)", async (t) => {
 	await t.throwsAsync(fs.readFile(`${destPath}/test.js`, {encoding: "utf8"}));
 });
 
+test.only("Build application.a (dependency content changes)", async (t) => {
+	const fixtureTester = new FixtureTester(t, "application.a");
+	const destPath = fixtureTester.destPath;
+
+	// This test should cover a scenario with an application depending on a library.
+	// Specifically, we're directly modifying the contents of the library
+	// which should have effects on the application because a custom task will detect it
+	// and modify the application's resources. The application is expected to get rebuilt.
+
+	// #1 build (no cache, no changes, no dependencies)
+	await fixtureTester.buildProject({
+		graphConfig: {rootConfigPath: "ui5-customTask-dependency-change.yaml"},
+		config: {destPath, cleanDest: true},
+		assertions: {
+			projects: {
+				"application.a": {}
+			}
+		}
+	});
+
+
+	// #2 build (with cache, no changes, no dependencies)
+	await fixtureTester.buildProject({
+		graphConfig: {rootConfigPath: "ui5-customTask-dependency-change.yaml"},
+		config: {destPath, cleanDest: true},
+		assertions: {
+			projects: {}
+		}
+	});
+
+
+	// Change content of library.d (this will not affect application.a):
+	const someJsOfLibrary = `${fixtureTester.fixturePath}/node_modules/library.d/main/src/library/d/some.js`;
+	await fs.appendFile(someJsOfLibrary, `\ntest("line added");\n`);
+
+	// #3 build (with cache, with changes, with dependencies)
+	await fixtureTester.buildProject({
+		graphConfig: {rootConfigPath: "ui5-customTask-dependency-change.yaml"},
+		config: {destPath, cleanDest: true, dependencyIncludes: {includeAllDependencies: true}},
+		assertions: {
+			projects: {
+				"library.d": {},
+				"library.a": {},
+				"library.b": {},
+				"library.c": {},
+			}
+		}
+	});
+
+	// Check if library contains correct changed content:
+	const builtFileContent = await fs.readFile(`${destPath}/resources/library/d/some.js`, {encoding: "utf8"});
+	t.true(builtFileContent.includes(`test("line added");`), "Build dest contains changed file content");
+
+
+	// Change content of library.d again (this time it affects application.a):
+	await fs.writeFile(`${fixtureTester.fixturePath}/node_modules/library.d/main/src/library/d/newLibraryFile.js`,
+		`console.log("SOME NEW CONTENT");`);
+
+	// #4 build (no cache, with changes, with dependencies)
+	// This build should execute the custom task "task.dependency-change.js" again which now detects "newLibraryFile.js"
+	// and modifies a resource of application.a (namely "test.js").
+	await fixtureTester.buildProject({
+		graphConfig: {rootConfigPath: "ui5-customTask-dependency-change.yaml"},
+		config: {destPath, cleanDest: true, dependencyIncludes: {includeAllDependencies: true}},
+		assertions: {
+			projects: {
+				"library.d": {},
+				"application.a": { // FIXME: currently failing (getting skipped entirely)
+					skippedTasks: [
+						"enhanceManifest",
+						"escapeNonAsciiCharacters",
+						"generateFlexChangesBundle",
+						"replaceCopyright",
+					]
+				},
+			}
+		}
+	});
+
+	// Check that application.a contains correct changed content (test.js):
+	const builtFileContent2 = await fs.readFile(`${destPath}/test.js`, {encoding: "utf8"});
+	t.true(builtFileContent2.includes(`console.log('something new');`), "Build dest contains changed file content");
+});
+
 test.serial("Build library.d project multiple times", async (t) => {
 	const fixtureTester = new FixtureTester(t, "library.d");
 	const destPath = fixtureTester.destPath;