Dcicutils

=====

`PR 222: Improved IntegratedFixture and static check cleanups <https://github.com/4dn-dcic/utils/pull/222>`_

* Show fewer uninteresting tracebacks on static test failures.

* Small incompatible changes to recently released qa-related items:

* In ``qa_checkers.confirm_no_uses``, remove the new ``if_used`` argument in favor of a simpler implementation.

* Slightly rerefactored the class hierarchy so that ``StaticChecker`` is a smaller class that doesn't have quite
as much functionality, and ``StaticSourcesChecker`` corresponds to what ``StaticChecker`` previously did.

Since this is all testing-only, not something used in production, and since there are believed to not yet be uses
outside the repo, we're treating this as a bug fix (patch version bump) not an incompatible change (which would
entail a major version bump and a lot of fussing for nothing).

* Make class initialization of ``IntegratedFixture`` happen at instance-creation time.
That simplifies the loading actions needed. Those can happen in ``conftest.py`` rather than in
``dcicutils.ff_mocks``, which in turn should allow ``dcicutils.ff_mocks`` to be imported without error,
fixing `C4-932 <https://hms-dbmi.atlassian.net/browse/C4-932>`_

=====

* Some functionality moved from ``qa_utils`` to ``qa_checkers``.
In each case, to be compatible, the ``qa_utils`` module will continue
to have the entity availble for import until the next major release.

* Class ``VersionChecker``
* Class ``ChangeLogChecker``
* Function ``confirm_no_uses``
* Function ``find_uses``
* Variable ``QA_EXCEPTION_PATTERN``

As an official matter, use of these moved entities from by importing
them from ``dcicutils.qa_utils`` is deprecated. Please update programs
to import these from ``dcicutils.qa_checkers`` instead.

* New functionality in ``qa_checkers``:

* New class ``DocsChecker``
* New class ``DebuggingArtifactChecker``

* In ``misc_utils``:

* New function ``lines_printed_to``.

* New ``pytest`` marker ``static`` for static tests.

* New ``make`` target ``test-static`` to run tests marked with
``pytest.mark.static``.

* New GithubActions (GA) workflow: ``static_checks.yml``

=====

* In ``qa_utils``:

* New class ChangeLogChecker, like VersionChecker, but it raises an error
if there's a change log inconsistency.

=====

* Drop support for Python 3.6 (**BREAKING CHANGE**)

=====

* New functionallity in ``ecr_utils.ECRUtils`` in support of planned changes to Foursight:

* Add ``ECRTagWatcher`` class that can be used to watch for a new image with a given tag in an ECS repository.

* New functionality in ``qa_utils`` to support a mock ECR client.

* Refactor parts of ``ecr_utils`` and ``ecr_scripts`` to move some general-purpose parts out of
``ecr_scripts`` (top-level variables and class ``ECRCommandContext``)
and into ``ecr_utils`` (class ``ECRUtils``):

* Changes to arguments for ``ECRUtils`` constructor:

* Allow additional arguments needed for moved methods.
* Default more arguments so that only relevant ones need be passed.

* Move some methods from ``ECRCommandContext`` to ``ECRUtils``:

* ``get_images_descriptions``
* ``_apply_image_descriptions_limit``

* Certain variables at ``ecr_scripts`` top-level became class variables in ``ecr_utils.ECRUtils``
(some with some renaming):


+------------------------+------------------------+--------------------------------+------------------------+
| .. raw:: html | .. raw:: html |
| | |
| <center><tt>ecr_scripts</code></tt> | <center><tt>ecr_utils.ECRUtils</tt></center> |
| | |
+------------------------+------------------------+--------------------------------+------------------------+
| module variable | module variable status | class variable | class variable status |
+========================+========================+================================+========================+
| DEFAULT_ECS_REPOSITORY | deprecated | DEFAULT_IMAGE_REPOSITORY | new |
+------------------------+------------------------+--------------------------------+------------------------+
| IMAGE_COUNT_LIMIT | deprecated | IMAGE_LIST_DEFAULT_COUNT_LIMIT | new |
+------------------------+------------------------+--------------------------------+------------------------+
| IMAGE_LIST_CHUNK_SIZE | deprecated | IMAGE_LIST_CHUNK_SIZE | new |
+------------------------+------------------------+--------------------------------+------------------------+
| RELEASED_TAG | deprecated | IMAGE_RELEASED_TAG | new |
+------------------------+------------------------+--------------------------------+------------------------+

* Unit tests for new functionality, and backfilled unit tests for some parts of ``ecr_utils``.

=====

* In ``env_utils``:

* New function ``foursight_env_name``, an alias for
``lambda envname: infer_foursight_from_env(envname=envname)``

* Add error checking for running tests that looks to see that we're in the right account before we move ahead
only to find this out in a less intelligible way.