Skip to content
This repository was archived by the owner on Nov 19, 2024. It is now read-only.

Conversation

eduard13
Copy link
Contributor

@eduard13 eduard13 commented May 7, 2019

Purpose of this pull request

This pull request (PR) improves the documentation of how to write testable code, by providing some simple of bad and good examples. Also it explains why it needs to keep it simple and maintainable.

Affected DevDocs pages

whatsnew
Added "Each function has one single purpose" section to Writing Testable Code.

@devops-devdocs
Copy link
Collaborator

An admin must run tests on this PR before it can be merged.

@rogyar rogyar self-assigned this May 7, 2019
@rogyar rogyar added the Major Update Significant original updates to existing content label May 7, 2019
@jeff-matthews jeff-matthews requested a review from dobooth May 7, 2019 13:23
@dobooth dobooth added 2.1.x 2.2.x 2.3.x Magento 2.3 related changes labels May 7, 2019
Copy link
Contributor

@dobooth dobooth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I made some small updates for clarity.

@dobooth
Copy link
Contributor

dobooth commented May 7, 2019

running tests

@dobooth dobooth merged commit ad54698 into magento:master May 7, 2019
@ghost
Copy link

ghost commented May 7, 2019

Hi @eduard13, thank you for your contribution!
Please, complete Contribution Survey, it will take less than a minute.
Your feedback will help us to improve contribution process.

@dshevtsov
Copy link
Collaborator

dshevtsov commented May 7, 2019

@dobooth please add whatsnew

@eduard13 eduard13 deleted the patch-writing-unit-tests branch July 27, 2019 06:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
2.1.x 2.2.x 2.3.x Magento 2.3 related changes Major Update Significant original updates to existing content Partner: Atwix partners-contribution PR created by Magento partner
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants