Adopt classic spring-boot-starter-web / -test; harden seed.sh with set -e

- pom.xml: use spring-boot-starter-web and the aggregate spring-boot-starter-test
  instead of the 4.x modular starters. Both resolve on 4.1.0 (verified), but the
  classic starters are the conventional choice for a sample; -web still brings
  embedded Tomcat (tomcat-embed-core 11.0.22) + spring-webmvc.
- seed.sh: enable 'set -e' so a failure in any phase (not just create()) aborts
  loudly. Hardened id_of to 'grep -m1' (no downstream head -> no SIGPIPE) so the
  pipeline stays clean under -e + pipefail, and guarded the id extraction with
  '|| true' to preserve the friendly no-id error.

Signed-off-by: dhananjay6561 <dhananjayaggarwal6561@gmail.com>

Author: dhananjay6561

spring-boot-product-catalog/pom.xml

@@ -31,7 +31,7 @@
 		</dependency>
 		<dependency>
 			<groupId>org.springframework.boot</groupId>
-			<artifactId>spring-boot-starter-webmvc</artifactId>
+			<artifactId>spring-boot-starter-web</artifactId>
 		</dependency>
 
 		<dependency>
@@ -41,22 +41,7 @@
 		</dependency>
 		<dependency>
 			<groupId>org.springframework.boot</groupId>
-			<artifactId>spring-boot-starter-actuator-test</artifactId>
-			<scope>test</scope>
-		</dependency>
-		<dependency>
-			<groupId>org.springframework.boot</groupId>
-			<artifactId>spring-boot-starter-data-jpa-test</artifactId>
-			<scope>test</scope>
-		</dependency>
-		<dependency>
-			<groupId>org.springframework.boot</groupId>
-			<artifactId>spring-boot-starter-validation-test</artifactId>
-			<scope>test</scope>
-		</dependency>
-		<dependency>
-			<groupId>org.springframework.boot</groupId>
-			<artifactId>spring-boot-starter-webmvc-test</artifactId>
+			<artifactId>spring-boot-starter-test</artifactId>
 			<scope>test</scope>
 		</dependency>
 	</dependencies>

spring-boot-product-catalog/seed.sh

@@ -5,12 +5,14 @@
 # Keploy suite is broad: ~50 test cases covering the full CRUD lifecycle across several
 # categories, filtering, and a wide range of 400 (validation) and 404 (not-found) paths.
 # Ids returned by POST are chained into later GET/PUT/DELETE calls so the suite is coherent.
-set -uo pipefail
+set -euo pipefail
 
 BASE="${BASE:-http://localhost:8080}"
 
 # --- helpers -----------------------------------------------------------------
-id_of() { grep -o '"id":[0-9]*' | head -1 | cut -d: -f2; }
+# grep -m1 stops after the first match and exits 0 (no SIGPIPE from a downstream `head`),
+# so it stays well-behaved under `set -e` + `pipefail`.
+id_of() { grep -m1 -o '"id":[0-9]*' | cut -d: -f2; }
 
 CREATED=()   # ids of products created, in order
 
@@ -22,7 +24,7 @@ create() {
     exit 1
   fi
   echo "  created: $body"
-  id=$(printf '%s' "$body" | id_of)
+  id=$(printf '%s' "$body" | id_of || true)
   if [ -z "$id" ]; then
     echo "  ERROR: no id in create response: $body" >&2
     exit 1