Skip to content

Commit

Permalink
Change the order
Browse files Browse the repository at this point in the history
  • Loading branch information
henryruhs committed Jul 10, 2024
1 parent 4e7dbdc commit 69fa171
Show file tree
Hide file tree
Showing 2 changed files with 27 additions and 27 deletions.
24 changes: 12 additions & 12 deletions chroma_feedback/helper.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,18 +5,6 @@
from typing import Any, Dict


def parse_slug(slug : str) -> Dict[str, str]:
if slug:
return dict(zip(['workspace', 'project'], slug.split('/')))
return {}


def create_description(name : str, selector : str) -> str:
if name and selector:
return name + ' [' + selector + ']'
return name or selector


def is_root() -> bool:
try:
return os.getuid() == 0 #type:ignore
Expand All @@ -39,6 +27,18 @@ def is_windows() -> bool:
return to_lower_case(platform.system()) == 'windows'


def parse_slug(slug : str) -> Dict[str, str]:
if slug:
return dict(zip(['workspace', 'project'], slug.split('/')))
return {}


def create_description(name : str, selector : str) -> str:
if name and selector:
return name + ' [' + selector + ']'
return name or selector


def to_lower_case(__string__ : Any) -> str:
return str(__string__).lower()

Expand Down
30 changes: 15 additions & 15 deletions tests/test_helper.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,21 +3,6 @@
from chroma_feedback import helper


def test_parse_slug() -> None:
assert 'workspace' in helper.parse_slug('henryruhs')
assert 'project' not in helper.parse_slug('henryruhs')
assert 'workspace' in helper.parse_slug('henryruhs/chroma-feedback')
assert 'project' in helper.parse_slug('henryruhs/chroma-feedback')
assert 'workspace' not in helper.parse_slug(None)
assert 'project' not in helper.parse_slug(None)


def test_create_description() -> None:
assert helper.create_description('name', 'selector') == 'name [selector]'
assert helper.create_description('name', None) == 'name'
assert helper.create_description(None, 'selector') == 'selector'


def test_is_root() -> None:
assert helper.is_root() is not None

Expand All @@ -34,6 +19,21 @@ def test_is_windows() -> None:
assert helper.is_linux() is not None


def test_parse_slug() -> None:
assert 'workspace' in helper.parse_slug('henryruhs')
assert 'project' not in helper.parse_slug('henryruhs')
assert 'workspace' in helper.parse_slug('henryruhs/chroma-feedback')
assert 'project' in helper.parse_slug('henryruhs/chroma-feedback')
assert 'workspace' not in helper.parse_slug(None)
assert 'project' not in helper.parse_slug(None)


def test_create_description() -> None:
assert helper.create_description('name', 'selector') == 'name [selector]'
assert helper.create_description('name', None) == 'name'
assert helper.create_description(None, 'selector') == 'selector'


def test_to_lower_case() -> None:
assert helper.to_lower_case(None) == 'none'
assert helper.to_lower_case('SUCCESS') == 'success'
Expand Down

0 comments on commit 69fa171

Please sign in to comment.