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

Tests folder added including the test file for the class DiffMotionDetector in the motion_detection.py file #1

Merged
merged 1 commit into from
Nov 26, 2023

Conversation

Umair0343
Copy link
Owner

This pull request is to incorporate the suggestion provided in the PR mpatacchiola#107 and closes issue mpatacchiola#77

@Umair0343 Umair0343 merged commit 2d49fd5 into master Nov 26, 2023
@Umair0343
Copy link
Owner Author

/review

@Umair0343
Copy link
Owner Author

/describe

@Umair0343
Copy link
Owner Author

/describe
--pr_description.extra_instructions="
For the title, use the format [type]: [summary]
"
--pr_description.publish_description_as_comment=true
--pr_description.generate_ai_title=true

Copy link

Title

[Tests] Add test suite for DiffMotionDetector in deepgaze module


User description

This pull request is to incorporate the suggestion provided in the PR mpatacchiola#107 and closes issue mpatacchiola#77


PR Type

Tests


Description

  • Introduced a new test suite for the DiffMotionDetector class in the deepgaze module.
  • The tests validate the functionality of setting background images, returning binary images, and handling None inputs effectively.
  • Ensures robustness in the motion detection functionality by testing various scenarios including valid and None inputs.

Changes walkthrough 📝

Relevant files
Tests
test.py
Add comprehensive tests for DiffMotionDetector class         

tests/test_diff_motion_detector/test.py

  • Added a new test class TestDiffMotionDetector for the
    DiffMotionDetector class.
  • Implemented tests to check the setting of the background image and the
    return of binary images.
  • Included tests for handling None inputs for background and foreground
    images.
  • +54/-0   

    💡 PR-Agent usage:
    Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Labels
    Projects
    None yet
    1 participant