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.

Sign up for GitHub

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

Jump to bottom

K-82: General refactor and CLI unit tests #90

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

K-82: General refactor and CLI unit tests #90

Show file tree
Hide file tree
Changes from 11 commits
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
5ec2575
[WIP] k-82: refactor, test_tf_service added
ms-shankar Nov 26, 2019
b21f8c1
[WIP] k-82 tests more tf_service tests
ms-shankar Nov 26, 2019
9777c09
removed unused imports
ms-shankar Nov 27, 2019
88c40ae
added tests for state service
ms-shankar Nov 27, 2019
99d5a93
k-91 state_service unit tests completed
ms-shankar Nov 28, 2019
2e5d9bd
refactor state service tests
ms-shankar Dec 2, 2019
b31bc79
[WIP] backend_facade unit tests
ms-shankar Jan 21, 2020
cdb65b3
test_backend_facade completed
ms-shankar Jan 23, 2020
6c1b5b9
Merge branch 'master' into k-82-general-refactor-for-unit-testability
aalhour Jan 27, 2020
c1c4ddd
Version updated to: 1.1.4
Jan 27, 2020
13b353a
review changes + minor refactor
ms-shankar Jan 28, 2020
1280f58
Version updated to: 1.1.4
Jan 28, 2020
b679cf0
removed unused switch and mocked file system manipulations
ms-shankar Feb 5, 2020
cffeb06
Merge branch 'master' into k-82-general-refactor-for-unit-testability
ms-shankar Feb 25, 2020
f323369
Version updated to: 1.1.5
Feb 25, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
62 changes: 31 additions & 31 deletions Pipfile.lock
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion VERSION
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1 +1 @@
1.1.3
1.1.4
2 changes: 2 additions & 0 deletions cli/kaos_cli/constants.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -98,3 +98,5 @@
CONTEXTS = 'contexts'
ACTIVE = 'active'
REMOTE = 'remote'

TEST_CONFIG_PATH = os.path.join(KAOS_HOME, 'kaos_cli', 'facades', 'tests', 'fixtures', 'config')
2 changes: 0 additions & 2 deletions cli/kaos_cli/facades/backend_facade.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,6 @@
from kaos_cli.utils.environment import check_environment
from kaos_cli.utils.helpers import build_dir
from kaos_cli.utils.validators import EnvironmentState
from kaos_cli.exceptions.handle_exceptions import handle_specific_exception, handle_exception


def is_cloud_provider(cloud):
Expand Down Expand Up @@ -199,7 +198,6 @@ def _remove_build_files(self, dir_build):
"""
Function to remove backend directory
"""
self.state_service.provider_delete(dir_build)
if not self.state_service.list_providers():
self.state_service.full_delete(dir_build)

Expand Down
Loading