Add report to downloadChromium

download.ts

@@ -6,7 +6,7 @@ import { fetchChromeZipFile } from './api'
 import { DEFAULT_FULL_CONFIG, DEFAULT_SINGLE_CONFIG } from './commons/constants'
 import { DOWNLOAD_ZIP, EXTRACT_ZIP } from './commons/loggerTexts'
 import { NoChromiumDownloadError } from './errors'
-import type { IChromeConfig } from './interfaces/interfaces'
+import type { DownloadReportEntry, IChromeConfig } from './interfaces/interfaces'
 import { logger } from './log/logger'
 import { progress } from './log/progress'
 import { spinner } from './log/spinner'
@@ -77,14 +77,14 @@ async function extractZip(downloadPath: string) {
  * @see DEFAULT_SINGLE_CONFIG
  * @param additionalConfig Manually set config, which will override the settings in the default config
  */
-async function downloadForConfig(config: IChromeConfig): Promise<void> {
+async function downloadForConfig(config: IChromeConfig): Promise<DownloadReportEntry[]> {
 
     const versions = await loadVersions()
     const store = await loadStore()
 
     const mappedVersions = mapVersions(versions, config, store)
 
-    const { chromeUrl, selectedVersion, filenameOS } = await getChromeDownloadUrl(config, mappedVersions)
+    const { chromeUrl, filenameOS, report, selectedVersion } = await getChromeDownloadUrl(config, mappedVersions)
 
     if (chromeUrl && selectedVersion && config.download) {
         const filename = `chrome-${filenameOS}-${config.arch}-${selectedVersion.value}`
@@ -128,7 +128,7 @@ async function downloadForConfig(config: IChromeConfig): Promise<void> {
                 if (config.autoUnzip) {
                     await extractZip(downloadPath)
                 }
-                resolve()
+                resolve(report)
             })
 
             zipFileResponse.body.on('error', () => {
@@ -140,6 +140,8 @@ async function downloadForConfig(config: IChromeConfig): Promise<void> {
     if (!chromeUrl && config.download && config.single) {
         throw new NoChromiumDownloadError()
     }
+
+    return report
 }
 
 /**

interfaces/function.interfaces.ts

@@ -1,13 +1,23 @@
 import type { MappedVersion } from '../commons/MappedVersion'
-import type { Nullable } from './interfaces'
+import type { DownloadReportEntry, IChromeFullConfig, Nullable } from './interfaces'
+import { OSSetting } from './os.interfaces'
 
 export interface ContinueFetchingChromeUrlReturn {
     chromeUrl: Nullable<string>
+    report: DownloadReportEntry[]
     selectedVersion: Nullable<MappedVersion>
 }
 
 export interface GetChromeDownloadUrlReturn {
     chromeUrl: Nullable<string>
     selectedVersion: Nullable<MappedVersion>
     filenameOS: string
+    report: DownloadReportEntry[]
+}
+
+export interface ContinueFetchingChromeUrlParams {
+    config: IChromeFullConfig
+    mappedVersions: MappedVersion[]
+    osSetting: OSSetting
+    version: Nullable<MappedVersion>
 }

interfaces/interfaces.ts

@@ -1,4 +1,5 @@
 import type { ComparableVersion } from '../commons/ComparableVersion'
+import { MappedVersion } from '../commons/MappedVersion'
 import type { OS } from './os.interfaces'
 import type { Arch } from './store.interfaces'
 
@@ -126,3 +127,9 @@ export interface IVersionWithDisabled extends IVersion {
 }
 
 export type Nullable<T> = T | undefined | null
+
+export interface DownloadReportEntry {
+    binaryExists: boolean
+    download: boolean
+    version: Nullable<MappedVersion>
+}

test/test.utils.ts

@@ -62,6 +62,7 @@ export const createGetChromeDownloadUrlReturn = (settings?: Partial<GetChromeDow
     chromeUrl: 'chromeUrl',
     selectedVersion: new MappedVersion(10, 0, 0, 0, false),
     filenameOS: 'filenameOS',
+    report: [],
     ...settings,
 })