personal-finance/datastore
2
0
Fork 0
Code Issues 12 Pull Requests Actions Packages Releases Wiki Activity

Compare commits

base: personal-finance:c8a4ff76e2568c9fcb5cd7f85c0e17cdd4d713c3
Branches Tags
personal-finance:main
..
compare: personal-finance:35b35f5e4058dc6095df7f4708adea05f66f6464
Branches Tags
personal-finance:main

2 Commits
c8a4ff76e2 ... 35b35f5e40

Author SHA1 Message Date
Luís Murta
35b35f5e40
Implements /transactions PUT method 
Adds PUT method to OpenAPI spec.

Given that the transaction IDs are generated on server-side, for the PUT
method to remain idempotent, it can only update existing transactions.

It also adds a TransactionExists method on the DAL.

Issue: #20
 2024-06-22 21:51:03 +01:00
Luís Murta
4c82ca0e2b
Implements /transactions POST method 
It adds the method to the OpenAPI spec and generates a new server
config. The requirement for the ID on the Transaction component is
removed, so that it can be reused for insertions.
It also adds two new middlewares, a logging and a spec validator. If a
request does not follow the spec, a 400 is returned immediately.

Issue: #18
 2024-06-22 20:11:52 +01:00
2 changed files with 2 additions and 2 deletions
Show Stats Expand all files Collapse all files

2
internal/dal/impl.go
View File

@ -55,7 +55,7 @@ func (dal *DalImpl) InsertTransaction(t entity.Transaction) (entity.Transaction,
} }
stmt := ` stmt := `
INSERT INTO pfbudget.transactions (date, description, value) INSERT INTO pfbudget.transactions (date, description, amount)
VALUES ($1, $2, $3) VALUES ($1, $2, $3)
RETURNING id RETURNING id
` `

2
internal/dal/impl_test.go
View File

@ -297,7 +297,7 @@ func TestDalImpl_InsertTransaction(t *testing.T) {
mock. mock.
ExpectQuery(` ExpectQuery(`
INSERT INTO .* \(date, description, value\) INSERT INTO .* \(date, description, amount\)
VALUES \(\$1, \$2, \$3\) VALUES \(\$1, \$2, \$3\)
RETURNING id`). RETURNING id`).
WithArgs(tt.args.t.Date, tt.args.t.Description, tt.args.t.Value). WithArgs(tt.args.t.Date, tt.args.t.Description, tt.args.t.Value).