fix default metadata on postings accounts

Author: Azorlogh

internal/controller/ledger/controller_default.go

@@ -428,12 +428,23 @@ func (ctrl *DefaultController) createTransaction(ctx context.Context, store Stor
 			for k, v := range values {
 				accountMetadata[account][k] = v
 			}
-
 			if schema != nil {
 				schema.Chart.InsertDefaultAccountMetadata(account, accountMetadata[account])
 			}
 		}
 	}
+	if schema != nil {
+		for _, posting := range result.Postings {
+			if accountMetadata[posting.Source] == nil {
+				accountMetadata[posting.Source] = metadata.Metadata{}
+			}
+			schema.Chart.InsertDefaultAccountMetadata(posting.Source, accountMetadata[posting.Source])
+			if accountMetadata[posting.Destination] == nil {
+				accountMetadata[posting.Destination] = metadata.Metadata{}
+			}
+			schema.Chart.InsertDefaultAccountMetadata(posting.Destination, accountMetadata[posting.Destination])
+		}
+	}
 
 	transaction := ledger.NewTransaction().
 		WithPostings(result.Postings...).

internal/controller/ledger/controller_default_test.go

@@ -90,9 +90,18 @@ func testCreateTransaction(t *testing.T, withSchema bool) {
 			Return(nil, nil)
 	}
 
-	store.EXPECT().
-		CommitTransaction(gomock.Any(), gomock.Any(), map[string]metadata.Metadata{}).
-		Return(nil)
+	if withSchema {
+		store.EXPECT().
+			CommitTransaction(gomock.Any(), gomock.Any(), map[string]metadata.Metadata{
+				"world": {},
+				"bank":  {},
+			}).
+			Return(nil)
+	} else {
+		store.EXPECT().
+			CommitTransaction(gomock.Any(), gomock.Any(), map[string]metadata.Metadata{}).
+			Return(nil)
+	}
 
 	store.EXPECT().
 		InsertLog(gomock.Any(), gomock.Cond(func(x any) bool {

test/e2e/api_schema_test.go

@@ -313,7 +313,7 @@ var _ = Context("Ledger schema API tests", func() {
 						V2PostTransaction: components.V2PostTransaction{
 							Force: pointer.For(true),
 							AccountMetadata: map[string]map[string]string{
-								"users:001": {
+								"users:002": {
 									"bar": "test2",
 								},
 							},
@@ -335,11 +335,22 @@ var _ = Context("Ledger schema API tests", func() {
 					})
 					Expect(err).To(BeNil())
 
+					// involved in a posting
 					res, err := Wait(specContext, DeferClient(testServer)).Ledger.V2.GetAccount(ctx, operations.V2GetAccountRequest{
 						Ledger:  "default",
 						Address: "users:001",
 					})
 					Expect(err).To(BeNil())
+					Expect(res.V2AccountResponse.Data.Metadata).To(Equal(map[string]string{
+						"foo": "test",
+					}))
+
+					// involved in accountMetadata
+					res, err = Wait(specContext, DeferClient(testServer)).Ledger.V2.GetAccount(ctx, operations.V2GetAccountRequest{
+						Ledger:  "default",
+						Address: "users:002",
+					})
+					Expect(err).To(BeNil())
 					Expect(res.V2AccountResponse.Data.Metadata).To(Equal(map[string]string{
 						"foo": "test",
 						"bar": "test2",