[ 
https://issues.apache.org/jira/browse/BEAM-8572?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16968713#comment-16968713
 ] 

Udi Meiri commented on BEAM-8572:
---------------------------------

Looks like some Cython-requiring tests are always skipped.

{code}
$ curl -s 
https://builds.apache.org/job/beam_PreCommit_Python_Phrase/900/consoleText | 
grep fast_coders_test.FastCoders consoleText 
test_using_fast_impl (apache_beam.coders.fast_coders_test.FastCoders) ... SKIP: 
Cython is not installed
test_using_fast_impl (apache_beam.coders.fast_coders_test.FastCoders) ... SKIP: 
Cython is not installed
test_using_fast_impl (apache_beam.coders.fast_coders_test.FastCoders) ... SKIP: 
Cython is not installed
test_using_fast_impl (apache_beam.coders.fast_coders_test.FastCoders) ... SKIP: 
Cython is not installed
test_using_fast_impl (apache_beam.coders.fast_coders_test.FastCoders) ... SKIP: 
Cython is not installed
test_using_fast_impl (apache_beam.coders.fast_coders_test.FastCoders) ... SKIP: 
Cython is not installed
test_using_fast_impl (apache_beam.coders.fast_coders_test.FastCoders) ... SKIP: 
Cython is not installed
test_using_fast_impl (apache_beam.coders.fast_coders_test.FastCoders) ... SKIP: 
Cython is not installed
test_using_fast_impl (apache_beam.coders.fast_coders_test.FastCoders) ... SKIP: 
Cython is not installed
test_using_fast_impl (apache_beam.coders.fast_coders_test.FastCoders) ... SKIP: 
Cython is not installed
test_using_fast_impl (apache_beam.coders.fast_coders_test.FastCoders) ... SKIP: 
Cython is not installed
test_using_fast_impl (apache_beam.coders.fast_coders_test.FastCoders) ... SKIP: 
Cython is not installed
{code}

> tox environment: assert on Cython source file presence
> ------------------------------------------------------
>
>                 Key: BEAM-8572
>                 URL: https://issues.apache.org/jira/browse/BEAM-8572
>             Project: Beam
>          Issue Type: Bug
>          Components: sdk-py-core, testing
>            Reporter: Udi Meiri
>            Assignee: Udi Meiri
>            Priority: Major
>
> Add an assertion somewhere that checks if Cythonized files are present in the 
> sdist tarball in use. That is for "tox -e py27" assert that these files are 
> not present, for "tox -e py27-cython" assert that they are present.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to