[WIP] First try of conu test implementation #198
Conversation
|
@TomasTomecek, FYI, we might chat about this sometime. |
| @@ -8,3 +10,10 @@ OPENSHIFT_NAMESPACES = 5.5 | |||
| .PHONY: $(shell test -f common/common.mk || echo >&2 'Please do "git submodule update --init" first.') | |||
|
|
|||
| include common/common.mk | |||
|
|
|||
| build-test-container: | |||
| docker build --network host --tag=$(TEST_IMAGE_NAME) -f ./test/Dockerfile.tests . | |||
There was a problem hiding this comment.
Dockerfile.tests seems to be missing
| app.rmi() | ||
|
|
||
|
|
||
| class GenericTestSuite: |
There was a problem hiding this comment.
I'm assuming you want this class to be in conu :)
There was a problem hiding this comment.
Yes, that was the intention, but I did not go far enough to propose it as a PR.
| backend = DockerBackend(logging_level=logging.DEBUG) | ||
|
|
||
|
|
||
| @pytest.fixture(scope="module", params=app_paths) |
There was a problem hiding this comment.
and we could also start shipping some fine-crafted pytest fixtures in conu
Since there is no pressure to have this PR finished asap, I'd like rather to do the changes in |
|
We have disabled conu in RHSCL CI testing. Closing this issue. |
Based on sclorg/s2i-ruby-container#167 with slight generalization. The point here is to see how much of the tests we can actually generalize, and put into conu library. It seems quite a lot.