apache / apache/beam

tox environment: assert on Cython source file presence

Open
#19,883 0 comments 0 reactions 0 assignees View on GitHub
bug core P3 python tests
Dominant language
Java
Stars
8.7k
Forks
4.7k
Avg merge
1d 20h
Merged PRs (30d)
196

Description

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.

Imported from Jira [BEAM-8572](https://issues.apache.org/jira/browse/BEAM-8572). Original Jira may contain additional context.
Reported by: udim.

Contributor guide

Open the contributing guide

Research direction

Locate the tox definitions for the py27 and py27-cython environments, then trace how the sdist tarball in use is selected or inspected. Add checks matching each environment's expected Cython source-file presence, and verify both tox environments pass with the appropriate assertion.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
build-system, testing
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.