fix(pdfengines): return a 400 Bad Request if no metadata to write

pkg/modules/pdfengines/routes.go

@@ -238,6 +238,9 @@ func writeMetadataRoute(engine gotenberg.PdfEngine) api.Route {
 							return fmt.Errorf("unmarshal metadata: %w", err)
 						}
 					}
+					if len(metadata) == 0 {
+						return errors.New("no metadata")
+					}
 					return nil
 				}).
 				Validate()

pkg/modules/pdfengines/routes_test.go

@@ -591,6 +591,25 @@ func TestWriteMetadataHandler(t *testing.T) {
 			expectHttpStatus:       http.StatusBadRequest,
 			expectOutputPathsCount: 0,
 		},
+		{
+			scenario: "no metadata",
+			ctx: func() *api.ContextMock {
+				ctx := &api.ContextMock{Context: new(api.Context)}
+				ctx.SetFiles(map[string]string{
+					"document.docx": "/document.docx",
+				})
+				ctx.SetValues(map[string][]string{
+					"metadata": {
+						"{}",
+					},
+				})
+				return ctx
+			}(),
+			expectError:            true,
+			expectHttpError:        true,
+			expectHttpStatus:       http.StatusBadRequest,
+			expectOutputPathsCount: 0,
+		},
 		{
 			scenario: "error from PDF engine",
 			ctx: func() *api.ContextMock {