Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Overhaul the fabric integration tests #1234

Closed
davidkel opened this issue Feb 22, 2022 · 3 comments
Closed

Overhaul the fabric integration tests #1234

davidkel opened this issue Feb 22, 2022 · 3 comments
Labels
build enhancement New feature or request test

Comments

@davidkel
Copy link
Contributor

davidkel commented Feb 22, 2022

There are 3 directories in caliper-test-integration

  • fabric_docker_distributed_tests
  • fabric_docker_local_tests
  • fabric_tests (currently the only one used as part of the caliper build process)

fabric_tests are the main tests for fabric, fabric_docker_distributed_tests tests using the docker image with remote workers, fabric_docker_local_tests tests using the docker image using local workers.

What we need is

  1. a test of the docker image
  2. a test where remote workers are used

we want to avoid creating to many different integration tests in fabriv as it is expensive to download and start the network currently so we could merge these 2 into a single integration test

@davidkel davidkel added the enhancement New feature or request label Feb 22, 2022
@davidkel davidkel changed the title x Overhaul the fabric integration tests Feb 22, 2022
@davidkel davidkel added the build label Feb 25, 2022
@davidkel davidkel added the test label May 1, 2024
@davidkel
Copy link
Contributor Author

davidkel commented May 2, 2024

see #1554 as this would allow for the testing of caliper for areas like the docker image and remote workers without having to spin up a real backend blockchain to run the tests.

@davidkel davidkel self-assigned this Sep 10, 2024
@davidkel
Copy link
Contributor Author

created individual issues #1621 and #1622

This will be about removing the out of date and integration tests that are not run

@davidkel
Copy link
Contributor Author

closing as covered elsewhere now

@davidkel davidkel removed their assignment Oct 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build enhancement New feature or request test
Projects
None yet
Development

No branches or pull requests

1 participant