Skip to content

Commit 4e050d3

Browse files
salonichf5salonichf5
committed
enable tests
1 parent 6666868 commit 4e050d3

File tree

2 files changed

+106
-104
lines changed

2 files changed

+106
-104
lines changed

‎config/tests/static-deployment.yaml‎

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -6,9 +6,11 @@ metadata:
66
name: nginx-gateway
77
namespace: nginx-gateway
88
labels:
9+
helm.sh/chart: nginx-gateway-fabric-1.4.0
910
app.kubernetes.io/name: nginx-gateway
1011
app.kubernetes.io/instance: nginx-gateway
1112
app.kubernetes.io/version: "edge"
13+
app.kubernetes.io/managed-by: Helm
1214
spec:
1315
replicas: 1
1416
selector:
@@ -23,8 +25,8 @@ spec:
2325
spec:
2426
initContainers:
2527
- name: copy-nginx-config
26-
image: ghcr.io/nginxinc/nginx-gateway-fabric:edge
27-
imagePullPolicy: Always
28+
image: nginx-gateway-fabric:sa.choudhary
29+
imagePullPolicy: Never
2830
command:
2931
- /usr/bin/gateway
3032
- copy
@@ -72,8 +74,8 @@ spec:
7274
valueFrom:
7375
fieldRef:
7476
fieldPath: metadata.name
75-
image: ghcr.io/nginxinc/nginx-gateway-fabric:edge
76-
imagePullPolicy: Always
77+
image: nginx-gateway-fabric:sa.choudhary
78+
imagePullPolicy: Never
7779
name: nginx-gateway
7880
ports:
7981
- name: health
@@ -109,8 +111,8 @@ spec:
109111
mountPath: /var/run/nginx
110112
- name: nginx-includes
111113
mountPath: /etc/nginx/includes
112-
- image: ghcr.io/nginxinc/nginx-gateway-fabric/nginx:edge
113-
imagePullPolicy: Always
114+
- image: nginx-gateway-fabric/nginx:sa.choudhary
115+
imagePullPolicy: Never
114116
name: nginx
115117
ports:
116118
- containerPort: 80

‎tests/suite/upstream_settings_test.go‎

Lines changed: 98 additions & 98 deletions
Original file line numberDiff line numberDiff line change
@@ -81,32 +81,32 @@ var _ = Describe("UpstreamSettingsPolicy", Ordered, Label("functional", "uspolic
8181
}
8282
})
8383

84-
// Context("verify working traffic", func() {
85-
// It("should return a 200 response for HTTPRoutes", func() {
86-
// port := 80
87-
// if portFwdPort != 0 {
88-
// port = portFwdPort
89-
// }
90-
// baseCoffeeURL := fmt.Sprintf("http://cafe.example.com:%d%s", port, "/coffee")
91-
// baseTeaURL := fmt.Sprintf("http://cafe.example.com:%d%s", port, "/tea")
92-
93-
// Eventually(
94-
// func() error {
95-
// return expectRequestToSucceed(baseCoffeeURL, address, "URI: /coffee")
96-
// }).
97-
// WithTimeout(timeoutConfig.RequestTimeout).
98-
// WithPolling(500 * time.Millisecond).
99-
// Should(Succeed())
100-
101-
// Eventually(
102-
// func() error {
103-
// return expectRequestToSucceed(baseTeaURL, address, "URI: /tea")
104-
// }).
105-
// WithTimeout(timeoutConfig.RequestTimeout).
106-
// WithPolling(500 * time.Millisecond).
107-
// Should(Succeed())
108-
// })
109-
// })
84+
Context("verify working traffic", func() {
85+
It("should return a 200 response for HTTPRoutes", func() {
86+
port := 80
87+
if portFwdPort != 0 {
88+
port = portFwdPort
89+
}
90+
baseCoffeeURL := fmt.Sprintf("http://cafe.example.com:%d%s", port, "/coffee")
91+
baseTeaURL := fmt.Sprintf("http://cafe.example.com:%d%s", port, "/tea")
92+
93+
Eventually(
94+
func() error {
95+
return expectRequestToSucceed(baseCoffeeURL, address, "URI: /coffee")
96+
}).
97+
WithTimeout(timeoutConfig.RequestTimeout).
98+
WithPolling(500 * time.Millisecond).
99+
Should(Succeed())
100+
101+
Eventually(
102+
func() error {
103+
return expectRequestToSucceed(baseTeaURL, address, "URI: /tea")
104+
}).
105+
WithTimeout(timeoutConfig.RequestTimeout).
106+
WithPolling(500 * time.Millisecond).
107+
Should(Succeed())
108+
})
109+
})
110110

111111
Context("nginx directives", func() {
112112
var conf *framework.Payload
@@ -128,78 +128,78 @@ var _ = Describe("UpstreamSettingsPolicy", Ordered, Label("functional", "uspolic
128128
Expect(framework.ValidateNginxFieldExists(conf, expCfg)).To(Succeed())
129129
}
130130
},
131-
// Entry("HTTP upstreams", []framework.ExpectedNginxField{
132-
// {
133-
// Directive: "upstream",
134-
// Value: "uspolicy_coffee_80",
135-
// File: "http.conf",
136-
// },
137-
// {
138-
// Directive: "upstream",
139-
// Value: "uspolicy_tea_80",
140-
// File: "http.conf",
141-
// },
142-
// {
143-
// Directive: "zone",
144-
// Value: "uspolicy_coffee_80 512k",
145-
// Upstream: "uspolicy_coffee_80",
146-
// File: "http.conf",
147-
// },
148-
// {
149-
// Directive: "zone",
150-
// Value: "uspolicy_tea_80 512k",
151-
// Upstream: "uspolicy_tea_80",
152-
// File: "http.conf",
153-
// },
154-
// {
155-
// Directive: "keepalive",
156-
// Value: "10",
157-
// Upstream: "uspolicy_coffee_80",
158-
// File: "http.conf",
159-
// },
160-
// {
161-
// Directive: "keepalive",
162-
// Value: "10",
163-
// Upstream: "uspolicy_tea_80",
164-
// File: "http.conf",
165-
// },
166-
// {
167-
// Directive: "keepalive_requests",
168-
// Value: "3",
169-
// Upstream: "uspolicy_coffee_80",
170-
// File: "http.conf",
171-
// },
172-
// {
173-
// Directive: "keepalive_requests",
174-
// Value: "3",
175-
// Upstream: "uspolicy_tea_80",
176-
// File: "http.conf",
177-
// },
178-
// {
179-
// Directive: "keepalive_time",
180-
// Value: "10s",
181-
// Upstream: "uspolicy_coffee_80",
182-
// File: "http.conf",
183-
// },
184-
// {
185-
// Directive: "keepalive_time",
186-
// Value: "10s",
187-
// Upstream: "uspolicy_tea_80",
188-
// File: "http.conf",
189-
// },
190-
// {
191-
// Directive: "keepalive_timeout",
192-
// Value: "50s",
193-
// Upstream: "uspolicy_coffee_80",
194-
// File: "http.conf",
195-
// },
196-
// {
197-
// Directive: "keepalive_timeout",
198-
// Value: "50s",
199-
// Upstream: "uspolicy_tea_80",
200-
// File: "http.conf",
201-
// },
202-
// }),
131+
Entry("HTTP upstreams", []framework.ExpectedNginxField{
132+
{
133+
Directive: "upstream",
134+
Value: "uspolicy_coffee_80",
135+
File: "http.conf",
136+
},
137+
{
138+
Directive: "upstream",
139+
Value: "uspolicy_tea_80",
140+
File: "http.conf",
141+
},
142+
{
143+
Directive: "zone",
144+
Value: "uspolicy_coffee_80 512k",
145+
Upstream: "uspolicy_coffee_80",
146+
File: "http.conf",
147+
},
148+
{
149+
Directive: "zone",
150+
Value: "uspolicy_tea_80 512k",
151+
Upstream: "uspolicy_tea_80",
152+
File: "http.conf",
153+
},
154+
{
155+
Directive: "keepalive",
156+
Value: "10",
157+
Upstream: "uspolicy_coffee_80",
158+
File: "http.conf",
159+
},
160+
{
161+
Directive: "keepalive",
162+
Value: "10",
163+
Upstream: "uspolicy_tea_80",
164+
File: "http.conf",
165+
},
166+
{
167+
Directive: "keepalive_requests",
168+
Value: "3",
169+
Upstream: "uspolicy_coffee_80",
170+
File: "http.conf",
171+
},
172+
{
173+
Directive: "keepalive_requests",
174+
Value: "3",
175+
Upstream: "uspolicy_tea_80",
176+
File: "http.conf",
177+
},
178+
{
179+
Directive: "keepalive_time",
180+
Value: "10s",
181+
Upstream: "uspolicy_coffee_80",
182+
File: "http.conf",
183+
},
184+
{
185+
Directive: "keepalive_time",
186+
Value: "10s",
187+
Upstream: "uspolicy_tea_80",
188+
File: "http.conf",
189+
},
190+
{
191+
Directive: "keepalive_timeout",
192+
Value: "50s",
193+
Upstream: "uspolicy_coffee_80",
194+
File: "http.conf",
195+
},
196+
{
197+
Directive: "keepalive_timeout",
198+
Value: "50s",
199+
Upstream: "uspolicy_tea_80",
200+
File: "http.conf",
201+
},
202+
}),
203203
Entry("GRPC upstreams", []framework.ExpectedNginxField{
204204
{
205205
Directive: "zone",

0 commit comments

Comments
 (0)