Skip to content

Commit ef31bfd

Browse files
[chore]: remove integration tests (#1406)
* yeet integration tests Signed-off-by: Pierre Tessier <pierre@pierretessier.com> * yeet integration tests Signed-off-by: Pierre Tessier <pierre@pierretessier.com> * yeet integration tests Signed-off-by: Pierre Tessier <pierre@pierretessier.com> --------- Signed-off-by: Pierre Tessier <pierre@pierretessier.com> Co-authored-by: Austin Parker <austin@ap2.io>
1 parent 6bd2ede commit ef31bfd

10 files changed

+6
-3425
lines changed

.github/workflows/build-images.yml

-4
Original file line numberDiff line numberDiff line change
@@ -115,10 +115,6 @@ jobs:
115115
tag_suffix: frontend-tests
116116
context: ./
117117
setup-qemu: true
118-
- file: ./test/Dockerfile
119-
tag_suffix: integrationTests
120-
context: ./
121-
setup-qemu: true
122118

123119
steps:
124120
- uses: actions/checkout@v4

CHANGELOG.md

+2
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,8 @@ the release.
1313
([#1397](https://github.com/open-telemetry/opentelemetry-demo/pull/1397))
1414
* [chore] add nightly releases
1515
([#1398](https://github.com/open-telemetry/opentelemetry-demo/pull/1398))
16+
* [chore] remove unused integration test
17+
([#1406](https://github.com/open-telemetry/opentelemetry-demo/pull/1406))
1618

1719
## 1.8.0
1820

Makefile

-2
Original file line numberDiff line numberDiff line change
@@ -98,8 +98,6 @@ build-env-file:
9898
.PHONY: run-tests
9999
run-tests:
100100
docker compose run frontendTests
101-
# integrationTests is deprecated in favor of traceBasedTests
102-
# docker compose run integrationTests
103101
docker compose run traceBasedTests
104102

105103
.PHONY: run-tracetesting

docker-compose.yml

-31
Original file line numberDiff line numberDiff line change
@@ -750,37 +750,6 @@ services:
750750
depends_on:
751751
- frontend
752752

753-
# Integration Tests
754-
integrationTests:
755-
image: ${IMAGE_NAME}:${DEMO_VERSION}-integrationTests
756-
container_name: integrationTests
757-
profiles:
758-
- tests
759-
build:
760-
context: ./
761-
dockerfile: ./test/Dockerfile
762-
environment:
763-
- AD_SERVICE_ADDR
764-
- CART_SERVICE_ADDR
765-
- CHECKOUT_SERVICE_ADDR
766-
- CURRENCY_SERVICE_ADDR
767-
- EMAIL_SERVICE_ADDR
768-
- PAYMENT_SERVICE_ADDR
769-
- PRODUCT_CATALOG_SERVICE_ADDR
770-
- RECOMMENDATION_SERVICE_ADDR
771-
- SHIPPING_SERVICE_ADDR
772-
depends_on:
773-
- adservice
774-
- cartservice
775-
- checkoutservice
776-
- currencyservice
777-
- emailservice
778-
- paymentservice
779-
- productcatalogservice
780-
- recommendationservice
781-
- shippingservice
782-
- quoteservice
783-
784753
# Tracebased Tests
785754
traceBasedTests:
786755
image: ${IMAGE_NAME}:${DEMO_VERSION}-traceBasedTests

test/Dockerfile

-26
This file was deleted.

test/README.md

+4-20
Original file line numberDiff line numberDiff line change
@@ -1,26 +1,10 @@
11
# Service Testing
22

3-
There are two ways to test the service APIs in the OpenTelemetry Demo:
3+
The OpenTelemetry Demo uses traced-based testing to validate the
4+
functionality of the services and the traces they generate.
45

5-
1. Using black box-testing, calling gRPC services
6-
and validating their direct response
7-
2. Using Trace-based tests, calling both HTTP and
8-
gRPC services and validating their direct response as well as
9-
the distributed traces they generate
10-
11-
## Testing gRPC services as black boxes
12-
13-
To run the entire test suite as a black box, run the command:
14-
15-
```sh
16-
docker compose run integrationTests
17-
```
18-
19-
If you want to run tests for a specific service, run:
20-
21-
1. Start the services you want to test with `docker compose up --build <service>`
22-
2. Run `npm install`
23-
3. Run `npm test` or `npx ava --match='<pattern>'` to match test names
6+
The trace-based tests will each service and validate the traces they
7+
generate and stored in Jaeger, to a known working trace for the same operation.
248

259
## Testing services with Trace-based tests
2610

test/data.json

-102
This file was deleted.

0 commit comments

Comments
 (0)