Feature/3230 - Clean Architecture#3235
Conversation
PR SummaryThis PR introduces a Clean Architecture implementation for a shopping cart application. It includes entities ( Changes
autogenerated by presubmit.ai |
There was a problem hiding this comment.
✅ LGTM!
Review Summary
Files Processed (19)
- clean-architecture/README.md (1 hunk)
- clean-architecture/etc/cleanArchitectureUMLDiagram.PNG (0 hunks)
- clean-architecture/img.png (0 hunks)
- clean-architecture/pom.xml (1 hunk)
- clean-architecture/src/main/java/com.iluwatar.cleanArchitecture/App.java (1 hunk)
- clean-architecture/src/main/java/com.iluwatar.cleanArchitecture/Cart.java (1 hunk)
- clean-architecture/src/main/java/com.iluwatar.cleanArchitecture/CartController.java (1 hunk)
- clean-architecture/src/main/java/com.iluwatar.cleanArchitecture/CartRepository.java (1 hunk)
- clean-architecture/src/main/java/com.iluwatar.cleanArchitecture/InMemoryCartRepository.java (1 hunk)
- clean-architecture/src/main/java/com.iluwatar.cleanArchitecture/InMemoryOrderRepository.java (1 hunk)
- clean-architecture/src/main/java/com.iluwatar.cleanArchitecture/InMemoryProductRepository.java (1 hunk)
- clean-architecture/src/main/java/com.iluwatar.cleanArchitecture/Order.java (1 hunk)
- clean-architecture/src/main/java/com.iluwatar.cleanArchitecture/OrderController.java (1 hunk)
- clean-architecture/src/main/java/com.iluwatar.cleanArchitecture/OrderRepository.java (1 hunk)
- clean-architecture/src/main/java/com.iluwatar.cleanArchitecture/Product.java (1 hunk)
- clean-architecture/src/main/java/com.iluwatar.cleanArchitecture/ProductRepository.java (1 hunk)
- clean-architecture/src/main/java/com.iluwatar.cleanArchitecture/ShoppingCartService.java (1 hunk)
- clean-architecture/src/test/java/com/iluwatar/cleanArchitecture/AppTest.java (1 hunk)
- clean-architecture/src/test/java/com/iluwatar/cleanArchitecture/CartControllerTest.java (1 hunk)
Actionable Comments (0)
Skipped Comments (0)
|
Hi @iluwatar Kindly give it a look. |
iluwatar
left a comment
There was a problem hiding this comment.
Please add the new module to parent pom.xml, othewise it's not built by CI.
There was a problem hiding this comment.
✅ LGTM!
Review Summary
Files Processed (20)
- clean-architecture/README.md (1 hunk)
- clean-architecture/etc/cleanArchitectureUMLDiagram.PNG (0 hunks)
- clean-architecture/img.png (0 hunks)
- clean-architecture/pom.xml (1 hunk)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/App.java (1 hunk)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/Cart.java (1 hunk)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/CartController.java (1 hunk)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/CartRepository.java (1 hunk)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/InMemoryCartRepository.java (1 hunk)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/InMemoryOrderRepository.java (1 hunk)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/InMemoryProductRepository.java (1 hunk)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/Order.java (1 hunk)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/OrderController.java (1 hunk)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/OrderRepository.java (1 hunk)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/Product.java (1 hunk)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/ProductRepository.java (1 hunk)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/ShoppingCartService.java (1 hunk)
- clean-architecture/src/test/java/com/iluwatar/cleanarchitecture/AppTest.java (1 hunk)
- clean-architecture/src/test/java/com/iluwatar/cleanarchitecture/CartControllerTest.java (1 hunk)
- pom.xml (1 hunk)
Actionable Comments (0)
Skipped Comments (0)
There was a problem hiding this comment.
✅ LGTM!
Review Summary
Files Processed (13)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/App.java (1 hunk)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/Cart.java (1 hunk)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/CartController.java (1 hunk)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/CartRepository.java (1 hunk)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/InMemoryCartRepository.java (1 hunk)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/InMemoryOrderRepository.java (1 hunk)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/InMemoryProductRepository.java (1 hunk)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/Order.java (1 hunk)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/OrderController.java (1 hunk)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/OrderRepository.java (1 hunk)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/Product.java (1 hunk)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/ProductRepository.java (1 hunk)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/ShoppingCartService.java (1 hunk)
Actionable Comments (0)
Skipped Comments (0)
There was a problem hiding this comment.
✅ LGTM!
Review Summary
Files Processed (14)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/App.java (1 hunk)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/Cart.java (1 hunk)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/CartController.java (1 hunk)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/CartRepository.java (1 hunk)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/InMemoryCartRepository.java (1 hunk)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/InMemoryOrderRepository.java (1 hunk)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/InMemoryProductRepository.java (1 hunk)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/Order.java (1 hunk)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/OrderController.java (1 hunk)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/OrderRepository.java (1 hunk)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/Product.java (1 hunk)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/ProductRepository.java (1 hunk)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/ShoppingCartService.java (1 hunk)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/package-info.java (1 hunk)
Actionable Comments (0)
Skipped Comments (0)
There was a problem hiding this comment.
✅ LGTM!
Review Summary
Files Processed (4)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/Cart.java (1 hunk)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/InMemoryProductRepository.java (1 hunk)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/Order.java (1 hunk)
- clean-architecture/src/main/java/com/iluwatar/cleanarchitecture/ShoppingCartService.java (1 hunk)
Actionable Comments (0)
Skipped Comments (0)
There was a problem hiding this comment.
✅ LGTM!
Review Summary
Files Processed (2)
- clean-architecture/src/test/java/com/iluwatar/cleanarchitecture/AppTest.java (1 hunk)
- clean-architecture/src/test/java/com/iluwatar/cleanarchitecture/CartControllerTest.java (1 hunk)
Actionable Comments (0)
Skipped Comments (1)
-
clean-architecture/src/test/java/com/iluwatar/cleanarchitecture/AppTest.java [15-19]
enhancement: "Add more comprehensive assertions to AppTest."
|
Hi @iluwatar kindly give it a look. |
|
|
Hi @iluwatar kindly give it a look and review it as per your convenience. I have done the changes. |
There was a problem hiding this comment.
✅ LGTM!
Review Summary
Commits Considered (1)
- 7521a8b: Merge branch 'master' into feature/3230
Files Processed (0)
Actionable Comments (0)
Skipped Comments (0)
|
Looks good! Thank you for the contribution 🎉 @all-contributors please add @Suchismita-Deb for code |
|
I've put up a pull request to add @Suchismita-Deb! 🎉 |





Added the new pattern Clean-Architecture.