Add conformance annotations for expansion and service tests

Signed-off-by: Brad Topol <btopol@us.ibm.com>
This commit is contained in:
Brad Topol 2017-10-20 13:29:25 -07:00
parent 7d4f21d0d7
commit 6e3a57f739
2 changed files with 29 additions and 0 deletions

View File

@ -31,6 +31,11 @@ import (
var _ = framework.KubeDescribe("Variable Expansion", func() { var _ = framework.KubeDescribe("Variable Expansion", func() {
f := framework.NewDefaultFramework("var-expansion") f := framework.NewDefaultFramework("var-expansion")
/*
Testname: var-expansion-env
Description: Make sure environment variables can be set using an
expansion of previously defined environment variables
*/
It("should allow composing env vars into new env vars [Conformance]", func() { It("should allow composing env vars into new env vars [Conformance]", func() {
podName := "var-expansion-" + string(uuid.NewUUID()) podName := "var-expansion-" + string(uuid.NewUUID())
pod := &v1.Pod{ pod := &v1.Pod{
@ -71,6 +76,11 @@ var _ = framework.KubeDescribe("Variable Expansion", func() {
}) })
}) })
/*
Testname: var-expansion-command
Description: Make sure a container's commands can be set using an
expansion of environment variables.
*/
It("should allow substituting values in a container's command [Conformance]", func() { It("should allow substituting values in a container's command [Conformance]", func() {
podName := "var-expansion-" + string(uuid.NewUUID()) podName := "var-expansion-" + string(uuid.NewUUID())
pod := &v1.Pod{ pod := &v1.Pod{
@ -101,6 +111,11 @@ var _ = framework.KubeDescribe("Variable Expansion", func() {
}) })
}) })
/*
Testname: var-expansion-arg
Description: Make sure a container's args can be set using an
expansion of environment variables.
*/
It("should allow substituting values in a container's args [Conformance]", func() { It("should allow substituting values in a container's args [Conformance]", func() {
podName := "var-expansion-" + string(uuid.NewUUID()) podName := "var-expansion-" + string(uuid.NewUUID())
pod := &v1.Pod{ pod := &v1.Pod{

View File

@ -65,11 +65,20 @@ var _ = SIGDescribe("Services", func() {
// TODO: We get coverage of TCP/UDP and multi-port services through the DNS test. We should have a simpler test for multi-port TCP here. // TODO: We get coverage of TCP/UDP and multi-port services through the DNS test. We should have a simpler test for multi-port TCP here.
/*
Testname: service-kubernetes-exists
Description: Make sure kubernetes service does exist.
*/
It("should provide secure master service [Conformance]", func() { It("should provide secure master service [Conformance]", func() {
_, err := cs.CoreV1().Services(metav1.NamespaceDefault).Get("kubernetes", metav1.GetOptions{}) _, err := cs.CoreV1().Services(metav1.NamespaceDefault).Get("kubernetes", metav1.GetOptions{})
Expect(err).NotTo(HaveOccurred()) Expect(err).NotTo(HaveOccurred())
}) })
/*
Testname: service-valid-endpoints
Description: Ensure a service with no pod, one pod or two pods has
valid/accessible endpoints (same port number for service and pods).
*/
It("should serve a basic endpoint from pods [Conformance]", func() { It("should serve a basic endpoint from pods [Conformance]", func() {
// TODO: use the ServiceTestJig here // TODO: use the ServiceTestJig here
serviceName := "endpoint-test2" serviceName := "endpoint-test2"
@ -130,6 +139,11 @@ var _ = SIGDescribe("Services", func() {
framework.ValidateEndpointsOrFail(cs, ns, serviceName, framework.PortsByPodName{}) framework.ValidateEndpointsOrFail(cs, ns, serviceName, framework.PortsByPodName{})
}) })
/*
Testname: service-valid-endpoints-multiple-ports
Description: Ensure a service with no pod, one pod or two pods has
valid/accessible endpoints (different port number for pods).
*/
It("should serve multiport endpoints from pods [Conformance]", func() { It("should serve multiport endpoints from pods [Conformance]", func() {
// TODO: use the ServiceTestJig here // TODO: use the ServiceTestJig here
// repacking functionality is intentionally not tested here - it's better to test it in an integration test. // repacking functionality is intentionally not tested here - it's better to test it in an integration test.