Output service name when creating Operator backed service (redhat-dev…

…eloper#4336)

* Output service name when creating Operator backed service

* Check for service name in tests

pkg/odo/cli/service/create.go

@@ -377,6 +377,13 @@ func (o *ServiceCreateOptions) Validate() (err error) {
 			// CRD is valid. We can use it further to create a service from it.
 			o.CustomResourceDefinition = d.OriginalCRD
 
+			if o.ServiceName == "" {
+				o.ServiceName, err = d.getServiceNameFromCRD()
+				if err != nil {
+					return err
+				}
+			}
+
 			return nil
 		} else {
 			// This block is executed only when user has neither provided a

tests/integration/operatorhub/cmd_service_test.go

@@ -77,7 +77,8 @@ var _ = Describe("odo service command tests for OperatorHub", func() {
 		It("should be able to create, list and then delete EtcdCluster from its alm example", func() {
 			operators := helper.CmdShouldPass("odo", "catalog", "list", "services")
 			etcdOperator := regexp.MustCompile(`etcdoperator\.*[a-z][0-9]\.[0-9]\.[0-9]-clusterwide`).FindString(operators)
-			helper.CmdShouldPass("odo", "service", "create", fmt.Sprintf("%s/EtcdCluster", etcdOperator), "--project", project)
+			stdOut := helper.CmdShouldPass("odo", "service", "create", fmt.Sprintf("%s/EtcdCluster", etcdOperator), "--project", project)
+			Expect(stdOut).To(ContainSubstring("Service 'example' was created"))
 
 			// now verify if the pods for the operator have started
 			pods := helper.CmdShouldPass("oc", "get", "pods", "-n", project)
@@ -90,7 +91,7 @@ var _ = Describe("odo service command tests for OperatorHub", func() {
 			})
 
 			// now test listing of the service using odo
-			stdOut := helper.CmdShouldPass("odo", "service", "list")
+			stdOut = helper.CmdShouldPass("odo", "service", "list")
 			Expect(stdOut).To(ContainSubstring("EtcdCluster/example"))
 
 			// now test the deletion of the service using odo