Skip to content
This repository has been archived by the owner on Jan 19, 2024. It is now read-only.

Functional tests #79

Merged
merged 11 commits into from
Dec 16, 2022
Merged

Functional tests #79

merged 11 commits into from
Dec 16, 2022

Conversation

ScottBailey
Copy link
Contributor

@ScottBailey ScottBailey commented Nov 30, 2022

Fixes #45

Testing is 90-ish-percent complete.

@jolly-fellow
Copy link
Contributor

As I remember first try to test DUNE and the discussing about this in telegram there were many problems related to shared files between of the host and the container. After that I made this issue #56 and propose to extend this description with all which helps further users to avoid difficulties related to the shares.

@mikelik
Copy link
Contributor

mikelik commented Dec 1, 2022

Is it possible to execute those tests in the Github Actions/whenever new merge or PR is created?

@ScottBailey
Copy link
Contributor Author

@mikelik I do not believe it's possible at this time. At least not for me. That said, maybe @kj4ezj has an opinion?

@jolly-fellow
Copy link
Contributor

jolly-fellow commented Dec 1, 2022

Is it possible to execute those tests in the Github Actions/whenever new merge or PR is created?

Theoretically it is possible but a prebuilt Docker image of EOS should be deployed on GitHub Container Registry. Github Actions will create a container from this image and run the tests inside. Bucky said it is a good idea to have a prebuilt image. It will be possible to create containers for CI, testing purposes and for work of end users instead of force each end user build his own image locally.

@mikelik
Copy link
Contributor

mikelik commented Dec 2, 2022

Theoretically it is possible but a prebuilt Docker image of EOS should be deployed on GitHub Container Registry.

I think prebuilt Docker image will be fine for release / regression testing.
Can we built a temporary (local) image and run functional tests against it on each PR / merge?
This way we could know if each commit builds valid image.

@jolly-fellow
Copy link
Contributor

jolly-fellow commented Dec 2, 2022

Theoretically it is possible but a prebuilt Docker image of EOS should be deployed on GitHub Container Registry.

I think prebuilt Docker image will be fine for release / regression testing. Can we built a temporary (local) image and run functional tests against it on each PR / merge? This way we could know if each commit builds valid image.

Yes of course. Why not? Everyone may build any number of images locally or in his own profile on GitHub or Docker Hub. It is possible to build images for each merge but I'm not sure it is possible for free. And I don't understand why we need to rebuild EOS image for testing DUNE which works outside of container. Anyway it is always possible to build an image locally and push it to the repository for free.
But you are right. Generally the idea is to do tests against official releases and for creation of containers for the end users.

@jolly-fellow
Copy link
Contributor

jolly-fellow commented Dec 2, 2022

@ScottBailey It seems solution of the problem with deployment of eosio.system will take more time than I thought so I added issues #80 and #81 to separate it and making of a test for --get-table from this PR, so IMHO you may merge it as is when it will be approved. Bucky is agree.

tests/show_untested_options.sh Show resolved Hide resolved
tests/test_boostrap.py Show resolved Hide resolved
tests/test_nodes.py Show resolved Hide resolved
tests/test_project.py Show resolved Hide resolved
tests/test_boostrap.py Show resolved Hide resolved
tests/test_all_options_tested.py Outdated Show resolved Hide resolved
tests/test_boostrap.py Show resolved Hide resolved
tests/test_debug.py Outdated Show resolved Hide resolved
tests/test_project.py Outdated Show resolved Hide resolved
tests/container.py Outdated Show resolved Hide resolved
tests/container.py Outdated Show resolved Hide resolved
tests/container.py Outdated Show resolved Hide resolved
tests/container.py Outdated Show resolved Hide resolved
@ScottBailey ScottBailey requested a review from mikelik December 12, 2022 20:42
@ScottBailey
Copy link
Contributor Author

ScottBailey commented Dec 14, 2022

The following issues have been written to resolve remaining problems: #85 #86

Fix failing test_project.py - merge to functional-tests branch
@@ -0,0 +1,2 @@
temp/
test_app/
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't understand why these directories should be here because normally they should be created before the tests and removed after the tests. If they exist after end of the tests it is a bug.

@ScottBailey ScottBailey merged commit 69f7e8e into main Dec 16, 2022
@ScottBailey ScottBailey deleted the functional-tests branch December 16, 2022 15:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Write functional tests for DUNE
3 participants