Skip to content

Commit 72bf07a

Browse files
committed
test(package): align LE tests with opt-in API from #9033
PR #9033 made AWS::LanguageExtensions local processing opt-in. Two API changes broke three tests on this branch after the merge from develop: - expand_language_extensions() now requires a keyword-only `enabled` arg - PackageContext reads self._language_extensions_enabled, set in __init__ Pass enabled=True to expand_language_extensions in the two artifact-exporter tests, and set _language_extensions_enabled on the PackageContext stub that bypasses __init__ via __new__. All three tests exercise templates with Transform: AWS::LanguageExtensions, so enabling LE is the intended behavior.
1 parent e0ec117 commit 72bf07a

2 files changed

Lines changed: 6 additions & 2 deletions

File tree

tests/unit/commands/package/test_package_context.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4524,6 +4524,10 @@ def setUp(self):
45244524
self.ctx.code_signer = MagicMock()
45254525
self.ctx.parameter_overrides = {}
45264526
self.ctx._global_parameter_overrides = {}
4527+
# Language Extensions is opt-in (#9033); the buried-AWS::Include regression
4528+
# test exercises a template that uses Transform: AWS::LanguageExtensions,
4529+
# so enable LE expansion explicitly.
4530+
self.ctx._language_extensions_enabled = True
45274531

45284532
def test_buried_aws_include_location_is_rewritten(self):
45294533
from samcli.yamlhelper import yaml_parse

tests/unit/lib/package/test_artifact_exporter.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2070,7 +2070,7 @@ def test_le_template_with_top_level_aws_include_merges_location(self):
20702070

20712071
parameter_values = dict(IntrinsicsSymbolTable.DEFAULT_PSEUDO_PARAM_VALUES)
20722072

2073-
result = expand_language_extensions(template_dict, parameter_values)
2073+
result = expand_language_extensions(template_dict, parameter_values, enabled=True)
20742074
self.assertTrue(result.had_language_extensions)
20752075

20762076
template = Template(
@@ -2168,7 +2168,7 @@ def test_le_template_with_serverless_repo_metadata_merges_license_url(self):
21682168

21692169
parameter_values = dict(IntrinsicsSymbolTable.DEFAULT_PSEUDO_PARAM_VALUES)
21702170

2171-
result = expand_language_extensions(template_dict, parameter_values)
2171+
result = expand_language_extensions(template_dict, parameter_values, enabled=True)
21722172
self.assertTrue(result.had_language_extensions)
21732173

21742174
template = Template(

0 commit comments

Comments
 (0)