zulip/tools/linter_lib
Steve Howell 7ec5fbab2e node tests: Remove templates test.
I believe we can remove these and rely on
other parts of our testing/code-review
to ensure template quality.

These tests never really exercised our
app code, as evidenced by us not regressing
any of the 100%-line-coverage files.

We have a couple other ways that we verify
the correct format of the templates:

    - webpack (can they compile?)
    - check-templates (are they nicely indented?)

For deep testing, we have Casper, which
exercises most of our most important templates
in some meaningful way.

I think it's pretty rare that we get bugs
now that are directly caused by bad templates,
and an even smaller subset of them would
have been caught by the node tests.

If that trend changes in the future, I would prefer to
just do something "greenfield" to address
any common problems rather than resurrect
this code, but we could always resurrect it
from git.

The template node tests did check a little bit of
detail about which fields are there, but not
in an integrated way, so that aspect of the tests
wasn't very useful either.
2020-02-12 09:58:23 -08:00
..
__init__.py Extract tools/linter_lib/custom_check.py. 2017-06-05 09:20:21 -07:00
custom_check.py node tests: Remove templates test. 2020-02-12 09:58:23 -08:00
exclude.py check-openapi: Fix lint errors and remove lint exclusion. 2020-02-07 14:09:47 -08:00
pep8.py tools: Remove unused imports. 2019-02-02 17:10:31 -08:00
pyflakes.py linter_lib: Fix mypy errors. 2019-08-09 17:22:45 -07:00