FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Add unit tests for exception handling by Beeram12 · Pull Request #151 · XortexAI/XMem · GitHub

This repository was archived by the owner on Jun 3, 2026. It is now read-only.
/ XMem Public archive

Add unit tests for exception handling - #151

Draft
Beeram12 wants to merge 2 commits into
XortexAI:mainfrom
Beeram12:feature/unit-tests
ConversationCommits (2)ChecksFiles changed
Draft

Add unit tests for exception handling#151
Beeram12 wants to merge 2 commits into
XortexAI:mainfrom
Beeram12:feature/unit-tests

Conversation

Beeram12 commented May 5, 2026 •
edited
Loading

Copy link
Copy Markdown

fixes #147

  1. This adds unit tests for exception.py .

github-actions Bot added the tests label May 5, 2026
gemini-code-assist Bot reviewed May 5, 2026
View reviewed changes

gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Code Review

This pull request unignores the tests/ directory and introduces a new test suite for the project's custom exception hierarchy. The feedback focuses on improving test accuracy by fixing a typo and directly verifying attributes, adhering to PEP 8 standards for None comparisons, and ensuring complete coverage of the exception hierarchy by adding a test for VectorStoreValidationError.

Comment thread tests/test_exception.py Outdated
Comment thread tests/test_exception.py Outdated
Comment thread tests/test_exception.py

ishaanxgupta commented May 8, 2026

Copy link
Copy Markdown
Contributor

@Beeram12 Please have a look on the gemini suggestions

Beeram12 commented May 8, 2026

Copy link
Copy Markdown
Author

@Beeram12 Please have a look on the gemini suggestions

yeah I have resovled all but I didnt commit

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

ishaanxgupta Awaiting requested review from ishaanxgupta ishaanxgupta will be requested when the pull request is marked ready for review ishaanxgupta is a code owner
ved015 Awaiting requested review from ved015 ved015 will be requested when the pull request is marked ready for review ved015 is a code owner
1 more reviewer
gemini-code-assist[bot] gemini-code-assist[bot] left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

tests

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Phase 1: Foundation and Core Unit Tests

2 participants

Back | FazBrowse Home | New Git URL