Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

✅ 계좌 생성, 등록 api 테스트코드 추가 #90

Merged
merged 1 commit into from
Feb 24, 2024
Merged

Conversation

mingi3314
Copy link
Owner

@mingi3314 mingi3314 commented Feb 24, 2024


Generated summary (powered by Graphite)

TL;DR

This pull request introduces a new test file test_api.py for testing the API endpoints related to account management. The tests cover creating a new account and getting the default account.

What changed

A new test file test_api.py was added under tests/controllers/. This file contains tests for the following scenarios:

  • Creating a new account
  • Getting the default account when an account has been created
  • Getting the default account when no account has been created

The tests use a temporary account repository to ensure that the actual production files are not modified during testing.

How to test

To run the tests, use the following command:

pytest tests/controllers/test_api.py

Ensure that all tests pass.

Why make this change

These tests are necessary to ensure that the API endpoints for account management are working as expected. They help catch any potential issues or bugs early in the development process, and ensure that any changes or updates to the code do not break existing functionality.

Copy link
Owner Author

mingi3314 commented Feb 24, 2024

Merge activity

  • Feb 24, 9:32 AM EST: @mingi3314 started a stack merge that includes this pull request via Graphite.
  • Feb 24, 9:41 AM EST: Graphite rebased this pull request as part of a merge.
  • Feb 24, 9:42 AM EST: @mingi3314 merged this pull request with Graphite.

Base automatically changed from 02-24-_fixture_ to main February 24, 2024 14:40
@mingi3314 mingi3314 merged commit 913de1e into main Feb 24, 2024
1 check passed
@mingi3314 mingi3314 deleted the 02-24-_api_ branch February 24, 2024 14:42
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.

1 participant