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

Add initial unit tests #70

Open
wants to merge 55 commits into
base: master
Choose a base branch
from
Open

Add initial unit tests #70

wants to merge 55 commits into from

Conversation

amruthvvkp
Copy link
Owner

@amruthvvkp amruthvvkp commented Apr 8, 2024

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Other (please describe):

Pre-PR Checklist

  • General

    • New code is unit tested.
    • All relevant documentation is updated.
    • This pull request references any related issue by including "closes <link to issue>" (or "fixes" or "resolves"). If the pull request doesn't completely fix the issue, it should include "addresses".
    • Create an issue if this pull request does not completely fix the issue.
  • Code changes

    • This pull request includes tests or only affects documentation.
    • This pull request includes a label categorizing the change e.g. bug, feature, enhancement, docs.

Changes

This pull request includes significant updates to the CI/CD configuration, pre-commit hooks, and the flaui package. The changes streamline the build process, update dependencies, and enhance the functionality of the Application class.

CI/CD Configuration Updates:

Pre-commit Hooks and Dependencies:

Code Enhancements:

  • flaui/core/application.py: Added detailed module-level docstrings, introduced a setter for close_timeout, and improved the get_all_top_level_windows and get_main_window methods to return Window objects. [1] [2] [3]

Miscellaneous:

Testing Instructions

  • Dependency manager changed to UV, so you need to install UV to run tests on this project

Screenshots

  • If applicable, provide screenshots of the changes made in this pull request.

Checklist

  • I have read the contributing guidelines.
  • I have followed the code style guidelines.
  • I have added tests to cover my changes.
  • I have updated the documentation to reflect my changes.
  • I have added labels to categorize my changes.
  • I have tested my changes locally and they pass all tests.
  • I have rebased my branch on the latest main branch.
  • I have squashed my commits into a single commit with a descriptive commit message.
  • I have signed the contributor license agreement.

amruthvvkp added 20 commits July 3, 2023 10:29
* Added code to dynamically fetch test applications during tes run
* Updated project settings for Ruff to work as expected
* Updated file path settings from string to pathlib.Path in config.py
* Upgraded Pydantic from v1 to v2
* Added validator to throw ValueError is element is missing
* Added mapping of common known element class names
* Updated broken tests
* Added few more unit tests to TestAutomationElement
* Added custom exceptions
* Added map of WPF application
Grouped similar imports with updated Ruff settings
…tion class in condition_factory.py

* Added unit tests for Listbox, Menu, POpUp, ProgressBar, RadioButton
* Added logo for repository
* Updated pre-commit hooks
@amruthvvkp amruthvvkp self-assigned this Feb 18, 2025
@amruthvvkp amruthvvkp linked an issue Feb 18, 2025 that may be closed by this pull request
@amruthvvkp amruthvvkp changed the title Amruthvvkp/issue46 Add initial unit tests Feb 18, 2025
@amruthvvkp amruthvvkp marked this pull request as ready for review February 18, 2025 17:11
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.

Add initial unit tests
1 participant