azurerm_dashboard_grafana: add support for SMTP

[jiaweitao001]

• Add support for SMTP configuration.

• fix Support for SMTP settings in azurerm_dashboard_grafana #24635

--- PASS: TestAccDashboardGrafana_complete (751.00s)
PASS

[katbyte]

i don't think we need this block? configuration is implied? could we remove this and bring up smtp a level?

[jiaweitao001]

Will do.

[katbyte]

Looks like we have a test failure

------- Stdout: -------
=== RUN   TestAccDashboardGrafana_basic
    testcase.go:120: Step 1/2 error: After applying this test step, the plan was not empty.
        stdout:
        
        
        Terraform used the selected providers to generate the following execution
        plan. Resource actions are indicated with the following symbols:
          ~ update in-place
        
        Terraform will perform the following actions:
        
          # azurerm_dashboard_grafana.test will be updated in-place
          ~ resource "azurerm_dashboard_grafana" "test" {
                id                                     = "/subscriptions/*******/resourceGroups/acctest-rg-240201193403749019/providers/Microsoft.Dashboard/grafana/a-dg-240201193403749019"
                name                                   = "a-dg-240201193403749019"
                # (12 unchanged attributes hidden)
        
              - smtp {
                  - enabled                   = false -> null
                  - verification_skip_enabled = false -> null
                }
            }
        
        Plan: 0 to add, 1 to change, 0 to destroy.
--- FAIL: TestAccDashboardGrafana_basic (600.71s)
FAIL

[katbyte]

there is still a test failure?

------- Stdout: -------
=== RUN   TestAccDashboardGrafana_update
    testcase.go:120: Step 2/4 error running import: ImportStateVerify attributes not equivalent. Difference is shown below. The - symbol indicates attributes missing after import.
        
          map[string]string{
        - 	"smtp.0.password": "password",
        + 	"smtp.0.password": "",
          }
--- FAIL: TestAccDashboardGrafana_update (632.32s)
FAIL

[jiaweitao001]

Tests are passing as they should.

[katbyte]

LGTM ✉️

[github-actions]

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.