Skip to content

test: add history endpoint tests (skipped due to missing implementati…#851

Open
Shruti1420 wants to merge 1 commit into
komalharshita:mainfrom
Shruti1420:test/history-coverage-833
Open

test: add history endpoint tests (skipped due to missing implementati…#851
Shruti1420 wants to merge 1 commit into
komalharshita:mainfrom
Shruti1420:test/history-coverage-833

Conversation

@Shruti1420

@Shruti1420 Shruti1420 commented Jun 11, 2026

Copy link
Copy Markdown

Summary

This PR improves test coverage for history-related functionality as described in issue #833.

Changes Made

  • Added tests for:

    • Maximum code length validation (max_length=50000)
    • Pagination limit boundary validation
    • Search limit validation
    • SHA256 code hash verification
    • Repeated deletion behavior (second delete returns 404)

Important Note

The repository currently does not implement /history/ endpoints.
To prevent CI failures, these tests are marked as skipped using pytest.

Benefits

  • Improves coverage for validation and edge cases
  • Prepares the test suite for future history feature implementation
  • Helps prevent regressions once endpoints are added

Verification

  • All existing tests pass ✅
  • History tests are safely skipped ⚠️

Issue

Closes #833

@vercel

vercel Bot commented Jun 11, 2026

Copy link
Copy Markdown

@Shruti1420 is attempting to deploy a commit to the komalsony234-1530's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for submitting your first pull request to DevPath.

Before review:

  • Complete the PR template fully
  • Ensure all tests pass
  • Link your PR to an issue
  • Keep changes scoped to the issue

A maintainer will review your contribution soon.

@Shruti1420

Copy link
Copy Markdown
Author

Hi! I have added tests for history endpoints as per issue #833.

Since the endpoints are not implemented yet, I have skipped those tests to avoid CI failures.

Please let me know if any changes are required.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve test coverage for history endpoints and database behavior

1 participant