pantsbuild / pants / 6421 / 3
0%
main: 93%

LAST BUILD BRANCH: nodejs-tool-bundled-lockfiles
DEFAULT BRANCH: main
Ran
Files 373
Run time 12s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 70.241% (+0.2%) from 70.011%
CI_FLAGS="-fkmsrcn -u 1/2 'Unit tests for pants and pants-plugins - shard 2'"

push

travis-ci

jsirois 
Fix PythonThriftBuilder to operate on 1 target.

Previously it operated on the closure of thrift files depended on by a
single PythonThriftLibrary and this trampled on work both PythonChroot
and SetupPy did to walk the closure and link together CodeGenerator
outputs.  This brings PythonThriftBuilder into line PythonAntlrBuilder
and fixes 2 bugs.

Tests are added that fail before the fix and pass after.

Testing Done:
After adding the 2 tests but before the fix:
```console
$ ./pants test tests/python/pants_test/backend/python/tasks/:setup_py -- -vktest_exported_thrift_issues_2005
...
         with extracted_sdist(target2, 'test.exported_dependee-0.0.0') as (py_files, path):
           self.assertEqual({path('setup.py'),
                             path('src/__init__.py'),
                             path('src/exported_dependee/__init__.py'),
                             path('src/exported_dependee/constants.py'),
                             path('src/exported_dependee/ttypes.py')},
 >                          py_files)
 E         AssertionError: Items in the second set but not the first:
 E         '/tmp/tmpMLYtFG/test.exported_dependee-0.0.0/src/exported/constants.py'
 E         '/tmp/tmpMLYtFG/test.exported_dependee-0.0.0/src/exported/__init__.py'
 E         '/tmp/tmpMLYtFG/test.exported_dependee-0.0.0/src/exported/ttypes.py'
...
```

```console
$ ./pants test tests/python/pants_test/backend/python:python_chroot -- -vktest_thrift_issues_2005
...
 tests/python/pants_test/backend/python/test_python_chroot.py:200: in do_test_thrift
     inspect_chroot(python_chroot)
 tests/python/pants_test/backend/python/test_python_chroot.py:241: in inspect_chroot
     self.assertEqual(2, len(all_constants_files))
 E   AssertionError: 2 != 3
...
```

Both green after the fix.

This pants also works on the test Wyatt created on the thrift-include
branch of https://github.com/wyattanderson/pantstest

CI went green here:
  https://travis-ci.org/pantsbuild/pants/builds/77183330

Bugs closed: 2005, 2054, 2062

Reviewed at https://rbcommons.com/s/twitter/r/2696/

15753 of 22427 relevant lines covered (70.24%)

0.7 hits per line

Source Files on job 6421.3 (CI_FLAGS="-fkmsrcn -u 1/2 'Unit tests for pants and pants-plugins - shard 2'")