Skip to content

Conversation

@ellistarn
Copy link
Contributor

@ellistarn ellistarn commented Mar 7, 2023

Issue #, if available:

Description of changes:

Implements test framework and basic test using HTTP Route.

make e2etest
cd test && go test -v ./... -count=1 --ginkgo.v
?   	github.com/aws/aws-application-networking-k8s/test/pkg/test	[no test files]
=== RUN   TestIntegration
Running Suite: Integration - /Users/etarn/workspaces/go/src/github.com/aws/aws-application-networking-k8s/test/suites/integration
=================================================================================================================================
Random Seed: 1678838640

Will run 1 of 1 specs
------------------------------
HTTPRoute HTTPRules httprules should support multiple paths
/Users/etarn/workspaces/go/src/github.com/aws/aws-application-networking-k8s/test/suites/integration/httproute_test.go:16
    logger.go:130: 2023-03-14T17:04:01.801-0700	INFO	test/framework.go:69	Expecting the test environment to be clean
    logger.go:130: 2023-03-14T17:04:01.917-0700	INFO	test/framework.go:154	Creating *v1beta1.Gateway default/roarhazel-1-bjtvjho2s5
    logger.go:130: 2023-03-14T17:04:01.951-0700	INFO	test/framework.go:154	Creating *v1beta1.HTTPRoute default/chatterlime-6-ypenkaldbz
    logger.go:130: 2023-03-14T17:04:01.983-0700	INFO	test/framework.go:154	Creating *v1.Service default/jesterslime-3-37ihj2ashd
    logger.go:130: 2023-03-14T17:04:02.018-0700	INFO	test/framework.go:154	Creating *v1.Deployment default/faceplain-2-1h5xirfudw
    logger.go:130: 2023-03-14T17:04:02.046-0700	INFO	test/framework.go:154	Creating *v1.Service default/antelopespangle-5-x5ouiedgfm
    logger.go:130: 2023-03-14T17:04:02.080-0700	INFO	test/framework.go:154	Creating *v1.Deployment default/chillerwild-4-csx9kaamqh
    logger.go:130: 2023-03-14T17:05:48.849-0700	INFO	test/framework.go:69	Expecting the test environment to be clean
• [147.750 seconds]
------------------------------
[AfterSuite]
/Users/etarn/workspaces/go/src/github.com/aws/aws-application-networking-k8s/test/pkg/test/framework.go:63
    logger.go:130: 2023-03-14T17:06:29.551-0700	INFO	test/framework.go:83	Cleaning the test environment
    logger.go:130: 2023-03-14T17:06:30.796-0700	INFO	test/framework.go:69	Expecting the test environment to be clean
[AfterSuite] PASSED [1.334 seconds]
------------------------------

Ran 1 of 1 Specs in 149.083 seconds
SUCCESS! -- 1 Passed | 0 Failed | 0 Pending | 0 Skipped
--- PASS: TestIntegration (150.54s)
PASS
ok  	github.com/aws/aws-application-networking-k8s/test/suites/integration	151.158s

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@ellistarn ellistarn marked this pull request as draft March 7, 2023 01:42
@ellistarn ellistarn force-pushed the e2e branch 7 times, most recently from 4f8350e to ad99512 Compare March 13, 2023 15:22
@ellistarn ellistarn changed the title [WIP] test: Implemented e2e test framework Implemented e2e test framework Mar 13, 2023
@ellistarn ellistarn changed the title Implemented e2e test framework test: Implemented e2e test framework Mar 13, 2023
@ellistarn ellistarn marked this pull request as ready for review March 13, 2023 15:23
@ellistarn ellistarn force-pushed the e2e branch 10 times, most recently from a343b99 to f2d2c5a Compare March 13, 2023 19:53
Copy link
Contributor

Choose a reason for hiding this comment

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

can you add annotation here? Here is one example.

@liwenwu-amazon
Copy link
Contributor

Is it possible to also test the e2e traffic?

@liwenwu-amazon
Copy link
Contributor

liwenwu-amazon commented Mar 15, 2023

Can you also add a README.md under test or modify existing README.md? thanks

Copy link
Contributor

Choose a reason for hiding this comment

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

Today, we only support HTTP. We need to add GRPC and HTTP2 in future

Copy link
Contributor

Choose a reason for hiding this comment

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

yes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants