conductor-sdk / conductor-python

Conductor OSS SDK for Python programming language
Apache License 2.0
62 stars 29 forks source link

Integration tests depend on already created workflows #117

Open marcocrasso opened 2 years ago

marcocrasso commented 2 years ago

There are two sets of integration tests, one for testing registration, the other execution. The registration test tries to register a workflow, which requires simple tasks and sub-workflow to be already registered. Furthermore, "override" doesn't work with community conductor server, thus the test requires to first unregister the workflow (if present). The other test has similar requirements.

How to reproduce the issue

git clone https://github.com/conductor-sdk/conductor-python
cd conductor-python
git checkout main
git pull 
docker run --name test-conductor-server --rm -d -p 8080:8080 ideaplexus/conductor-server:3.9.1 && echo "Waiting 10secs for Conductor Server to be ready..." && sleep 10
export PYTHON_INTEGRATION_TESTS_SERVER_API_URL=http://localhost:8080/api
export CONDUCTOR_PYTHON_VERSION=3.7
virtualenv conductor
source conductor/bin/activate
python3 setup.py install
python3 tests/integration.py