Configure tests to pull Docker images from Docker Hub automatically

Co-authored-by: StuartFerguson <16325469+StuartFerguson@users.noreply.github.com>

Author: Copilot

EstateManagementUI.PlaywrightTests/Infrastructure/DockerHelper.cs

@@ -1,5 +1,6 @@
 using DotNet.Testcontainers.Builders;
 using DotNet.Testcontainers.Containers;
+using DotNet.Testcontainers.Images;
 using DotNet.Testcontainers.Networks;
 using System.Runtime.InteropServices;
 
@@ -70,7 +71,8 @@ private async Task StartSecurityServiceContainer()
 
         _securityServiceContainer = new ContainerBuilder()
             .WithName(_securityServiceContainerName)
-            .WithImage("securityservice:latest")
+            .WithImage("stuartferguson/securityservice:latest")
+            .WithImagePullPolicy(PullPolicy.Always)
             .WithEnvironment(environmentVariables)
             .WithNetwork(_network)
             .WithPortBinding(5001, true)
@@ -102,7 +104,8 @@ private async Task StartEstateManagementUiContainer()
 
         _estateManagementUiContainer = new ContainerBuilder()
             .WithName(_estateManagementUiContainerName)
-            .WithImage("estatemanagementui:latest")
+            .WithImage("stuartferguson/estatemanagementui:latest")
+            .WithImagePullPolicy(PullPolicy.Always)
             .WithEnvironment(environmentVariables)
             .WithNetwork(_network)
             .WithPortBinding(5004, true)

EstateManagementUI.PlaywrightTests/README.md

@@ -20,12 +20,12 @@ The tests use:
 
 ## Required Docker Images
 
-The tests require the following Docker images to be available:
+The tests automatically pull the required Docker images from Docker Hub:
 
-1. **securityservice:latest** - The authorization/identity server
-2. **estatemanagementui:latest** - The Estate Management UI application
+1. **stuartferguson/securityservice:latest** - The authorization/identity server
+2. **stuartferguson/estatemanagementui:latest** - The Estate Management UI application
 
-You need to build these images before running the tests. See the main project's integration tests or build scripts for how to create these images.
+These images are automatically pulled when you run the tests for the first time. No manual Docker image building is required.
 
 ## Setup
 
@@ -46,21 +46,10 @@ dotnet build
 playwright install chromium
 ```
 
-### 2. Ensure Docker Images are Available
-
-Build or pull the required Docker images:
-
-```bash
-# Build Estate Management UI image (from the solution root)
-docker build -t estatemanagementui:latest -f EstateManagementUI.BlazorServer/Dockerfile .
-
-# The Security Service image should be available from the existing integration tests
-# Check if it exists:
-docker images | grep securityservice
-```
-
 ## Running the Tests
 
+The Docker images will be automatically pulled from Docker Hub on the first test run.
+
 ### Run All Tests
 
 ```bash

IMPLEMENTATION_SUMMARY.md

@@ -131,9 +131,9 @@ The implementation follows the pattern of the existing integration tests but use
 ### Prerequisites
 1. .NET 10.0 SDK installed
 2. Docker installed and running
-3. Required Docker images built:
-   - `securityservice:latest`
-   - `estatemanagementui:latest`
+3. Required Docker images (automatically pulled from Docker Hub):
+   - `stuartferguson/securityservice:latest`
+   - `stuartferguson/estatemanagementui:latest`
 
 ### Quick Start
 ```bash
@@ -163,10 +163,11 @@ dotnet test
 These can be modified in the test class constants if needed.
 
 ### Container Configuration
-Containers use environment variables to configure:
+Containers are automatically pulled from Docker Hub and use environment variables to configure:
 - Security Service runs on internal port 5001
 - Estate Management UI runs on internal port 5004
 - Both get dynamically assigned external ports
+- Images: `stuartferguson/securityservice:latest` and `stuartferguson/estatemanagementui:latest`
 
 ### Browser Options
 - Headless mode in CI (when `CI=true` or `IsCI=true`)
@@ -177,10 +178,9 @@ Containers use environment variables to configure:
 ## Limitations and Notes
 
 ### Current Limitations
-1. **Requires Docker images** - Tests need pre-built images
-2. **No backend services** - Only tests UI and authentication
-3. **Mock data** - Uses in-memory databases, no real data
-4. **Limited coverage** - Focuses on login and dashboard only
+1. **No backend services** - Only tests UI and authentication
+2. **Mock data** - Uses in-memory databases, no real data
+3. **Limited coverage** - Focuses on login and dashboard only
 
 ### Future Enhancements
 As noted in the README, potential improvements include:

setup-playwright-tests.sh

@@ -33,33 +33,11 @@ fi
 echo "✓ Docker found and running"
 echo ""
 
-# Check for required Docker images
-echo "Checking for required Docker images..."
-
-if docker images | grep -q "securityservice.*latest"; then
-    echo "✓ securityservice:latest found"
-else
-    echo "✗ securityservice:latest NOT found"
-    echo "  You need to build this image from the existing integration tests."
-    MISSING_IMAGES=true
-fi
-
-if docker images | grep -q "estatemanagementui.*latest"; then
-    echo "✓ estatemanagementui:latest found"
-else
-    echo "✗ estatemanagementui:latest NOT found"
-    echo "  You can build this image by running:"
-    echo "  docker build -t estatemanagementui:latest -f EstateManagementUI.BlazorServer/Dockerfile ."
-    MISSING_IMAGES=true
-fi
-
-if [ "$MISSING_IMAGES" = true ]; then
-    echo ""
-    echo "WARNING: Some required Docker images are missing."
-    echo "Tests will fail without these images."
-    echo "Please build the required images before running tests."
-    echo ""
-fi
+# Docker images will be pulled automatically from Docker Hub
+echo "Note: Docker images will be automatically pulled from Docker Hub when tests run."
+echo "  - stuartferguson/securityservice:latest"
+echo "  - stuartferguson/estatemanagementui:latest"
+echo ""
 
 # Restore NuGet packages
 echo "Restoring NuGet packages..."