ENH: Test development container

[adrienpacifico]

Feature Type

• Adding new functionality to pandas

• Changing existing functionality in pandas

• Removing existing functionality in pandas

Problem Description

I am currently trying to set en development environment on a X86 Linux system by passing all test on my machine with a the provided Dockerfile.

The current development environment with this Dockerfile is not working since fast_test.sh is not passing.

This Dockerfile is tested in the CI to validate that it builds, and that it print the pandas versions :

pandas/.github/workflows/code-checks.yml

Line 132 in 78a5f71

build_docker_dev_environment:

However, this dev environment is not tested to determine if it can run all the unit tests.

Feature Description

The Docker dev environment test should be that it passes at least tests ran in the test_fast.sh file.

[theoniko]

Hello @lithomas1, @mroeschke
I had a look to this issue. Does it mean we need to add add only the following in code-checks.yml
- name: Test
uses: ./.github/actions/run-tests
Does it need to run some other shell file called fast_test.sh? Btw such file does not exist...

[adrienpacifico]

@theoniko fast_test.sh has been deleted by #48958

Does it mean we need to add add only the following in code-checks.yml

I think that you need to do more than that, currently, if I launch the dev container from the GitHub URL, the test suite does not even start.

[theoniko]

@adrienpacifico: I am sure about your feature request as @mroeschke commented in the PR 52349.

For the test suite you are right. I checked the logs and i see the problem

[mroeschke]

For the test suite you are right. I checked the logs and i see the problem

That indicates that the C extensions have not been built

As mentioned in #52349 (review), the development container is just for development and is not used in our CI to set the environment and run tests necessarily so I think it would be overkill to also run the same tests in the container as the CI. I think the current check showing that the environment is available is sufficient so closing