Skip to content
This repository was archived by the owner on Feb 23, 2024. It is now read-only.

Commit e1f19f0

Browse files
levimatheriLevi MuriukiRupengLiu
authored
For #476. [Creator] Fix missing product in generated ARM template (#507)
Co-authored-by: Levi Muriuki <muriukilm@upmc.edu> Co-authored-by: RupengLiu <rliu1211@terpmail.umd.edu>
1 parent 8874459 commit e1f19f0

File tree

3 files changed

+7
-3
lines changed

3 files changed

+7
-3
lines changed

src/APIM_ARMTemplate/apimtemplate.test/Creator/TemplateCreatorFactories/APITemplateCreatorFactory.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ public static APITemplateCreator GenerateAPITemplateCreator()
1414
DiagnosticTemplateCreator diagnosticTemplateCreator = new DiagnosticTemplateCreator();
1515
ReleaseTemplateCreator releaseTemplateCreator = new ReleaseTemplateCreator();
1616
TagAPITemplateCreator tagAPITemplateCreator = new TagAPITemplateCreator();
17-
APITemplateCreator apiTemplateCreator = new APITemplateCreator(fileReader, policyTemplateCreator, tagAPITemplateCreator, diagnosticTemplateCreator, releaseTemplateCreator);
17+
APITemplateCreator apiTemplateCreator = new APITemplateCreator(fileReader, policyTemplateCreator, productAPITemplateCreator, tagAPITemplateCreator, diagnosticTemplateCreator, releaseTemplateCreator);
1818
return apiTemplateCreator;
1919
}
2020
}

src/APIM_ARMTemplate/apimtemplate/Commands/Create.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ public CreateCommand()
101101
ProductTemplateCreator productTemplateCreator = new ProductTemplateCreator(policyTemplateCreator, productGroupTemplateCreator, productSubscriptionsTemplateCreator);
102102
PropertyTemplateCreator propertyTemplateCreator = new PropertyTemplateCreator();
103103
TagTemplateCreator tagTemplateCreator = new TagTemplateCreator();
104-
APITemplateCreator apiTemplateCreator = new APITemplateCreator(fileReader, policyTemplateCreator, tagAPITemplateCreator, diagnosticTemplateCreator, releaseTemplateCreator);
104+
APITemplateCreator apiTemplateCreator = new APITemplateCreator(fileReader, policyTemplateCreator, productAPITemplateCreator, tagAPITemplateCreator, diagnosticTemplateCreator, releaseTemplateCreator);
105105
MasterTemplateCreator masterTemplateCreator = new MasterTemplateCreator();
106106

107107
// create templates from provided configuration

src/APIM_ARMTemplate/apimtemplate/Creator/TemplateCreators/APITemplateCreator.cs

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,14 +12,16 @@ public class APITemplateCreator : TemplateCreator
1212
{
1313
private FileReader fileReader;
1414
private PolicyTemplateCreator policyTemplateCreator;
15+
private ProductAPITemplateCreator productAPITemplateCreator;
1516
private TagAPITemplateCreator tagAPITemplateCreator;
1617
private DiagnosticTemplateCreator diagnosticTemplateCreator;
1718
private ReleaseTemplateCreator releaseTemplateCreator;
1819

19-
public APITemplateCreator(FileReader fileReader, PolicyTemplateCreator policyTemplateCreator, TagAPITemplateCreator tagAPITemplateCreator, DiagnosticTemplateCreator diagnosticTemplateCreator, ReleaseTemplateCreator releaseTemplateCreator)
20+
public APITemplateCreator(FileReader fileReader, PolicyTemplateCreator policyTemplateCreator, ProductAPITemplateCreator productAPITemplateCreator, TagAPITemplateCreator tagAPITemplateCreator, DiagnosticTemplateCreator diagnosticTemplateCreator, ReleaseTemplateCreator releaseTemplateCreator)
2021
{
2122
this.fileReader = fileReader;
2223
this.policyTemplateCreator = policyTemplateCreator;
24+
this.productAPITemplateCreator = productAPITemplateCreator;
2325
this.tagAPITemplateCreator = tagAPITemplateCreator;
2426
this.diagnosticTemplateCreator = diagnosticTemplateCreator;
2527
this.releaseTemplateCreator = releaseTemplateCreator;
@@ -94,6 +96,7 @@ public List<TemplateResource> CreateChildResourceTemplates(APIConfig api)
9496

9597
PolicyTemplateResource apiPolicyResource = api.policy != null ? this.policyTemplateCreator.CreateAPIPolicyTemplateResource(api, dependsOn) : null;
9698
List<PolicyTemplateResource> operationPolicyResources = api.operations != null ? this.policyTemplateCreator.CreateOperationPolicyTemplateResources(api, dependsOn) : null;
99+
List<ProductAPITemplateResource> productAPIResources = api.products != null ? this.productAPITemplateCreator.CreateProductAPITemplateResources(api, dependsOn) : null;
97100
List<TagAPITemplateResource> tagAPIResources = api.tags != null ? this.tagAPITemplateCreator.CreateTagAPITemplateResources(api, dependsOn) : null;
98101
DiagnosticTemplateResource diagnosticTemplateResource = api.diagnostic != null ? this.diagnosticTemplateCreator.CreateAPIDiagnosticTemplateResource(api, dependsOn) : null;
99102
// add release resource if the name has been appended with ;rev{revisionNumber}
@@ -102,6 +105,7 @@ public List<TemplateResource> CreateChildResourceTemplates(APIConfig api)
102105
// add resources if not null
103106
if (apiPolicyResource != null) resources.Add(apiPolicyResource);
104107
if (operationPolicyResources != null) resources.AddRange(operationPolicyResources);
108+
if (productAPIResources != null) resources.AddRange(productAPIResources);
105109
if (tagAPIResources != null) resources.AddRange(tagAPIResources);
106110
if (diagnosticTemplateResource != null) resources.Add(diagnosticTemplateResource);
107111
if (releaseTemplateResource != null) resources.Add(releaseTemplateResource);

0 commit comments

Comments
 (0)