Fix live tests for azappconfig (Azure#19921)

Recent feature introduced some bugs in a live test.

Author: jhendrixMSFT

sdk/data/azappconfig/client.go

@@ -150,6 +150,7 @@ func fromGeneratedAdd(g generated.AzureAppConfigurationClientPutKeyValueResponse
 type AddSettingOptions struct {
 	// Configuration setting label.
 	Label *string
+
 	// Configuration setting content type.
 	ContentType *string
 }
@@ -397,6 +398,7 @@ func fromGeneratedSet(g generated.AzureAppConfigurationClientPutKeyValueResponse
 type SetSettingOptions struct {
 	// Configuration setting label.
 	Label *string
+
 	// Configuration setting content type.
 	ContentType *string
 

sdk/data/azappconfig/client_test.go

@@ -92,11 +92,10 @@ func TestClient(t *testing.T) {
 	require.NotEmpty(t, setResp2)
 	require.NotNil(t, setResp2.Key)
 	require.NotNil(t, setResp2.Label)
-	require.NotNil(t, setResp2.ContentType)
+	require.Nil(t, setResp2.ContentType)
 	require.NotNil(t, setResp2.Value)
 	require.Equal(t, key, *setResp2.Key)
 	require.Equal(t, label, *setResp2.Label)
-	require.Equal(t, contentType, *setResp2.ContentType)
 	require.Equal(t, value, *setResp2.Value)
 
 	setResp3, err8 := client.SetSetting(context.TODO(), key, &value, &SetSettingOptions{Label: &label, OnlyIfUnchanged: etag})
@@ -108,12 +107,11 @@ func TestClient(t *testing.T) {
 	require.NotEmpty(t, roResp)
 	require.NotNil(t, roResp.Key)
 	require.NotNil(t, roResp.Label)
-	require.NotNil(t, roResp.ContentType)
+	require.Nil(t, roResp.ContentType)
 	require.NotNil(t, roResp.Value)
 	require.NotNil(t, roResp.IsReadOnly)
 	require.Equal(t, key, *roResp.Key)
 	require.Equal(t, label, *roResp.Label)
-	require.Equal(t, contentType, *roResp.ContentType)
 	require.Equal(t, value, *roResp.Value)
 	require.True(t, *roResp.IsReadOnly)
 
@@ -122,12 +120,11 @@ func TestClient(t *testing.T) {
 	require.NotEmpty(t, roResp2)
 	require.NotNil(t, roResp2.Key)
 	require.NotNil(t, roResp2.Label)
-	require.NotNil(t, roResp2.ContentType)
+	require.Nil(t, roResp2.ContentType)
 	require.NotNil(t, roResp2.Value)
 	require.NotNil(t, roResp2.IsReadOnly)
 	require.Equal(t, key, *roResp2.Key)
 	require.Equal(t, label, *roResp2.Label)
-	require.Equal(t, contentType, *roResp2.ContentType)
 	require.Equal(t, value, *roResp2.Value)
 	require.False(t, *roResp2.IsReadOnly)
 
@@ -141,12 +138,11 @@ func TestClient(t *testing.T) {
 	require.NotEmpty(t, roResp4)
 	require.NotNil(t, roResp4.Key)
 	require.NotNil(t, roResp4.Label)
-	require.NotNil(t, roResp4.ContentType)
+	require.Nil(t, roResp4.ContentType)
 	require.NotNil(t, roResp4.Value)
 	require.NotNil(t, roResp4.IsReadOnly)
 	require.Equal(t, key, *roResp4.Key)
 	require.Equal(t, label, *roResp4.Label)
-	require.Equal(t, contentType, *roResp4.ContentType)
 	require.Equal(t, value, *roResp4.Value)
 	require.True(t, *roResp4.IsReadOnly)
 
@@ -160,12 +156,11 @@ func TestClient(t *testing.T) {
 	require.NotEmpty(t, roResp6)
 	require.NotNil(t, roResp6.Key)
 	require.NotNil(t, roResp6.Label)
-	require.NotNil(t, roResp6.ContentType)
+	require.Nil(t, roResp6.ContentType)
 	require.NotNil(t, roResp6.Value)
 	require.NotNil(t, roResp6.IsReadOnly)
 	require.Equal(t, key, *roResp6.Key)
 	require.Equal(t, label, *roResp6.Label)
-	require.Equal(t, contentType, *roResp6.ContentType)
 	require.Equal(t, value, *roResp6.Value)
 	require.False(t, *roResp6.IsReadOnly)
 
@@ -179,7 +174,6 @@ func TestClient(t *testing.T) {
 	require.NotEmpty(t, revResp)
 	require.Equal(t, key, *revResp.Settings[0].Key)
 	require.Equal(t, label, *revResp.Settings[0].Label)
-	require.Equal(t, contentType, *revResp.Settings[0].ContentType)
 
 	settsPgr := client.NewListSettingsPager(SettingSelector{KeyFilter: &any, LabelFilter: &any, Fields: AllSettingFields()}, nil)
 	require.NotEmpty(t, settsPgr)
@@ -190,7 +184,6 @@ func TestClient(t *testing.T) {
 	require.NotEmpty(t, settsResp)
 	require.Equal(t, key, *settsResp.Settings[0].Key)
 	require.Equal(t, label, *settsResp.Settings[0].Label)
-	require.Equal(t, contentType, *settsResp.Settings[0].ContentType)
 	require.Equal(t, value, *settsResp.Settings[0].Value)
 	require.False(t, *settsResp.Settings[0].IsReadOnly)
 
@@ -199,14 +192,13 @@ func TestClient(t *testing.T) {
 	require.NotEmpty(t, delResp)
 	require.NotNil(t, delResp.Key)
 	require.NotNil(t, delResp.Label)
-	require.NotNil(t, delResp.ContentType)
+	require.Nil(t, delResp.ContentType)
 	require.NotNil(t, delResp.Value)
 	require.Equal(t, key, *delResp.Key)
 	require.Equal(t, label, *delResp.Label)
-	require.Equal(t, contentType, *delResp.ContentType)
 	require.Equal(t, value, *delResp.Value)
 
-	addResp2, err18 := client.AddSetting(context.TODO(), key, &value, &AddSettingOptions{Label: &label})
+	addResp2, err18 := client.AddSetting(context.TODO(), key, &value, &AddSettingOptions{Label: &label, ContentType: &contentType})
 	require.NoError(t, err18)
 	require.NotEmpty(t, addResp2)
 	require.NotNil(t, addResp2.Key)