Skip to content

Conversation

cagla88
Copy link
Collaborator

@cagla88 cagla88 commented Feb 3, 2025

Pull Request

Description:
Create Password positive and negative tests have been added.

Changes Made:
List the changes you made in this pull request.

Additional Information:
Add any additional information that might be relevant to reviewers.

Reviewer Instructions:
Provide instructions or guidelines for reviewers to follow when assessing your pull request.

Checklist

Before submitting your pull request, ensure the following:

  • Title and Branch Naming Conventions:

  • Local Testing:

    • I have added positive and negative test scenarios (if applicable)
    • All tests are passing.
    • I tagged all test as smoke or regression
    • I added newly added tests to related xml file in test suite
  • Code Quality:

    • The code is formatted according to the project's coding guidelines and style.
    • The code has been reviewed to ensure its quality.
    • The code does not contain any issues flagged by SonarLint.
  • Documentation:

    • Necessary documentation has been added or existing documentation has been updated.
  • Reviewers and Assignees:

    • Default reviewers have been assigned to this pull request.
    • Assignees have been added if necessary.
  • Labels and Associations:

    • No specific actions are required in the Labels and Associations section for this pull request.
  • Reviewer Instructions:

    • Provide instructions or guidelines for reviewers to follow when assessing your pull request.

@cagla88 cagla88 self-assigned this Feb 3, 2025
@cagla88
Copy link
Collaborator Author

cagla88 commented Feb 4, 2025

@agitrubard @Nbhtozcn selamlar, hata veren testler AYS-683 issue'su ile düzenlenen hata mesajından kaynaklı burada yapılan düzenlemelerle ilgili değildir. O issue test edilirken düzenlenmesi daha uygun olur.

@agitrubard
Copy link
Contributor

@cagla88 bu durumda o testleri disable etmemiz ve issue'yu not düşmemiz daha iyi olmaz mı?

@cagla88
Copy link
Collaborator Author

cagla88 commented Feb 4, 2025

@cagla88 bu durumda o testleri disable etmemiz ve issue'yu not düşmemiz daha iyi olmaz mı?

@agitrubard Bunun yerine iki issue'yu birlikte ilerletme kararı aldım zira tek bir kelime eklemek daha kolay bir yol. AYS-683 ile bu issue yu ilişkilendirip not düştüm

@agitrubard
Copy link
Contributor

@agitrubard Bunun yerine iki issue'yu birlikte ilerletme kararı aldım zira tek bir kelime eklemek daha kolay bir yol. AYS-683 ile bu issue yu ilişkilendirip not düştüm

@cagla88 eklemek daha kolay bir yol ama kolay yolu seçtiğimizde o issue Jira'da otomasyona yansıtılmamış gibi görünecek, bu durumda geri alıp farklı bir branch'te eklemeyi yapıp onu mergeleyip bunu sonra mergeleyebiliriz daha iyi olur 👍🏼

Görünmemesinin sebebi Squash and Merge yapıyor olmamız ve atılan commitlerin mergelendikten sonra pull request başlığına göre birleştiriliyor olması;
https://medium.com/@agitrubard/neden-squash-and-merge-41aa5b93992c

@cagla88 cagla88 requested a review from agitrubard February 6, 2025 06:14
@cagla88 cagla88 merged commit c083129 into main Feb 10, 2025
1 check passed
@cagla88 cagla88 deleted the test/AYS-412/createPassword branch February 10, 2025 12:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants