Skip to content

Commit

Permalink
adding unit tests
Browse files Browse the repository at this point in the history
  • Loading branch information
eamonnfaherty committed Aug 31, 2022
1 parent 14e6d8c commit 6c0ec6a
Show file tree
Hide file tree
Showing 21 changed files with 527 additions and 127 deletions.
2 changes: 0 additions & 2 deletions servicecatalog_puppet/workflow/apps/provision_app_task.py
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,6 @@ class ProvisionAppTask(tasks.TaskWithParameters):
key = luigi.Parameter()
version_id = luigi.Parameter()

puppet_account_id = luigi.Parameter()

ssm_param_inputs = luigi.ListParameter(default=[], significant=False)

launch_parameters = luigi.DictParameter(default={}, significant=False)
Expand Down
52 changes: 0 additions & 52 deletions servicecatalog_puppet/workflow/apps/provision_app_task_test.py

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,6 @@ class DoAssertTask(tasks.TaskWithParameters):
region = luigi.Parameter()
account_id = luigi.Parameter()

puppet_account_id = luigi.Parameter()
execution = luigi.Parameter()

expected = luigi.DictParameter()
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@

class DoExecuteCodeBuildRunTask(tasks.TaskWithParameters):
code_build_run_name = luigi.Parameter()
puppet_account_id = luigi.Parameter()

region = luigi.Parameter()
account_id = luigi.Parameter()
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,67 @@
from unittest import skip
from servicecatalog_puppet.workflow import tasks_unit_tests_helper


class DoExecuteCodeBuildRunTaskTest(tasks_unit_tests_helper.PuppetTaskUnitTest):
code_build_run_name = "code_build_run_name"
region = "region"
account_id = "account_id"
project_name = "project_name"
manifest_file_path = "manifest_file_path"

def setUp(self) -> None:
from servicecatalog_puppet.workflow.codebuild_runs import (
do_execute_code_build_run_task,
)

self.module = do_execute_code_build_run_task

self.sut = self.module.DoExecuteCodeBuildRunTask(
**self.get_common_args(),
code_build_run_name=self.code_build_run_name,
region=self.region,
account_id=self.account_id,
project_name=self.project_name,
manifest_file_path=self.manifest_file_path,
)

self.wire_up_mocks()

def test_params_for_results_display(self):
# setup
expected_result = {
"puppet_account_id": self.puppet_account_id,
"code_build_run_name": self.code_build_run_name,
"region": self.region,
"account_id": self.account_id,
"cache_invalidator": self.cache_invalidator,
}

# exercise
actual_result = self.sut.params_for_results_display()

# verify
self.assertEqual(expected_result, actual_result)

@skip
def test_api_calls_used(self):
# setup
expected_result = [
f"codebuild.start_build_{self.get_account_used()}_{self.project_name}",
f"codebuild.batch_get_projects_{self.get_account_used()}_{self.project_name}",
]

# exercise
actual_result = self.sut.api_calls_used()

# verify
self.assertEqual(expected_result, actual_result)

@skip
def test_run(self):
# setup
# exercise
actual_result = self.sut.run()

# verify
raise NotImplementedError()
2 changes: 0 additions & 2 deletions servicecatalog_puppet/workflow/generic/__init__.py

This file was deleted.

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -19,8 +19,6 @@ class DoInvokeLambdaTask(tasks.TaskWithParameters):
qualifier = luigi.Parameter()
invocation_type = luigi.Parameter()

puppet_account_id = luigi.Parameter()

manifest_file_path = luigi.Parameter()

section_name = constants.LAMBDA_INVOCATIONS
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,70 @@
from unittest import skip
from servicecatalog_puppet.workflow import tasks_unit_tests_helper


class DoInvokeLambdaTaskTest(tasks_unit_tests_helper.PuppetTaskUnitTest):
lambda_invocation_name = "lambda_invocation_name"
region = "region"
account_id = "account_id"
function_name = "function_name"
qualifier = "qualifier"
invocation_type = "invocation_type"
manifest_file_path = "manifest_file_path"

def setUp(self) -> None:
from servicecatalog_puppet.workflow.lambda_invocations import (
do_invoke_lambda_task,
)

self.module = do_invoke_lambda_task

self.sut = self.module.DoInvokeLambdaTask(
**self.get_common_args(),
lambda_invocation_name=self.lambda_invocation_name,
region=self.region,
account_id=self.account_id,
function_name=self.function_name,
qualifier=self.qualifier,
invocation_type=self.invocation_type,
manifest_file_path=self.manifest_file_path,
)

self.wire_up_mocks()

def test_params_for_results_display(self):
# setup
expected_result = {
"puppet_account_id": self.puppet_account_id,
"lambda_invocation_name": self.lambda_invocation_name,
"region": self.region,
"account_id": self.account_id,
"cache_invalidator": self.cache_invalidator,
}

# exercise
actual_result = self.sut.params_for_results_display()

# verify
self.assertEqual(expected_result, actual_result)

@skip
def test_api_calls_used(self):
# setup
expected_result = {
f"lambda.invoke_{self.get_account_used()}_{self.region}": 1,
}

# exercise
actual_result = self.sut.api_calls_used()

# verify
self.assertEqual(expected_result, actual_result)

@skip
def test_run(self):
# setup
# exercise
actual_result = self.sut.run()

# verify
raise NotImplementedError()
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@

class DoTerminateProductTask(tasks.TaskWithReference):
launch_name = luigi.Parameter()
puppet_account_id = luigi.Parameter()

region = luigi.Parameter()
account_id = luigi.Parameter()
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,80 @@
from unittest import skip
from servicecatalog_puppet.workflow import tasks_unit_tests_helper


class DoTerminateProductTaskTest(tasks_unit_tests_helper.PuppetTaskUnitTest):
launch_name = "launch_name"
region = "region"
account_id = "account_id"
portfolio = "portfolio"
product = "product"
version = "version"
ssm_param_inputs = []
launch_parameters = {}
manifest_parameters = {}
account_parameters = {}
ssm_param_outputs = []
execution = "execution"

def setUp(self) -> None:
from servicecatalog_puppet.workflow.launch import do_terminate_product_task

self.module = do_terminate_product_task

self.sut = self.module.DoTerminateProductTask(
**self.get_common_args(),
launch_name=self.launch_name,
region=self.region,
account_id=self.account_id,
portfolio=self.portfolio,
product=self.product,
version=self.version,
ssm_param_inputs=self.ssm_param_inputs,
launch_parameters=self.launch_parameters,
manifest_parameters=self.manifest_parameters,
account_parameters=self.account_parameters,
ssm_param_outputs=self.ssm_param_outputs,
execution=self.execution,
)

self.wire_up_mocks()

def test_params_for_results_display(self):
# setup
expected_result = {
"puppet_account_id": self.puppet_account_id,
"launch_name": self.launch_name,
"account_id": self.account_id,
"region": self.region,
"cache_invalidator": self.cache_invalidator,
}

# exercise
actual_result = self.sut.params_for_results_display()

# verify
self.assertEqual(expected_result, actual_result)

def test_api_calls_used(self):
# setup
uniq = f"{self.account_id}_{self.region}"
expected_result = [
f"servicecatalog.describe_provisioned_product_{uniq}",
f"servicecatalog.terminate_provisioned_product_{uniq}",
f"servicecatalog.describe_record_{uniq}",
]

# exercise
actual_result = self.sut.api_calls_used()

# verify
self.assertEqual(expected_result, actual_result)

@skip
def test_run(self):
# setup
# exercise
actual_result = self.sut.run()

# verify
raise NotImplementedError()
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@
class ProvisionProductTask(tasks.TaskWithParameters):
manifest_file_path = luigi.Parameter()
launch_name = luigi.Parameter()
puppet_account_id = luigi.Parameter()

region = luigi.Parameter()
account_id = luigi.Parameter()
Expand Down

0 comments on commit 6c0ec6a

Please sign in to comment.