Skip to content

Add airport to test superuser#2962

Open
maneesha wants to merge 3 commits into
developfrom
superuser-airport
Open

Add airport to test superuser#2962
maneesha wants to merge 3 commits into
developfrom
superuser-airport

Conversation

@maneesha

@maneesha maneesha commented May 4, 2026

Copy link
Copy Markdown
Contributor

Adds the now-required airport field when creating superuser admin in dev database.

@maneesha

maneesha commented May 4, 2026

Copy link
Copy Markdown
Contributor Author

@pbanaszkiewicz What is the preferred styling when lines are too long?

@pbanaszkiewicz

Copy link
Copy Markdown
Contributor

@maneesha if you run uv run ruff format it will automatically correct the line for you. Depending on the length, it will either use this format:

def func(
    arg1, arg2, arg3, arg4, arg5
):
    ...

or this style:

def func(
    arg1,
    arg2,
    arg3,
    arg4,
    arg5,
):
    ...

@maneesha

maneesha commented May 5, 2026

Copy link
Copy Markdown
Contributor Author

@pbanaszkiewicz Now tests are failing because of the added field. Would you recommend I manually update each test? Or is there another way about this?

@pbanaszkiewicz

Copy link
Copy Markdown
Contributor

@maneesha Both mypy and tests are failing. You should follow mypy (see the 'lint' pipeline step) output - there are 13 errors, should be easy to fix. The tests are most likely failing because they all depend on create_superuser.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants