feat: add new parameter to build in source and pass parameter into wo…

…rkflows (#416)

* add new parameter and pass into workflows

* change default to None

* update docstring

aws_lambda_builders/__main__.py

@@ -131,6 +131,7 @@ def main(): # pylint: disable=too-many-statements
  architecture=params.get("architecture", X86_64),
  is_building_layer=params.get("is_building_layer", False),
  experimental_flags=params.get("experimental_flags", []),
+ build_in_source=params.get("build_in_source", None),
  )
 
  # Return a success response

aws_lambda_builders/builder.py

@@ -71,6 +71,7 @@ def build(
  architecture=X86_64,
  is_building_layer=False,
  experimental_flags=None,
+ build_in_source=None,
  ):
  # pylint: disable-msg=too-many-locals
  """
@@ -138,6 +139,11 @@ def build(
  :type experimental_flags: list
  :param experimental_flags:
  List of strings, which will indicate enabled experimental flags for the current build session
+
+ :type build_in_source: Optional[bool]
+ :param build_in_source:
+ Optional, will execute the build operation in the source directory if True.
+
  """
 
  if not os.path.exists(scratch_dir):
@@ -159,6 +165,7 @@ def build(
  architecture=architecture,
  is_building_layer=is_building_layer,
  experimental_flags=experimental_flags,
+ build_in_source=build_in_source,
  )
 
  return workflow.run()

aws_lambda_builders/workflow.py

@@ -165,6 +165,7 @@ def __init__(
  architecture=X86_64,
  is_building_layer=False,
  experimental_flags=None,
+ build_in_source=None,
  ):
  # pylint: disable-msg=too-many-locals
  """
@@ -208,6 +209,9 @@ def __init__(
 
  experimental_flags: list, optional
  List of strings, which will indicate enabled experimental flags for the current build session
+
+ build_in_source: Optional[bool]
+ Optional, will execute the build operation in the source directory if True.
  """
 
  self.source_dir = source_dir
@@ -225,6 +229,7 @@ def __init__(
  self.architecture = architecture
  self.is_building_layer = is_building_layer
  self.experimental_flags = experimental_flags if experimental_flags else []
+ self.build_in_source = build_in_source
 
  # Actions are registered by the subclasses as they seem fit
  self.actions = []

tests/functional/test_cli.py

@@ -83,6 +83,7 @@ def test_run_hello_workflow_with_backcompat(self, flavor, protocol_version):
  "architecture": "x86_64",
  "is_building_layer": False,
  "experimental_flags": ["experimental"],
+ "build_in_source": False,
  },
  }
 
@@ -149,6 +150,7 @@ def test_run_hello_workflow_incompatible(self, flavor):
  "combine_dependencies": False,
  "is_building_layer": False,
  "experimental_flags": ["experimental"],
+ "build_in_source": False,
  },
  }
  )

tests/unit/test_builder.py

@@ -127,6 +127,7 @@ def setUp(self):
  [True, False], # combine_dependencies
  [True, False], # is_building_layer
  [None, [], ["a", "b"]], # experimental flags
+ [True, False], # build_in_source
  )
  )
  @patch("aws_lambda_builders.builder.os")
@@ -139,6 +140,7 @@ def test_with_mocks(
  combine_dependencies,
  is_building_layer,
  experimental_flags,
+ build_in_source,
  get_workflow_mock,
  os_mock,
  ):
@@ -167,6 +169,7 @@ def test_with_mocks(
  combine_dependencies=combine_dependencies,
  is_building_layer=is_building_layer,
  experimental_flags=experimental_flags,
+ build_in_source=build_in_source,
  )
 
  workflow_cls.assert_called_with(
@@ -185,6 +188,7 @@ def test_with_mocks(
  combine_dependencies=combine_dependencies,
  is_building_layer=is_building_layer,
  experimental_flags=experimental_flags,
+ build_in_source=build_in_source,
  )
  workflow_instance.run.assert_called_once()
  os_mock.path.exists.assert_called_once_with("scratch_dir")