TheHomecraft/bankserver_rust
1
0
Fork 0
mirror of https://git.dirksys.ovh/dirk/bankserver.git synced 2025-12-20 11:09:21 +01:00
Code Issues Packages Projects Releases Wiki Activity

make account names unique per user not globally

Browse Source
This commit is contained in:
DSeeLP 2025-03-11 11:12:56 +01:00
parent a4e233398f
commit 83c5768935
5 changed files with 11 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
migrations/000000_baseline.sql
View File

@ -7,8 +7,9 @@ create table users(
create table accounts( create table accounts(
id uuid primary key, id uuid primary key,
"user" uuid not null references users(id), "user" uuid not null references users(id),
name varchar(32) not null unique, name varchar(32) not null,
balance bigint not null default 0 constraint positive_balance check (balance >= 0) balance bigint not null default 0 constraint positive_balance check (balance >= 0),
unique ("user", name)
); );
create table transactions( create table transactions(

2
src/model/user.rs
View File

@ -59,7 +59,7 @@ impl User {
tx.execute(&stmt, &[&id, &name, &hash]) tx.execute(&stmt, &[&id, &name, &hash])
.await .await
.map_err(|err| unique_violation(err, conflict_error))?; .map_err(|err| unique_violation(err, conflict_error))?;
Account::create(&mut tx, Some(id), id, name) Account::create(&mut tx, Some(id), id, "personal")
.await .await
.map_err(|err| unique_violation(err, conflict_error))?; .map_err(|err| unique_violation(err, conflict_error))?;
tx.commit().await?; tx.commit().await?;

14
tests/integration/account-list.hurl
View File

@ -12,14 +12,12 @@ GET {{host}}/api/accounts?limit=50
Authorization: Bearer {{token}} Authorization: Bearer {{token}}
HTTP 200 HTTP 200
[Asserts] [Asserts]
jsonpath "$.pagination.total" == 7
jsonpath "$.pagination.limit" == 50 jsonpath "$.pagination.limit" == 50
jsonpath "$.pagination.offset" == 0 jsonpath "$.pagination.offset" == 0
jsonpath "$.result" isCollection jsonpath "$.result" isCollection
jsonpath "$.result[0].name" == "user1" jsonpath "$.result[0].name" == "personal"
jsonpath "$.result[1].name" == "user2" jsonpath "$.result[1].name" == "personal"
jsonpath "$.result[2].name" == "user3" jsonpath "$.result[2].name" == "personal"
jsonpath "$.result[3].name" == "user4" jsonpath "$.result[3].name" == "personal"
jsonpath "$.result[4].name" == "user5" jsonpath "$.result[4].name" == "personal"
jsonpath "$.result[5].name" == "user6" jsonpath "$.result[5].name" == "personal"
jsonpath "$.result[6].name" == "test-user"

2
tests/integration/user-account-list.hurl
View File

@ -13,5 +13,5 @@ Authorization: Bearer {{token}}
HTTP 200 HTTP 200
[Asserts] [Asserts]
jsonpath "$.result" isCollection jsonpath "$.result" isCollection
jsonpath "$.result[0].name" == "user1" jsonpath "$.result[0].name" == "personal"
jsonpath "$.result[0].balance" == 100000 jsonpath "$.result[0].balance" == 100000

2
tests/integration/user-list.hurl
View File

@ -12,7 +12,6 @@ GET {{host}}/api/users?limit=50
Authorization: Bearer {{token}} Authorization: Bearer {{token}}
HTTP 200 HTTP 200
[Asserts] [Asserts]
jsonpath "$.pagination.total" == 7
jsonpath "$.pagination.limit" == 50 jsonpath "$.pagination.limit" == 50
jsonpath "$.pagination.offset" == 0 jsonpath "$.pagination.offset" == 0
jsonpath "$.result" isCollection jsonpath "$.result" isCollection
@ -22,4 +21,3 @@ jsonpath "$.result[2].name" == "user3"
jsonpath "$.result[3].name" == "user4" jsonpath "$.result[3].name" == "user4"
jsonpath "$.result[4].name" == "user5" jsonpath "$.result[4].name" == "user5"
jsonpath "$.result[5].name" == "user6" jsonpath "$.result[5].name" == "user6"
jsonpath "$.result[6].name" == "test-user"