changed few formatting issues

Signed-off-by: vivekr-splunk <94569031+vivekr-splunk@users.noreply.github.com>
splunk · Sep 13, 2023 · ad0882a · ad0882a
1 parent fed6f8a
commit ad0882a
Show file tree

Hide file tree

Showing 5 changed files with 6 additions and 7 deletions.
diff --git a/pkg/provisioner/splunk/implementation/license.go b/pkg/provisioner/splunk/implementation/license.go
@@ -59,8 +59,8 @@ func (p *splunkProvisioner) GetLicenseLocalPeer(ctx context.Context, conditions
 	return result, err
 }
 
-// GetClusterManagerStatus Access cluster node configuration details.
-func (p *splunkProvisioner) GetLicense(ctx context.Context, conditions *[]metav1.Condition) (result provmodel.Result, err error) {
+// GetLicenseStatus Access cluster node configuration details.
+func (p *splunkProvisioner) GetLicenseStatus(ctx context.Context, conditions *[]metav1.Condition) (result provmodel.Result, err error) {
 	_, err = callLicense(ctx, p)
 	lslistptr, err := callLicense(ctx, p)
 	if err != nil {

diff --git a/pkg/provisioner/splunk/implementation/license_test.go b/pkg/provisioner/splunk/implementation/license_test.go
@@ -20,7 +20,7 @@ func TestGetGetLicense(t *testing.T) {
 
 	ctx := context.TODO()
 
-	_, err := provisioner.GetLicense(ctx, conditions)
+	_, err := provisioner.GetLicenseStatus(ctx, conditions)
 	if err != nil {
 		t.Errorf("fixture: error in set cluster manager %v", err)
 	}

diff --git a/pkg/provisioner/splunk/provisioner.go b/pkg/provisioner/splunk/provisioner.go
@@ -33,5 +33,5 @@ type Provisioner interface {
 	// GetLicenseLocalPeer
 	GetLicenseLocalPeer(ctx context.Context, conditions *[]metav1.Condition) (result provmodel.Result, err error)
 
-	GetLicense(ctx context.Context, conditions *[]metav1.Condition) (result provmodel.Result, err error)
+	GetLicenseStatus(ctx context.Context, conditions *[]metav1.Condition) (result provmodel.Result, err error)
 }
diff --git a/pkg/splunk/enterprise/licensemanager.go b/pkg/splunk/enterprise/licensemanager.go
@@ -183,7 +183,7 @@ func (p *splunkManager) ApplyLicenseManager(ctx context.Context, client splcommo
 		// Verification of splunk instance update CR status
 		// We are using Conditions to update status information
 		provResult := provmodel.Result{}
-		provResult, err = p.provisioner.GetLicense(ctx, &cr.Status.Conditions)
+		provResult, err = p.provisioner.GetLicenseStatus(ctx, &cr.Status.Conditions)
 		if err != nil {
 			cr.Status.ErrorMessage = provResult.ErrorMessage
 		}

diff --git a/test/licensemanager/lm_s1_test.go b/test/licensemanager/lm_s1_test.go
@@ -58,8 +58,7 @@ var _ = Describe("Licensemanager test", func() {
 			// Download License File
 			downloadDir := "licenseFolder"
 			switch testenv.ClusterProvider {
-			case "eks":
-				licenseFilePath, err := testenv.DownloadLicenseFromS3Bucket()
+			case "eks":				licenseFilePath, err := testenv.DownloadLicenseFromS3Bucket()
 				Expect(err).To(Succeed(), "Unable to download license file from S3")
 				// Create License Config Map
 				testcaseEnvInst.CreateLicenseConfigMap(licenseFilePath)