Merge pull request #22 from Jalen-Stephens/9-service-implement-imageservice-core-logic

9 service implement imageservice core logic

Author: ikeschmack

citations.md

@@ -726,3 +726,114 @@ Guidance on finalizing Spring Security JWT resource server configuration for Sup
 > Portions of this commit or configuration were generated with assistance from OpenAI ChatGPT (GPT-5) on October 21, 2025. All AI-generated content was reviewed, verified, and finalized by the development team.
 
 ---
+
+### **Commit / Ticket Reference**
+
+* **Commit:** `feat(#9): add ImageRepository and implement ImageService with ownership and service-layer exceptions`
+* **Ticket:** `#9 — Implement ImageService core logic`
+* **Date:** October 22, 2025
+* **Team Member:** Jalen Stephens
+
+---
+
+### **AI Tool Information**
+
+* **Tool Used:** OpenAI ChatGPT (GPT-5)
+* **Access Method:** ChatGPT Web (.edu academic access)
+* **Configuration:** Default model settings
+* **Cost:** $0 (no paid API calls)
+
+---
+
+### **Purpose of AI Assistance**
+
+The AI provided guidance and implementation help for creating the `ImageRepository`, wiring it into the `ImageService`, and introducing service-layer exceptions (`NotFoundException`, `ForbiddenException`) to support ownership enforcement and clean error semantics.
+
+---
+
+### **Prompts / Interaction Summary**
+
+* Asked how to start on the ImageService ticket.
+* Shared existing schema for the `images` table.
+* Requested matching repository + service implementation.
+* Asked whether service-layer exceptions are standard practice.
+* Requested a one-line commit message referencing ticket `#9`.
+
+---
+
+### **Resulting Artifacts**
+
+* `ImageRepository.java` (new)
+* `ImageService.java` (updated core logic + ownership enforcement)
+* `ForbiddenException.java` (new)
+* `NotFoundException.java` (new)
+
+---
+
+### **Verification**
+
+* Build completed successfully using `mvn clean verify`.
+* Code reviewed manually to confirm schema alignment and method signatures.
+* Successfully wired into the service layer with no startup issues.
+
+---
+
+### **Attribution Statement**
+
+> Portions of this commit or configuration were generated with assistance from OpenAI ChatGPT (GPT-5) on October 22, 2025. All AI-generated content was reviewed, verified, and finalized by the development team.
+
+---
+
+### **Commit / Ticket Reference**
+
+* **Commit:** `test(#9): add ImageService unit tests and update pom to run under Java 17`
+* **Ticket:** `#9 — Implement ImageService core logic`
+* **Date:** October 22, 2025
+* **Team Member:** Jalen Stephens
+
+---
+
+### **AI Tool Information**
+
+* **Tool Used:** OpenAI ChatGPT (GPT-5)
+* **Access Method:** ChatGPT Web (.edu academic access)
+* **Configuration:** Default model settings
+* **Cost:** $0 (no paid API calls)
+
+---
+
+### **Purpose of AI Assistance**
+
+Assistance was used to design and implement comprehensive branch-coverage unit tests for `ImageService`, including mocking strategies, repository interaction expectations, and handling JDK/Jacoco compatibility issues for coverage instrumentation.
+
+---
+
+### **Prompts / Interaction Summary**
+
+* “Can we create unit test now on the code we just added…”
+* Debugging JaCoCo crash and version mismatch
+* Fixing Mockito inline instrumentation conflict
+* Request for one-line commit message referencing #9
+
+---
+
+### **Resulting Artifacts**
+
+* `src/test/java/dev/coms4156/project/metadetect/service/ImageServiceTest.java`
+* Updated `pom.xml` (ensuring Java 17 execution for tests/coverage)
+
+---
+
+### **Verification**
+
+* Successfully executed `mvn clean test` under Java 17
+* Verified branch coverage logic (success + failure paths)
+* Confirmed green test suite and valid JaCoCo run
+
+---
+
+### **Attribution Statement**
+
+> Portions of this commit or configuration were generated with assistance from OpenAI ChatGPT (GPT-5) on October 22, 2025. All AI-generated content was reviewed, verified, and finalized by the development team.
+
+---

pom.xml

@@ -147,12 +147,23 @@
 		</extensions>
 		<finalName>${project.artifactId}-${project.version}</finalName>
 		<plugins>
+			<!-- Spring Boot (single declaration) -->
 			<plugin>
 				<groupId>org.springframework.boot</groupId>
 				<artifactId>spring-boot-maven-plugin</artifactId>
+				<configuration>
+					<jvmArguments>--enable-native-access=ALL-UNNAMED</jvmArguments>
+				</configuration>
+				<executions>
+					<execution>
+						<goals>
+							<goal>repackage</goal>
+						</goals>
+					</execution>
+				</executions>
 			</plugin>
 
-			<!-- Checkstyle (already configured to run at validate) -->
+			<!-- Checkstyle -->
 			<plugin>
 				<groupId>org.apache.maven.plugins</groupId>
 				<artifactId>maven-checkstyle-plugin</artifactId>
@@ -168,12 +179,14 @@
 					<execution>
 						<id>validate</id>
 						<phase>validate</phase>
-						<goals><goal>check</goal></goals>
+						<goals>
+							<goal>check</goal>
+						</goals>
 					</execution>
 				</executions>
 			</plugin>
 
-			<!-- PMD static analysis -->
+			<!-- PMD -->
 			<plugin>
 				<groupId>org.apache.maven.plugins</groupId>
 				<artifactId>maven-pmd-plugin</artifactId>
@@ -185,12 +198,14 @@
 				<executions>
 					<execution>
 						<phase>verify</phase>
-						<goals><goal>check</goal></goals>
+						<goals>
+							<goal>check</goal>
+						</goals>
 					</execution>
 				</executions>
 			</plugin>
 
-			<!-- JaCoCo coverage -->
+			<!-- JaCoCo -->
 			<plugin>
 				<groupId>org.jacoco</groupId>
 				<artifactId>jacoco-maven-plugin</artifactId>
@@ -201,12 +216,9 @@
 							<goal>prepare-agent</goal>
 						</goals>
 						<configuration>
-							<!-- Instrument ONLY our code -->
 							<includes>
-								<include>dev/coms4156/project/metadetect/*</include>
 								<include>dev/coms4156/project/metadetect/**</include>
 							</includes>
-							<!-- Make sure we don't try to touch JDK or Spring classes -->
 							<excludes>
 								<exclude>java/**</exclude>
 								<exclude>jdk/**</exclude>
@@ -215,7 +227,6 @@
 							</excludes>
 						</configuration>
 					</execution>
-
 					<execution>
 						<id>report</id>
 						<phase>verify</phase>
@@ -225,13 +236,6 @@
 					</execution>
 				</executions>
 			</plugin>
-			<plugin>
-				<groupId>org.springframework.boot</groupId>
-				<artifactId>spring-boot-maven-plugin</artifactId>
-				<configuration>
-					<jvmArguments>--enable-native-access=ALL-UNNAMED</jvmArguments>
-				</configuration>
-			</plugin>
 		</plugins>
 	</build>
 </project>

src/main/java/dev/coms4156/project/metadetect/model/Image.java

@@ -0,0 +1,118 @@
+package dev.coms4156.project.metadetect.model;
+
+import java.time.Instant;
+import java.util.UUID;
+import org.springframework.data.annotation.Id;
+import org.springframework.data.relational.core.mapping.Column;
+import org.springframework.data.relational.core.mapping.Table;
+
+/**
+ * Represents an uploaded image belonging to a user.
+ * Each image stores metadata such as filename, optional labels,
+ * storage location, and its upload timestamp.
+ */
+@Table("images")
+public class Image {
+
+  @Id
+  private UUID id;
+
+  @Column("user_id")
+  private UUID userId;
+
+  @Column("filename")
+  private String filename;
+
+  @Column("storage_path")
+  private String storagePath;
+
+  @Column("labels")
+  private String[] labels;
+
+  @Column("note")
+  private String note;
+
+  @Column("uploaded_at")
+  private Instant uploadedAt;
+
+  public Image() {}
+
+  /**
+   * Constructs a new Image record.
+   *
+   * @param id          unique identifier of the image
+   * @param userId      id of the owning user
+   * @param filename    original filename from the upload
+   * @param storagePath path of the stored file (if present)
+   * @param labels      optional labels (Postgres text[])
+   * @param note        user note attached to the image
+   * @param uploadedAt  timestamp when the image was uploaded
+   */
+  public Image(UUID id, UUID userId, String filename, String storagePath,
+               String[] labels, String note, Instant uploadedAt) {
+    this.id = id;
+    this.userId = userId;
+    this.filename = filename;
+    this.storagePath = storagePath;
+    this.labels = labels;
+    this.note = note;
+    this.uploadedAt = uploadedAt;
+  }
+
+  // Getters/setters
+  public UUID getId() {
+    return id;
+  }
+
+  public void setId(UUID id) {
+    this.id = id;
+  }
+
+  public UUID getUserId() {
+    return userId;
+  }
+
+  public void setUserId(UUID userId) {
+    this.userId = userId;
+  }
+
+  public String getFilename() {
+    return filename;
+  }
+
+  public void setFilename(String filename) {
+    this.filename = filename;
+  }
+
+  public String getStoragePath() {
+    return storagePath;
+  }
+
+  public void setStoragePath(String storagePath) {
+    this.storagePath = storagePath;
+  }
+
+  public String[] getLabels() {
+    return labels;
+  }
+
+  public void setLabels(String[] labels) {
+    this.labels = labels;
+  }
+
+  public String getNote() {
+    return note;
+  }
+
+  public void setNote(String note) {
+    this.note = note;
+  }
+
+  public Instant getUploadedAt() {
+    return uploadedAt;
+  }
+
+  public void setUploadedAt(Instant uploadedAt) {
+    this.uploadedAt = uploadedAt;
+  }
+}

src/main/java/dev/coms4156/project/metadetect/repository/ImageRepository.java

@@ -0,0 +1,20 @@
+package dev.coms4156.project.metadetect.repository;
+
+import dev.coms4156.project.metadetect.model.Image;
+import java.util.List;
+import java.util.Optional;
+import java.util.UUID;
+import org.springframework.data.repository.CrudRepository;
+import org.springframework.stereotype.Repository;
+
+/**
+ * Repository for accessing and querying {@link Image} entities.
+ * Provides user-scoped lookup operations to ensure ownership checks
+ * can be enforced at the service layer.
+ */
+@Repository
+public interface ImageRepository extends CrudRepository<Image, UUID> {
+  Optional<Image> findByIdAndUserId(UUID id, UUID userId);
+
+  List<Image> findAllByUserIdOrderByUploadedAtDesc(UUID userId);
+}