Skip to content

Commit 13c8a21

Browse files
AkhileshNegiavirajsingh7
authored andcommitted
fixing testcases and migrations
1 parent a7a2a9c commit 13c8a21

File tree

5 files changed

+70
-3
lines changed

5 files changed

+70
-3
lines changed

backend/app/alembic/env.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
from logging.config import fileConfig
33

44
from alembic import context
5-
from sqlmodel import SQLModel
5+
from app.models import SQLModel
66
from sqlalchemy import engine_from_config, pool
77

88
# this is the Alembic Config object, which provides
Lines changed: 64 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,64 @@
1+
"""add_organization
2+
3+
Revision ID: 0d84ca709cfd
4+
Revises: 1a31ce608336
5+
Create Date: 2025-03-21 20:22:30.716746
6+
7+
"""
8+
from alembic import op
9+
import sqlalchemy as sa
10+
import sqlmodel.sql.sqltypes
11+
12+
13+
# revision identifiers, used by Alembic.
14+
revision = '0d84ca709cfd'
15+
down_revision = '1a31ce608336'
16+
branch_labels = None
17+
depends_on = None
18+
19+
20+
def upgrade():
21+
# ### commands auto generated by Alembic - please adjust! ###
22+
op.create_table('organization',
23+
sa.Column('name', sqlmodel.sql.sqltypes.AutoString(length=255), nullable=False),
24+
sa.Column('is_active', sa.Boolean(), nullable=False),
25+
sa.Column('id', sa.Integer(), nullable=False),
26+
sa.PrimaryKeyConstraint('id')
27+
)
28+
op.create_index(op.f('ix_organization_name'), 'organization', ['name'], unique=True)
29+
op.create_table('project',
30+
sa.Column('name', sqlmodel.sql.sqltypes.AutoString(length=255), nullable=False),
31+
sa.Column('description', sqlmodel.sql.sqltypes.AutoString(length=500), nullable=True),
32+
sa.Column('is_active', sa.Boolean(), nullable=False),
33+
sa.Column('id', sa.Integer(), nullable=False),
34+
sa.Column('organization_id', sa.Integer(), nullable=False),
35+
sa.ForeignKeyConstraint(['organization_id'], ['organization.id'], ),
36+
sa.PrimaryKeyConstraint('id')
37+
)
38+
op.create_index(op.f('ix_project_name'), 'project', ['name'], unique=False)
39+
op.create_index(op.f('ix_project_organization_id'), 'project', ['organization_id'], unique=False)
40+
op.create_table('projectuser',
41+
sa.Column('project_id', sa.Integer(), nullable=False),
42+
sa.Column('user_id', sa.Uuid(), nullable=False),
43+
sa.Column('is_admin', sa.Boolean(), nullable=False),
44+
sa.Column('id', sa.Integer(), nullable=False),
45+
sa.Column('created_at', sa.DateTime(), nullable=False),
46+
sa.Column('updated_at', sa.DateTime(), nullable=False),
47+
sa.Column('is_deleted', sa.Boolean(), nullable=False),
48+
sa.Column('deleted_at', sa.DateTime(), nullable=True),
49+
sa.ForeignKeyConstraint(['project_id'], ['project.id'], ondelete='CASCADE'),
50+
sa.ForeignKeyConstraint(['user_id'], ['user.id'], ondelete='CASCADE'),
51+
sa.PrimaryKeyConstraint('id')
52+
)
53+
# ### end Alembic commands ###
54+
55+
56+
def downgrade():
57+
# ### commands auto generated by Alembic - please adjust! ###
58+
op.drop_table('projectuser')
59+
op.drop_index(op.f('ix_project_organization_id'), table_name='project')
60+
op.drop_index(op.f('ix_project_name'), table_name='project')
61+
op.drop_table('project')
62+
op.drop_index(op.f('ix_organization_name'), table_name='organization')
63+
op.drop_table('organization')
64+
# ### end Alembic commands ###

backend/app/api/routes/users.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -178,7 +178,7 @@ def read_user_by_id(
178178
dependencies=[Depends(get_current_active_superuser)],
179179
response_model=UserPublic,
180180
)
181-
def update_user(
181+
def update_user_endpoint(
182182
*,
183183
session: SessionDep,
184184
user_id: uuid.UUID,

backend/app/models/__init__.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
from sqlmodel import SQLModel
2+
13
from .auth import Token, TokenPayload
24
from .item import Item, ItemCreate, ItemPublic, ItemsPublic, ItemUpdate
35
from .message import Message

backend/app/tests/api/routes/test_users.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -110,7 +110,7 @@ def test_get_existing_user_permissions_error(
110110
headers=normal_user_token_headers,
111111
)
112112
assert r.status_code == 403
113-
assert r.json() == {"detail": "The user doesn't have enough privileges"}
113+
assert r.json()["detail"] == "The user doesn't have enough privileges"
114114

115115

116116
def test_create_user_existing_username(
@@ -334,6 +334,7 @@ def test_update_user(
334334
headers=superuser_token_headers,
335335
json=data,
336336
)
337+
print(r)
337338
assert r.status_code == 200
338339
updated_user = r.json()
339340

0 commit comments

Comments
 (0)