Inventory customers tests

[malekelkssas]

No description provided.

[Copilot]

Pull Request Overview

This PR introduces extensive testing for analytics services and controllers, and updates repository interfaces for consistency.

• Added unit tests for inventory and customer analytics services
• Added integration tests for inventory and customer reporting controllers
• Updated repository signatures: changed ProductSnapshot ID type and modified the join strategy in the top spenders query

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
src/test/java/com/Podzilla/analytics/services/InventoryReportServiceTest.java	Added unit tests for InventoryAnalyticsService
src/test/java/com/Podzilla/analytics/services/CustomerReportServiceTest.java	Added unit tests for CustomerAnalyticsService
src/test/java/com/Podzilla/analytics/controllers/InventoryReportControllerTest.java	Added integration tests for inventory REST endpoints
src/test/java/com/Podzilla/analytics/controllers/CustomerReportControllerTest.java	Added integration tests for customer REST endpoints
src/main/java/com/Podzilla/analytics/repositories/ProductSnapshotRepository.java	Updated JpaRepository ID type from Long to UUID
src/main/java/com/Podzilla/analytics/repositories/CustomerRepository.java	Changed order join from LEFT JOIN to INNER JOIN in top spenders query

Comments suppressed due to low confidence (2)

src/test/java/com/Podzilla/analytics/services/InventoryReportServiceTest.java:31

• [nitpick] The test class and file name reference InventoryReportServiceTest, but it tests InventoryAnalyticsService; consider renaming both the file and class to InventoryAnalyticsServiceTest for consistency.

public class InventoryReportServiceTest {

src/test/java/com/Podzilla/analytics/services/CustomerReportServiceTest.java:32

• [nitpick] The test class and file name reference CustomerReportServiceTest, but it tests CustomerAnalyticsService; consider renaming both the file and class to CustomerAnalyticsServiceTest for consistency.

public class CustomerReportServiceTest {

[Copilot]

The custom query for top spenders lacks an ORDER BY clause, which can lead to nondeterministic result ordering; add ORDER BY COALESCE(SUM(o.totalAmount), 0) DESC to ensure consistent sorting by total spending.