Merge pull request #186 from espressif/fix/empty-collect-file

fix: collect file not created when no apps built

Author: hfudev

idf_build_apps/main.py

@@ -10,6 +10,7 @@
 import sys
 import textwrap
 import typing as t
+from pathlib import Path
 
 import argcomplete
 from pydantic import (
@@ -170,6 +171,13 @@ def build_apps(
             LOGGER.debug('Remove existing collect file %s', f)
 
     exit_code = 0
+
+    # create empty files, avoid no file when no app is built
+    if build_arguments.collect_app_info:
+        Path(build_arguments.collect_app_info).touch()
+    if build_arguments.collect_size_info:
+        Path(build_arguments.collect_size_info).touch()
+
     for i, app in enumerate(apps):
         index = i + 1  # we use 1-based
         if index < start or index > stop:

tests/test_build.py

@@ -1,4 +1,4 @@
-# SPDX-FileCopyrightText: 2022-2024 Espressif Systems (Shanghai) CO LTD
+# SPDX-FileCopyrightText: 2022-2025 Espressif Systems (Shanghai) CO LTD
 # SPDX-License-Identifier: Apache-2.0
 
 import os
@@ -22,6 +22,7 @@
 from idf_build_apps.app import (
     CMakeApp,
 )
+from idf_build_apps.args import BuildArguments
 from idf_build_apps.constants import (
     IDF_PATH,
     BuildStatus,
@@ -202,3 +203,18 @@ def test_build_apps_without_passing_apps(self, tmp_path):
         assert test_suite.attrib['skipped'] == '0'
 
         assert test_suite.findall('testcase')[0].attrib['name'] == 'foo/bar/build'
+
+
+def test_build_apps_collect_files_when_no_apps_built(tmp_path):
+    os.chdir(tmp_path)
+
+    build_apps(
+        build_arguments=BuildArguments(
+            target='esp32',
+            collect_app_info_filename='app_info.txt',
+            collect_size_info_filename='size_info.txt',
+        )
+    )
+
+    assert os.path.exists('app_info.txt')
+    assert os.path.exists('size_info.txt')