-
-
Notifications
You must be signed in to change notification settings - Fork 4
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 hugie ui
command (Fixes #30)
#70
Conversation
hugie ui
commandhugie ui
command (Resolves #30)
hugie ui
command (Resolves #30)hugie ui
command (Fixes #30)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, a test would also be good
hard to test because it requires opening a browser, not sure it is worth mocking |
you want to mock that |
tests/test_main.py
Outdated
|
||
|
||
def test_ui_command(monkeypatch): | ||
def mock_open(url): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
better to define the mock_open
as a Mock
https://docs.python.org/3/library/unittest.mock.html#the-mock-class and then test it is called assert_called
https://docs.python.org/3/library/unittest.mock.html#unittest.mock.Mock.assert_called
Description
Resolves #'30 by adding the
hugie ui
command which opens the Hugging Face Inference Endpoints dashboard in a browser.This may be the killer function for this tool 😬
Checklist