Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | test: use the coverage-command fixture throughout the venv tests | Ned Batchelder | 2021-10-31 | 1 | -4/+4 | |
| | ||||||
* | fix: don't warn about already imported files for namespace packages #888 | Ned Batchelder | 2021-10-31 | 1 | -2/+52 | |
| | | | | Ignore namespace packages in the already-imported check. #888 | |||||
* | refactor(test): the virtualenv tests deserve their own file | Ned Batchelder | 2021-10-31 | 2 | -263/+277 | |
| | ||||||
* | debug: code we can enable to see where tests are running and in what order | Ned Batchelder | 2021-10-30 | 1 | -1/+9 | |
| | ||||||
* | chore: update HTML gold files | Ned Batchelder | 2021-10-30 | 24 | -74/+77 | |
| | | | | test: I keep forgetting about the 3.10-only gold file | |||||
* | test: fix the trailing-space test, and suppress warnings about \<space> | Ned Batchelder | 2021-10-30 | 2 | -12/+18 | |
| | ||||||
* | style: remove implicit string concatenation | Ned Batchelder | 2021-10-30 | 2 | -25/+25 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | .. and also add trailing commas where needed to remove ambiguity. I used semgrep with this config: ``` rules: - id: implicit-string-concatenation pattern: | "$X" "$Y" languages: - python message: string cat BAD severity: INFO ``` Running it showed this: ``` $ semgrep --config=../lab/string-cat.yml coverage tests doc ci igor.py setup.py __main__.py Running 1 rules... setup.py severity:info rule:lab.implicit-string-concatenation: string cat BAD 128: 'https://tidelift.com/subscription/pkg/pypi-coverage' 129: '?utm_source=pypi-coverage&utm_medium=referral&utm_campaign=pypi' tests/test_api.py severity:info rule:lab.implicit-string-concatenation: string cat BAD 319: "!coverage.py: This is a private format, don't read it directly!" 320: '{"lines":{"/private/tmp/foo.py":[1,5,2,3]}}' tests/test_html.py severity:info rule:lab.implicit-string-concatenation: string cat BAD 614: ('<span class="key">if</span> <span class="num">1</span> ' 615: '<span class="op"><</span> <span class="num">2</span>'), -------------------------------------------------------------------------------- 616: (' <span class="nam">a</span> ' 617: '<span class="op">=</span> <span class="num">3</span>'), -------------------------------------------------------------------------------- 665: ('<span class="key">if</span> <span class="nam">x</span> ' 666: '<span class="op"><</span> <span class="num">2</span>'), -------------------------------------------------------------------------------- 667: (' <span class="nam">a</span> <span class="op">=</span> ' 668: '<span class="num">3</span>'), -------------------------------------------------------------------------------- 949: ('<span class="key">if</span> <span class="num">1</span> ' 950: '<span class="op"><</span> <span class="num">2</span>'), -------------------------------------------------------------------------------- 951: (' <span class="nam">a</span> <span class="op">=</span> ' 952: '<span class="num">3</span>'), ran 1 rules on 134 files: 8 findings ``` This wasn't all of the instances, so I wrote a bug report: https://github.com/returntocorp/semgrep/issues/4184 | |||||
* | deprecated: the annotate command will be removed in a future version | Ned Batchelder | 2021-10-29 | 1 | -1/+6 | |
| | ||||||
* | refactor(test): make re_lines (et al) look like re.search | Ned Batchelder | 2021-10-28 | 7 | -74/+68 | |
| | | | | and also replace some calls with just-plain re.search. | |||||
* | refactor(test): name this gold file properly | Ned Batchelder | 2021-10-28 | 2 | -1/+1 | |
| | ||||||
* | test: add a annotate test, and simplify the logic | Ned Batchelder | 2021-10-28 | 2 | -0/+30 | |
| | ||||||
* | feat: html output directory gets a .gitignore file. #1244 | Ned Batchelder | 2021-10-27 | 1 | -0/+1 | |
| | ||||||
* | fix(debug): ast_dump failed on a few things | Ned Batchelder | 2021-10-27 | 3 | -16/+37 | |
| | ||||||
* | refactor(test): re_lines is more useful if it returns a list | Ned Batchelder | 2021-10-27 | 4 | -19/+28 | |
| | ||||||
* | feat: xml and json say what they are doing, and -q quiets everything. #1254nedbat/dashq | Ned Batchelder | 2021-10-26 | 5 | -16/+59 | |
| | ||||||
* | test: a true unit test for render_report | Ned Batchelder | 2021-10-26 | 1 | -0/+50 | |
| | ||||||
* | refactor: alphabetize the options | Ned Batchelder | 2021-10-26 | 1 | -53/+53 | |
| | ||||||
* | fix: changes for PyPy3.8 | Ned Batchelder | 2021-10-25 | 3 | -1/+4 | |
| | | | | | | | | | | - Update tox.ini to let us run against PyPy3.8 - Some 3.8 behavior is (apparently) only on CPython - PyPy3.8 doesn't get along with virtualenv yet (https://github.com/pypa/virtualenv/issues/2182), so use venv instead for our virtualenv tests. | |||||
* | test: also test goldtest bad xml | Ned Batchelder | 2021-10-25 | 2 | -14/+37 | |
| | ||||||
* | test: finishing covering goldtest.py with test_goldtest.py | Ned Batchelder | 2021-10-24 | 4 | -24/+78 | |
| | ||||||
* | test: test compare() from test/goldtest.py | Ned Batchelder | 2021-10-24 | 3 | -45/+125 | |
| | ||||||
* | refactor(test): os_sep and remove_tree helpers | Ned Batchelder | 2021-10-24 | 6 | -24/+30 | |
| | ||||||
* | fix(html): ariaSort isn't supported in Firefox yet | Ned Batchelder | 2021-10-23 | 3 | -14/+12 | |
| | | | | The HTML report index page wasn't indicating the sort order properly | |||||
* | chore(test): update gold files | Ned Batchelder | 2021-10-23 | 27 | -213/+217 | |
| | ||||||
* | test: update gold files | Ned Batchelder | 2021-10-23 | 43 | -2063/+1890 | |
| | ||||||
* | fix(html): fix a few problems with the html report | Ned Batchelder | 2021-10-23 | 3 | -15/+32 | |
| | | | | | | | | | | - highlights weren't showing - anchored lines were not visible - some j/k motions were broken - clicking the big buttons at the top didn't work | |||||
* | test: sometimes pytest directories have a "popen" component | Ned Batchelder | 2021-10-23 | 1 | -3/+4 | |
| | | | | | Those have to be scrubbed too. I'm not sure why it's sometimes there, sometimes not. Maybe xdist? | |||||
* | refactor: no more need for a search path for static files | Ned Batchelder | 2021-10-23 | 1 | -57/+1 | |
| | | | | | | | We used to search an OS-specific directory in addition to our own, specifically so that Debian could use an OS-installed copy of jQuery and its plugins. But we no longer have jQuery or any third-party JavaScript code, so we don't need to search the Debian directories. | |||||
* | refactor(html): remove all uses of jQuery, only vanilla JS now | Septatrix | 2021-10-23 | 1 | -8/+12 | |
| | | | | This is squashed from pull request #1248 | |||||
* | refactor(html): a one-header sticky header | Ned Batchelder | 2021-10-23 | 1 | -31/+33 | |
| | ||||||
* | fix(html): scrolling to an anchor ensures the line is visible | Ned Batchelder | 2021-10-20 | 1 | -3/+3 | |
| | | | | | | | The sticky header was hiding the line scrolled to the top of the window. Along the way, also changed to use classes on <body> to control the sticky header, and moved the header code into a function. | |||||
* | fix: avoid measuring generated code. #1160 | Ned Batchelder | 2021-10-18 | 1 | -3/+19 | |
| | ||||||
* | test: remove some obsolete HTML scrubs | Ned Batchelder | 2021-10-17 | 1 | -5/+0 | |
| | ||||||
* | chore: updated HTML gold files | Ned Batchelder | 2021-10-17 | 38 | -383/+626 | |
| | ||||||
* | test: don't show these functions when HTML gold files fail | Ned Batchelder | 2021-10-17 | 2 | -0/+3 | |
| | ||||||
* | test: html scrubs need to account for pytest dir naming | Ned Batchelder | 2021-10-17 | 1 | -0/+1 | |
| | ||||||
* | fix(html): adjust some colors | Ned Batchelder | 2021-10-17 | 1 | -8/+8 | |
| | | | | | | | | - make the yellow border more visible - make the context-blue less vibrant - all colors are lower-case | |||||
* | feat(html): sticky header on pyfile | Ned Batchelder | 2021-10-17 | 1 | -5/+15 | |
| | ||||||
* | refactor(html): use header/main/footer html5 elements | Ned Batchelder | 2021-10-17 | 1 | -8/+8 | |
| | ||||||
* | test: fix the tests/gold/html/Makefile to copy actual output | Ned Batchelder | 2021-10-17 | 1 | -4/+3 | |
| | ||||||
* | test: improve this one html test | Ned Batchelder | 2021-10-17 | 1 | -3/+3 | |
| | ||||||
* | feat: `coverage run` now sets the COVERAGE_RUN environment variable | Ned Batchelder | 2021-10-16 | 1 | -1/+16 | |
| | ||||||
* | test: very minor clean up | Ned Batchelder | 2021-10-16 | 1 | -1/+4 | |
| | ||||||
* | test: subprocess diagnostic for when we need it | Ned Batchelder | 2021-10-16 | 1 | -0/+6 | |
| | ||||||
* | test: we don't need two sys.path restorers | Ned Batchelder | 2021-10-14 | 3 | -15/+6 | |
| | ||||||
* | test: parametrize test_pathologically_long_code_object | Ned Batchelder | 2021-10-14 | 1 | -21/+18 | |
| | | | | | While experimenting with packing branch ends into one int, this test showed a mistake I had made. Turns out it wasn't an obsolete test after all. | |||||
* | test: don't use subprocesses where we don't have to | Ned Batchelder | 2021-10-13 | 3 | -56/+74 | |
| | | | | | | When running tests with `-n 0`, this shaves 4 seconds off the run time. When running the complete test suite with xdist, it takes 2 seconds longer!? | |||||
* | refactor: use the get_report helper | Ned Batchelder | 2021-10-13 | 1 | -6/+1 | |
| | ||||||
* | refactor: no longer need to explicitly omit site-packages | Ned Batchelder | 2021-10-13 | 1 | -15/+0 | |
| | | | | 6.x omits site-packages automatically. | |||||
* | refactor: convert existing imports to start_stop_import | Ned Batchelder | 2021-10-13 | 1 | -15/+6 | |
| |