Skip to content

Commit 3bd8706

Browse files
committed
test: add missing cases
1 parent 9ae7f20 commit 3bd8706

File tree

4 files changed

+222
-44
lines changed

4 files changed

+222
-44
lines changed

src/strategies/export-rest-operations-group.strategy.ts

Lines changed: 19 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,13 @@ export class ExportRestOperationsGroupStrategy implements BuilderStrategy {
5757
}
5858

5959
async function exportMergedDocument(config: ExportRestOperationsGroupBuildConfig, buildResult: BuildResult, contexts: BuildTypeContexts): Promise<BuildResult> {
60-
const { packageId, version: versionWithRevision, groupName, format = JSON_EXPORT_GROUP_FORMAT, allowedOasExtensions } = config
60+
const {
61+
packageId,
62+
version: versionWithRevision,
63+
groupName,
64+
format = JSON_EXPORT_GROUP_FORMAT,
65+
allowedOasExtensions,
66+
} = config
6167
const [version] = getSplittedVersionKey(versionWithRevision)
6268
const { templateResolver, packageResolver } = contexts.builderContext(config)
6369
const { name: packageName } = await packageResolver(packageId)
@@ -97,7 +103,13 @@ async function exportMergedDocument(config: ExportRestOperationsGroupBuildConfig
97103
}
98104

99105
async function exportReducedDocuments(config: ExportRestOperationsGroupBuildConfig, buildResult: BuildResult, contexts: BuildTypeContexts): Promise<BuildResult> {
100-
const { packageId, version: versionWithRevision, groupName, format = JSON_EXPORT_GROUP_FORMAT, allowedOasExtensions } = config
106+
const {
107+
packageId,
108+
version: versionWithRevision,
109+
groupName,
110+
format = JSON_EXPORT_GROUP_FORMAT,
111+
allowedOasExtensions,
112+
} = config
101113
const [version] = getSplittedVersionKey(versionWithRevision)
102114
const { templateResolver, packageResolver } = contexts.builderContext(config)
103115
const { name: packageName } = await packageResolver(packageId)
@@ -116,8 +128,11 @@ async function exportReducedDocuments(config: ExportRestOperationsGroupBuildConf
116128
buildResult.exportDocuments.push(...transformedDocuments)
117129

118130
if (format === HTML_EXPORT_GROUP_FORMAT) {
119-
buildResult.exportDocuments.push(createExportDocument('index.html', await generateIndexHtmlPage(packageName, version, generatedHtmlExportDocuments, templateResolver)))
120131
buildResult.exportDocuments.push(...await createCommonStaticExportDocuments(packageName, version, templateResolver))
132+
133+
if (generatedHtmlExportDocuments.length > 1) {
134+
buildResult.exportDocuments.push(createExportDocument('index.html', await generateIndexHtmlPage(packageName, version, generatedHtmlExportDocuments, templateResolver)))
135+
}
121136
}
122137

123138
if (buildResult.exportDocuments.length > 1) {
@@ -150,7 +165,7 @@ async function createTransformedDocument(
150165
}
151166

152167
return {
153-
fileId,
168+
fileId, // todo unused
154169
type,
155170
data: isRestDocument(document) ? removeOasExtensions(document.data, allowedOasExtensions) : '',
156171
description: '',

src/strategies/export-version.strategy.ts

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -112,12 +112,11 @@ export class ExportVersionStrategy implements BuilderStrategy {
112112

113113
buildResult.exportDocuments.push(...transformedDocuments)
114114

115-
const restDocuments = documents.filter(isRestDocument)
116-
if (format === HTML_EXPORT_GROUP_FORMAT && restDocuments.length > 0) {
115+
if (format === HTML_EXPORT_GROUP_FORMAT && generatedHtmlExportDocuments.length > 0) {
117116
buildResult.exportDocuments.push(...await createCommonStaticExportDocuments(packageName, version, templateResolver))
118117
const readme = buildResult.exportDocuments.find(({ fileId }) => fileId.toLowerCase() === 'readme.md')?.description
119118

120-
if (restDocuments.length > 1 || readme) {
119+
if (generatedHtmlExportDocuments.length > 1 || readme) {
121120
buildResult.exportDocuments.push(createExportDocument('index.html', await generateIndexHtmlPage(packageName, version, generatedHtmlExportDocuments, templateResolver, readme)))
122121
}
123122
}

0 commit comments

Comments
 (0)