From bada5a184ec5b125b5ee0428166e1b989f090b8b Mon Sep 17 00:00:00 2001 From: Jim McStanton Date: Fri, 14 Jun 2019 09:24:18 -0500 Subject: [PATCH] Cleared up sentence in unit test docs --- doc/topics/development/tests/unit.rst | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/doc/topics/development/tests/unit.rst b/doc/topics/development/tests/unit.rst index d129ad478ef5..fa9c2d8657fc 100644 --- a/doc/topics/development/tests/unit.rst +++ b/doc/topics/development/tests/unit.rst @@ -99,8 +99,8 @@ Salt loader modules use a series of globally available dunder variables, ``__salt__``, ``__opts__``, ``__pillar__``, etc. To facilitate testing these modules a mixin class was created, ``LoaderModuleMockMixin`` which can be found in ``tests/support/mixins.py``. The reason for the existence of this class is -because historiclly and because it was easier, one would add these dunder -variables directly on the imported module. This however, introduces unexpected +because historically one would add these dunder +variables directly on the imported module. This, however, introduces unexpected behavior when running the full test suite since those attributes would not be removed once we were done testing the module and would therefore leak to other modules being tested with unpredictable results. This is the kind of work that