-
Notifications
You must be signed in to change notification settings - Fork 15
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
AYS-661 | Refactor email validation limits #431
Conversation
…min length to 6 characters - AYS-661 : Set email validation max length to 254 characters - Add test cases for email validation limits
@m1erla selamlar, |
Ben test ederken, yaptığım düzenlemeler sorunsuz geçti testlerden ama build aldığımda diğer test dosyalarından hata almıştım ama benim düzenlememde olmayan test dosyalarıydı. Onlara ayrı bir issue açmak gerek sanırım. |
@m1erla Belli ki yaptığımız geliştirmeler bir şeyleri bozmuş. Bu sebeple oradaki testlerin de düzeltilmesi gerekiyor. Ayrı bir issue açmamız gerekmiyor şu an için. Bu issue kapsamında yapılan değişikliklerin test etkileri olmuş gibi görünüyor. |
src/main/java/org/ays/auth/model/request/AdminRegistrationApplicationCompleteRequest.java
Outdated
Show resolved
Hide resolved
src/main/java/org/ays/auth/model/request/AysPasswordForgotRequest.java
Outdated
Show resolved
Hide resolved
src/main/java/org/ays/auth/model/request/AysUserCreateRequest.java
Outdated
Show resolved
Hide resolved
src/main/java/org/ays/auth/model/request/AysUserCreateRequest.java
Outdated
Show resolved
Hide resolved
src/main/java/org/ays/auth/model/request/AysUserCreateRequest.java
Outdated
Show resolved
Hide resolved
src/test/java/org/ays/emergency_application/model/EmergencyEvacuationApplicationBuilder.java
Outdated
Show resolved
Hide resolved
...test/java/org/ays/emergency_application/model/request/EmergencyEvacuationRequestBuilder.java
Outdated
Show resolved
Hide resolved
...test/java/org/ays/emergency_application/model/request/EmergencyEvacuationRequestBuilder.java
Outdated
Show resolved
Hide resolved
...test/java/org/ays/emergency_application/model/request/EmergencyEvacuationRequestBuilder.java
Outdated
Show resolved
Hide resolved
...test/java/org/ays/emergency_application/model/request/EmergencyEvacuationRequestBuilder.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.
Açık olan yorumlara ek olarak eklenen validasyona bağlı olarak testleri de düzenleyebilir miyiz?
...ava/org/ays/emergency_application/controller/EmergencyEvacuationApplicationEndToEndTest.java
Outdated
Show resolved
Hide resolved
@agitrubard Hocam sizleri çok yordum biliyorum fakat bunu biraz daha açma şansımız var mı? Yani yaptığım validasyon düzenlemesinin testleri mi eksik kaldı? Bir de açık olan yorumlar derken kaçırdığım ya da düzeltemediğim yorumunuz var mı halen? Tekrardan teşekkürler beni yönlendirdiğiniz için. |
@m1erla evet testleri göremedim o yüzden belirtmiştim 👍🏼 Eksik olan yorumlar unresolved şekilde duruyor yani açık şekilde duruyor oradan takip edilerek ilerlenebilir ✅ |
...ava/org/ays/emergency_application/controller/EmergencyEvacuationApplicationEndToEndTest.java
Outdated
Show resolved
Hide resolved
…l-validation-limits
…licationEndToEndTest." This reverts commit da7ef84.
… Has Been Formatted
…ts for Length Check
…ter` and Controller Test Has Been Created for Email Address Size Check
Checklist
Before submitting your pull request, ensure the following:
Title and Branch Naming Conventions:
standard: Pull Request Naming Conventions.
the Branch Naming Conventions.
Local Testing:
Code Quality:
Documentation:
Testing:
Reviewers and Assignees:
Labels and Associations: