Merge pull request #216 from karthich/admin-recommended-sprintf-fix

Fixing the sprintf formatting

Author: seansain

sumologic/data_source_sumologic_admin_recommended_folder_test.go

@@ -0,0 +1,34 @@
+package sumologic
+
+import (
+	"testing"
+
+	"github.com/hashicorp/terraform-plugin-sdk/helper/resource"
+)
+
+func TestAccDataSourceAdminRecommendedFolder_basic(t *testing.T) {
+	resource.Test(t, resource.TestCase{
+		Providers: testAccProviders,
+		Steps: []resource.TestStep{
+			{
+				Config: testDataSourceSumologicAdminRecommendedFolderConfig,
+				Check: resource.ComposeTestCheckFunc(
+					testAccDataSourceAdminRecommendedFolderCheck("data.sumologic_admin_recommended_folder.test"),
+				),
+			},
+		},
+	})
+}
+
+func testAccDataSourceAdminRecommendedFolderCheck(name string) resource.TestCheckFunc {
+	return resource.ComposeTestCheckFunc(
+		resource.TestCheckResourceAttrSet(name, "id"),
+		resource.TestCheckResourceAttr(name, "name", "Admin Recommended"),
+		resource.TestCheckResourceAttrSet(name, "description"),
+	)
+}
+
+var testDataSourceSumologicAdminRecommendedFolderConfig = `
+data "sumologic_admin_recommended_folder" "test" {
+}
+`

sumologic/sumologic_folder.go

@@ -160,7 +160,7 @@ func (s *Client) getAdminRecommendedFolder(timeout time.Duration) (*Folder, erro
 	}
 	log.Printf("JobId: %s", jid.ID)
 
-	url = fmt.Sprintf("v2/content/folders/adminRecommended/#{jid.ID}/status")
+	url = fmt.Sprintf("v2/content/folders/adminRecommended/%s/status", jid.ID)
 	log.Printf("Admin recommended folder job status url: %s", url)
 
 	//Ensure the job has completed before proceeding
@@ -171,7 +171,7 @@ func (s *Client) getAdminRecommendedFolder(timeout time.Duration) (*Folder, erro
 	}
 
 	// Request the results of the job
-	url = fmt.Sprintf("v2/content/folders/adminRecommended/#{jid.ID}/result")
+	url = fmt.Sprintf("v2/content/folders/adminRecommended/%s/result", jid.ID)
 	log.Printf("Admin recommended folder job result url: %s", url)
 
 	rawContent, _, err := s.Get(url, false)