Angular version update

.github/workflows/mutation.yml

@@ -28,7 +28,7 @@ jobs:
     - name: Set up Node
       uses: actions/setup-node@1d0ff469b7ec7b3cb9d8673fde0c81c44821de2a # v4.2.0
       with:
-        node-version: '14.15.1'
+        node-version: '18.19.1'
         cache: 'npm'
         cache-dependency-path: report/report-ng/package-lock.json
 

.github/workflows/release.yml

@@ -22,7 +22,7 @@ jobs:
     - name: Set up Node
       uses: actions/setup-node@1d0ff469b7ec7b3cb9d8673fde0c81c44821de2a # v4.2.0
       with:
-        node-version: '14.15.1'
+        node-version: '18.19.1'
         cache: 'npm'
         cache-dependency-path: report/report-ng/package-lock.json
 

.github/workflows/sonar.yml

@@ -29,7 +29,7 @@ jobs:
       - name: Set up Node
         uses: actions/setup-node@1d0ff469b7ec7b3cb9d8673fde0c81c44821de2a # v4.2.0
         with:
-          node-version: '14.15.1'
+          node-version: '18.19.1'
           cache: 'npm'
           cache-dependency-path: report/report-ng/package-lock.json
 

.github/workflows/static_artifacts.yml

@@ -22,24 +22,24 @@ jobs:
     - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
 
     - name: Set up JDK 17
-      uses: actions/setup-java@3a4f6e1af504cf6a31855fa899c6aa5355ba6c12 # v4.7.0
+      uses: actions/setup-java@3a4f6e1af504cf6a31855fa899c6aa5355ba6c12 # v4.7.0
       with:
         java-version: '17'
         distribution: 'temurin'
         cache: maven
 
     - name: Set up Node
-      uses: actions/setup-node@1d0ff469b7ec7b3cb9d8673fde0c81c44821de2a # v4.2.0
+      uses: actions/setup-node@1d0ff469b7ec7b3cb9d8673fde0c81c44821de2a # v4.2.0
       with:
-        node-version: '14.15.1'
+        node-version: '18.19.1'
         cache: 'npm'
         cache-dependency-path: report/report-ng/package-lock.json
 
     - name: Build with Maven
       run: mvn -B -Dnode=system package
 
     - name: Save flow execution reports
-      uses: actions/upload-artifact@4cec3d8aa04e39d1a68397de0c4cd6fb9dce8ec1 # v4.6.1
+      uses: actions/upload-artifact@4cec3d8aa04e39d1a68397de0c4cd6fb9dce8ec1 # v4.6.1
       id: upload_flow
       with:
         name: flow_execution_reports
@@ -62,12 +62,12 @@ jobs:
       run: rm -rf static
 
     - name: Download flow reports
-      uses: actions/download-artifact@cc203385981b70ca67e1cc392babf9cc229d5806 # v4.1.9
+      uses: actions/download-artifact@cc203385981b70ca67e1cc392babf9cc229d5806 # v4.1.9
       with:
         name: flow_execution_reports
         path: static
 
     - name: Commit
-      uses: stefanzweifel/git-auto-commit-action@e348103e9026cc0eee72ae06630dbe30c8bf7a79 # v5.1.0
+      uses: stefanzweifel/git-auto-commit-action@e348103e9026cc0eee72ae06630dbe30c8bf7a79 # v5.1.0
       with:
         branch: slim_pages

.github/workflows/test.yml

@@ -26,7 +26,7 @@ jobs:
     - name: Set up Node
       uses: actions/setup-node@1d0ff469b7ec7b3cb9d8673fde0c81c44821de2a # v4.2.0
       with:
-        node-version: '14.15.1'
+        node-version: '18.19.1'
         cache: 'npm'
         cache-dependency-path: report/report-ng/package-lock.json
 

doc/src/test/java/com/mastercard/test/flow/doc/IconEmbedTest.java

@@ -28,7 +28,7 @@ class IconEmbedTest {
 	private static final String ICON_SERVICE_NAME = "icon-embed.service.ts";
 
 	private static final Pattern ICON_SERVICE_INJECTION_PATTERN = Pattern.compile(
-			"(\\S*?): IconEmbedService" );
+			"(\\S*?) = inject\\(IconEmbedService\\);" );
 
 	/**
 	 * Matches SVG definitions out of {@value #ICON_SERVICE_NAME}, captures the icon
@@ -79,7 +79,7 @@ Stream<DynamicTest> icons() {
 													"Undefined icon '%s' in mat-icon invocation '%s'",
 													icon, usage.group() ) );
 											assertTrue( registered.contains( icon ), String.format(
-													"Uregistered icon '%s' in mat-icon invocation '%s'",
+													"Unregistered icon '%s' in mat-icon invocation '%s'",
 													icon, usage.group() ) );
 										}
 									}

example/app-assert/pom.xml

@@ -33,24 +33,6 @@
 			<version>${project.version}</version>
 		</dependency>
 
-		<dependency>
-			<!-- browser interaction driver management -->
-			<groupId>io.github.bonigarcia</groupId>
-			<artifactId>webdrivermanager</artifactId>
-			<exclusions>
-				<exclusion>
-					<!-- This conflicts with the newer version that selenium needs -->
-					<groupId>com.google.guava</groupId>
-					<artifactId>guava</artifactId>
-				</exclusion>
-				<exclusion>
-					<!-- Avoid incompatible slf4j -->
-					<groupId>org.slf4j</groupId>
-					<artifactId>slf4j-api</artifactId>
-				</exclusion>
-			</exclusions>
-		</dependency>
-
 		<dependency>
 			<!-- embedded database -->
 			<groupId>com.h2database</groupId>

example/app-assert/src/main/java/com/mastercard/test/flow/example/app/assrt/Browser.java

@@ -15,8 +15,6 @@
 import org.openqa.selenium.chrome.ChromeDriver;
 import org.openqa.selenium.chrome.ChromeOptions;
 
-import io.github.bonigarcia.wdm.WebDriverManager;
-
 /**
  * Handles the lifecycle of the chrome instance we use to test web UI. Add this
  * class as an {@link ExtendWith} annotation on your test class that uses the
@@ -69,7 +67,6 @@ public static WebDriver get() {
 						+ "Only call Browser.get() in test classes with @ExtendWith(Browser.class) annotation" );
 			}
 
-			WebDriverManager.chromedriver().setup();
 			ChromeOptions options = new ChromeOptions();
 			if( !SHOW ) {
 				options.addArguments( "--headless=new" );

pom.xml

@@ -128,14 +128,6 @@
 				<version>4.13.0</version>
 			</dependency>
 
-			<dependency>
-				<!-- We use a headless chrome instance to test the report, this 
-			     provides automatic management of the drivers for that -->
-				<groupId>io.github.bonigarcia</groupId>
-				<artifactId>webdrivermanager</artifactId>
-				<version>6.0.1</version>
-			</dependency>
-
 			<dependency>
 				<!-- text diffs -->
 				<groupId>io.github.java-diff-utils</groupId>

report/report-core/pom.xml

@@ -71,21 +71,6 @@
 			</exclusions>
 		</dependency>
 
-		<dependency>
-			<!-- We use a headless chrome instance to test the report, this 
-			     provides automatic management of the drivers for that -->
-			<groupId>io.github.bonigarcia</groupId>
-			<artifactId>webdrivermanager</artifactId>
-			<scope>test</scope>
-			<exclusions>
-				<exclusion>
-					<!-- Avoid incompatible slf4j -->
-					<groupId>org.slf4j</groupId>
-					<artifactId>slf4j-api</artifactId>
-				</exclusion>
-			</exclusions>
-		</dependency>
-
 		<dependency>
 			<!-- This is how we drive the browser when testing the report -->
 			<groupId>org.seleniumhq.selenium</groupId>

report/report-core/src/test/java/com/mastercard/test/flow/report/WriterTest.java

@@ -258,9 +258,9 @@ void chunkLoadingPath() throws Exception {
 						+ delta.getTarget().getLines().get( 0 ).length() )
 				.replaceAll( "\\d", "#" );
 
-		assertEquals( "),[])),a.u=e=>(###===e?\"comm",
+		assertEquals( "),[])),a.u=e=>(##===e?\"commo",
 				before, "raw resource runtime snippet" );
-		assertEquals( "),[])),a.u=e=>\"res/\"+(###===e?\"comm",
+		assertEquals( "),[])),a.u=e=>\"res/\"+(##===e?\"commo",
 				after, "written runtime snippet" );
 	}
 

report/report-core/src/test/java/com/mastercard/test/flow/report/seq/AbstractSequence.java

@@ -10,6 +10,7 @@
 import java.util.List;
 import java.util.Map;
 import java.util.TreeMap;
+import java.util.Objects;
 import java.util.concurrent.TimeUnit;
 import java.util.function.Function;
 import java.util.stream.Stream;
@@ -237,8 +238,8 @@ protected <T extends AbstractSequence<T>> T navTo( String navItem,
 		driver.findElement( By.id( "menu_trigger" ) ).click();
 
 		Function<WebDriver, Stream<WebElement>> menuItems = dr -> dr
-				.findElement( By.className( "mat-menu-panel" ) )
-				.findElements( By.className( "mat-menu-item" ) )
+				.findElement( By.className( "mat-mdc-menu-panel" ) )
+				.findElements( By.className( "mat-mdc-menu-item" ) )
 				.stream();
 
 		new WebDriverWait( driver, Duration.ofSeconds( 1 ) )
@@ -266,7 +267,7 @@ protected <T extends AbstractSequence<T>> T navTo( String navItem,
 	 */
 	public <T extends AbstractSequence<T>> T tab( String name,
 			Function<AbstractSequence<S>, T> seqConstructor ) {
-		List<WebElement> tabs = driver.findElements( By.className( "mat-tab-label-content" ) );
+		List<WebElement> tabs = driver.findElements( By.className( "mdc-tab__text-label" ) );
 		tabs.stream()
 				.filter( e -> e.getText().startsWith( name ) )
 				.findFirst()
@@ -298,11 +299,13 @@ protected void waitForTabTransition() {
 		// wait for the tab transition to complete
 		new WebDriverWait( driver, Duration.ofSeconds( 3 ) )
 				.withMessage( () -> "Failed to find a relaxed tab body in:\n  " + driver
-						.findElements( By.className( "mat-tab-body-content" ) ).stream()
+						.findElements( By.className( "mat-mdc-tab-body-content" ) ).stream()
 						.map( e -> e.getAttribute( "style" ) )
 						.collect( joining( "\n  " ) ) )
-				.until( dr -> dr.findElements( By.className( "mat-tab-body-content" ) ).stream()
-						.anyMatch( e -> "transform: none;".equals( e.getAttribute( "style" ) ) ) );
+				.until( dr -> dr.findElements( By.className( "mat-mdc-tab-body-content" ) ).stream()
+						.map( e -> e.getAttribute( "style" ) )
+						.filter( Objects::nonNull )
+						.anyMatch( style -> style.contains( "transform: none;" ) ) );
 	}
 
 	/**

report/report-core/src/test/java/com/mastercard/test/flow/report/seq/Browser.java

@@ -24,8 +24,6 @@
 import org.openqa.selenium.remote.RemoteWebDriver;
 import org.slf4j.LoggerFactory;
 
-import io.github.bonigarcia.wdm.WebDriverManager;
-
 /**
  * Handles the lifecycle of the chrome instance we use to test the report. Add
  * this class as an {@link ExtendWith} annotation on your test class that uses
@@ -150,7 +148,6 @@ private enum Driver {
 		CHROME {
 			@Override
 			public WebDriver get() {
-				WebDriverManager.chromedriver().setup();
 				ChromeOptions options = new ChromeOptions();
 				if( !SHOW ) {
 					options.addArguments( "--headless=new" );
@@ -183,8 +180,6 @@ public WebDriver get() {
 		FIREFOX {
 			@Override
 			public WebDriver get() {
-				WebDriverManager.firefoxdriver().setup();
-
 				FirefoxOptions options = new FirefoxOptions();
 
 				if( !SHOW ) {

report/report-core/src/test/java/com/mastercard/test/flow/report/seq/DiffSequence.java

@@ -18,6 +18,7 @@
 import org.junit.jupiter.api.Assertions;
 import org.openqa.selenium.By;
 import org.openqa.selenium.WebElement;
+import org.openqa.selenium.support.ui.ExpectedCondition;
 import org.openqa.selenium.support.ui.WebDriverWait;
 
 import com.mastercard.test.flow.report.Copy;
@@ -280,6 +281,8 @@ private DiffSequence drag( String id, String from, String to ) {
 				.moveByOffset( 0, 5 ) // shrug
 				.moveToElement( items.get( to ) )
 				.release()
+				.moveByOffset( 0, 5 ) // shrug
+				.release()
 				.pause( 400 )
 				.build()
 				.perform();
@@ -505,9 +508,11 @@ public DiffSequence analysisDiffs( String... expected ) {
 
 		// wait for expansion to finish
 		new WebDriverWait( driver, ofSeconds( 2 ) )
-				.until( dr -> "visibility: visible;"
-						.equals( changed.findElement( By.className( "mat-expansion-panel-content" ) )
-								.getAttribute( "style" ) ) );
+				.until( (ExpectedCondition<Boolean>) dr -> {
+					List<WebElement> headers = changed
+							.findElements( By.className( "diff_display" ) );
+					return headers.stream().allMatch( WebElement::isDisplayed );
+				} );
 
 		WebElement changes = changed.findElement( By.id( "changes" ) );
 
@@ -539,11 +544,13 @@ public DiffSequence expandAnalysis( String diff ) {
 				.orElseThrow( () -> new IllegalStateException( "" ) );
 		diffPanel.findElement( By.tagName( "mat-expansion-panel-header" ) ).click();
 
-		// wait for expansion to finish
+		// wait for changes to load
 		new WebDriverWait( driver, ofSeconds( 2 ) )
-				.until( dr -> "visibility: visible;"
-						.equals( diffPanel.findElement( By.className( "mat-expansion-panel-content" ) )
-								.getAttribute( "style" ) ) );
+				.until( (ExpectedCondition<Boolean>) dr -> {
+					List<WebElement> headers = diffPanel
+							.findElements( By.className( "mat-mdc-list-item" ) );
+					return headers.stream().allMatch( WebElement::isDisplayed );
+				} );
 
 		return this;
 	}

report/report-core/src/test/java/com/mastercard/test/flow/report/seq/IndexSequence.java

@@ -548,15 +548,15 @@ private static String summariseSVG( String svg ) {
 
 	private IndexSequence dragChip( String text, String sourceId, String destId ) {
 		WebElement source = driver.findElement( By.id( sourceId ) );
-		WebElement chip = source.findElements( By.tagName( "mat-chip" ) ).stream()
+		WebElement chip = source.findElements( By.tagName( "mat-chip-row" ) ).stream()
 				.filter( c -> text.equals( c.findElement( By.className( "tag_text" ) ).getText() ) )
 				.findFirst()
 				.orElseThrow( () -> new AssertionError( ""
 						+ "Couldn't find chip with text '" + text + "'\n"
 						+ "Have you called expandFilters()?" ) );
 		WebElement destination = driver
 				.findElement( By.id( destId ) )
-				.findElement( By.tagName( "mat-chip-list" ) );
+				.findElement( By.tagName( "mat-chip-grid" ) );
 		actions.moveToElement( chip )
 				.clickAndHold( chip )
 				.moveToElement( destination )

report/report-core/src/test/java/com/mastercard/test/flow/report/seq/LogSequence.java

@@ -63,7 +63,7 @@ public LogSequence levels( String... levels ) {
 
 		List<WebElement> fields = driver.findElements( By.tagName( "mat-form-field" ) );
 		WebElement select = fields.stream()
-				.filter( e -> e.getText().endsWith( "Levels" ) )
+				.filter( e -> e.getText().contains( "Levels" ) )
 				.findFirst()
 				.orElse( null );
 
@@ -138,7 +138,7 @@ public LogSequence hasMessages( String... lines ) {
 		trace( "hasMessages", (Object[]) lines );
 		Assertions.assertEquals(
 				Copy.pasta( lines ),
-				Copy.pasta( driver.findElements( By.className( "mat-row" ) ).stream()
+				Copy.pasta( driver.findElements( By.className( "mat-mdc-row" ) ).stream()
 						.map( r -> r.getText().replace( '\n', ' ' ) ) ) );
 		return this;
 	}

report/report-ng/angular.json

@@ -69,18 +69,18 @@
           "builder": "@angular-devkit/build-angular:dev-server",
           "configurations": {
             "production": {
-              "browserTarget": "report:build:production"
+              "buildTarget": "report:build:production"
             },
             "development": {
-              "browserTarget": "report:build:development"
+              "buildTarget": "report:build:development"
             }
           },
           "defaultConfiguration": "development"
         },
         "extract-i18n": {
           "builder": "@angular-devkit/build-angular:extract-i18n",
           "options": {
-            "browserTarget": "report:build"
+            "buildTarget": "report:build"
           }
         },
         "test": {