-
-
Notifications
You must be signed in to change notification settings - Fork 27.2k
Feature/3230 - Clean Architecture #3235
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✅ 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. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add the new module to parent pom.xml
, othewise it's not built by CI.
clean-architecture/src/main/java/com.iluwatar.cleanArchitecture/App.java
Outdated
Show resolved
Hide resolved
clean-architecture/src/main/java/com.iluwatar.cleanArchitecture/App.java
Outdated
Show resolved
Hide resolved
clean-architecture/src/main/java/com.iluwatar.cleanArchitecture/App.java
Outdated
Show resolved
Hide resolved
clean-architecture/src/main/java/com.iluwatar.cleanArchitecture/Cart.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✅ 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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✅ 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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✅ 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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✅ 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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✅ 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. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✅ 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.