testserver: add missing synchronization (#2932)

Follow up to #2816

Author: denik

libs/testserver/apps.go

@@ -19,6 +19,8 @@ func (s *FakeWorkspace) AppsUpsert(req Request, name string) Response {
 		}
 	}
 
+	defer s.LockUnlock()()
+
 	if name != "" {
 		_, ok := s.Apps[name]
 		if !ok {

libs/testserver/quality_monitors.go

@@ -27,6 +27,8 @@ func (s *FakeWorkspace) QualityMonitorUpsert(req Request, tableName string, chec
 		}
 	}
 
+	defer s.LockUnlock()()
+
 	if checkExists {
 		_, ok := s.Monitors[tableName]
 		if !ok {

libs/testserver/schemas.go

@@ -10,6 +10,8 @@ import (
 )
 
 func (s *FakeWorkspace) SchemasCreate(req Request) Response {
+	defer s.LockUnlock()()
+
 	var schema catalog.SchemaInfo
 
 	if err := json.Unmarshal(req.Body, &schema); err != nil {
@@ -27,6 +29,8 @@ func (s *FakeWorkspace) SchemasCreate(req Request) Response {
 }
 
 func (s *FakeWorkspace) SchemasUpdate(req Request, name string) Response {
+	defer s.LockUnlock()()
+
 	existing, ok := s.Schemas[name]
 	if !ok {
 		return Response{

libs/testserver/volumes.go

@@ -8,7 +8,7 @@ import (
 	"github.com/databricks/databricks-sdk-go/service/catalog"
 )
 
-func (s *FakeWorkspace) VolumesCreate(req Request) Response {
+func (*FakeWorkspace) VolumesCreate(req Request) Response {
 	var volume catalog.VolumeInfo
 
 	if err := json.Unmarshal(req.Body, &volume); err != nil {