idf_build_apps 1.0.0__tar.gz → 1.0.0.dev1__tar.gz

{idf_build_apps-1.0.0 → idf_build_apps-1.0.0.dev1}/.github/workflows/test-build-idf-apps.yml

@@ -66,10 +66,9 @@ jobs:
           python -m idf_build_apps build -vv -t esp32 \
             -p $IDF_PATH/examples/get-started/hello_world \
             --size-file size_info.json \
-            --ignore-warning-str \
-              "Python 3 versions older than 3.6 are not supported" \
-              "Python 2 is no longer supported" \
-              "Support for Python 2 is deprecated"
+            --ignore-warning-str "Python 3 versions older than 3.6 are not supported" \
+            --ignore-warning-str "Python 2 is no longer supported" \
+            --ignore-warning-str "Support for Python 2 is deprecated"
 
   build-apps-on-idf-env:
     if: ${{ github.ref == 'refs/heads/main' }}

{idf_build_apps-1.0.0 → idf_build_apps-1.0.0.dev1}/CHANGELOG.md

@@ -2,12 +2,11 @@
 
 All notable changes to this project will be documented in this file.
 
-## [1.0.0] (2023-05-25)
+## [Unreleased - 1.0.0]
 
 ## Added
 
 - Support keyword `depends_filepatterns` in the manifest file
-- Support expanding environment variables in the manifest files
 
 ## BREAKING CHANGES
 
@@ -24,13 +23,6 @@ All notable changes to this project will be documented in this file.
 - CLI Options Renamed
   - `--depends-on-components` renamed to `--modified-components`
   - `--depends-on-files` renamed to `--modified-files`
-  - `--ignore-components-dependencies-file-patterns` renamed to `--ignore-app-dependencies-filepatterns`
-- Removed the deprecated CLI call methods, now these options only support space-separated list
-  - `--exclude`
-  - `--config`
-  - `--manifest-file`
-  - `--ignore-warning-str`
-  - `--default-build-targets`
 
 ## [0.6.1] (2023-05-10)
 
@@ -130,7 +122,7 @@ This is the last version to support ESP-IDF v4.1 since it's EOL on Feb. 24th, 20
 
 ### Fixed
 
-- Relative path defined in the manifest files depend on the current work path
+- Ralative path defined in the manifest files depend on the current work path
 
   Added `manifest_rootpath` argument in `find_apps()`. Will use this value instead as the root folder for calculating absolute path
 

{idf_build_apps-1.0.0 → idf_build_apps-1.0.0.dev1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: idf-build-apps
-Version: 1.0.0
+Version: 1.0.0.dev1
 Summary: Tools for building ESP-IDF related apps.
 Author-email: Fu Hanxi <fuhanxi@espressif.com>
 Requires-Python: >=2.7, !=3.0.*, !=3.1.*, !=3.2.*, !=3.3.*

{idf_build_apps-1.0.0 → idf_build_apps-1.0.0.dev1}/docs/find_build.md

@@ -114,7 +114,7 @@ By default, `idf-build-apps` would use the project directory as the work directo
 
 #### Build Directory
 
-Build directory is the directory where the binary files output would be generated. If it is set to a relative path, the full path would be calculated based on the work directory. If it is an absolute path, it would override the work directory settings.
+Build directory is the directory where the binary files output would be generated. If it is set to a relative path, the full path would be calculated based on the work directory. If it is a absolute path, it would override the work directory settings.
 
 By default, `idf-build-apps` would follow what ESP-IDF does, use `build` as the build directory.
 

{idf_build_apps-1.0.0 → idf_build_apps-1.0.0.dev1}/docs/manifest.md

@@ -23,14 +23,12 @@ One typical manifest file look like this:
 
 ### Operands
 
-- Capitalized Words
-  - Variables start with `SOC_`. The value would be parsed from `IDF_PATH/components/soc/[TARGET]/include/soc/*_caps.h`
-  - `IDF_TARGET`
-  - `IDF_VERSION_MAJOR`
-  - `IDF_VERSION_MINOR`
-  - `IDF_VERSION_PATCH`
-  - `INCLUDE_DEFAULT` (The default value of officially supported targets is 1, otherwise is 0)
-  - environment variables, default to `0` if not set
+- Variables start with `SOC_`. The value would be parsed from `IDF_PATH/components/soc/[TARGET]/include/soc/*_caps.h`
+- `IDF_TARGET`
+- `IDF_VERSION_MAJOR`
+- `IDF_VERSION_MINOR`
+- `IDF_VERSION_PATCH`
+- `INCLUDE_DEFAULT` (The default value of officially supported targets is 1, otherwise is 0)
 - String, must be double-quoted. e.g., `"esp32"`, `"12345"`
 - Integer, support decimal and hex. e.g., `1`, `0xAB`
 - List of strings or integers, or both types at the same time. e.g., `["esp32", 1]`
@@ -114,57 +112,3 @@ examples/get-started/blink:
     - if: INCLUDE_DEFAULT == 1 or IDF_TARGET == "linux"
       reason: This one supports all supported targets and linux
 ```
-
-## Check App Dependencies
-
-`idf-build-apps` also supports building only related apps by checking app dependencies via modified components and modified files.
-
-### Basic Usage
-
-To enable this feature, you need to declare the modified files or components using the CLI option:
-
-- `--modified-files`
-- `--modified-components`
-
-To declare the dependencies for an app, you could add two more keywords in the corresponding manifest file:
-
-- `depends_components`
-- `depends_filepatterns`
-
-One app will be built when:
-
-- The app itself is modified (`.md` files are excluded)
-- Any of the specified `depends_components` in the corresponding manifest file are modified
-- Files that matches any of the specified `depends_filepatterns` in the corresponding manifest file are modified
-- Any of the `build_components` are modified. `build_components` are defined in the `project_description.json`, which is generated by running `idf.py reconfigure` or `idf.py build`
-
-For example, this is an app `example/foo`, which depends on `comp1`, `comp2`, `comp3` and all files under `common_header_files`:
-
-```yaml
-examples/foo:
-  depends_components:
-    - comp1
-    - comp2
-    - comp3
-  depends_files:
-    - "common_header_files/**/*"
-```
-
-This app will be built with the following CLI options:
-
-- `--modified-files examples/foo/main/foo.c`
-- `--modified-components comp1`
-- `--modified-components comp2 comp3`
-- `--modified-files common_header_files/foo.h`
-
-This app will not be built with the following CLI options:
-
-- `--modified-files examples/foo/main/foo.md`
-- `--modified-components bar`
-- `/tmp/foo.c`
-
-### Advanced Usage
-
-Sometimes, we have some root dependencies. All apps should be built if these root dependencies are modified.
-
-After adding `--ignore-app-dependencies-filepatterns` to the CLI options, if files that matches any of the specified patterns are modified, the check app dependency feature will be disabled.

{idf_build_apps-1.0.0 → idf_build_apps-1.0.0.dev1}/idf_build_apps/__init__.py

@@ -7,7 +7,7 @@ Tools for building ESP-IDF related apps.
 
 import logging
 
-__version__ = '1.0.0'
+__version__ = '1.0.0.dev1'
 
 LOGGER = logging.getLogger('idf_build_apps')
 

{idf_build_apps-1.0.0 → idf_build_apps-1.0.0.dev1}/idf_build_apps/finder.py

@@ -17,7 +17,6 @@ from .app import (
 )
 from .utils import (
     config_rules_from_str,
-    to_absolute_path,
     to_list,
 )
 
@@ -166,18 +165,15 @@ def _find_apps(
 
     # The remaining part is for recursive == True
     apps = []
-    # handle the exclude list, since the config file might use linux style, but run in windows
-    exclude_list = [to_absolute_path(p) for p in exclude_list]
     for root, dirs, _ in os.walk(path):
         LOGGER.debug('Entering %s', root)
-        root_path = to_absolute_path(root)
-        if root_path in exclude_list:
+        if root in exclude_list:
             LOGGER.debug('=> Skipping %s (excluded)', root)
             del dirs[:]
             continue
 
-        if root_path.parts[-1] == 'managed_components':  # idf-component-manager
-            LOGGER.debug('=> Skipping %s (managed components)', root_path)
+        if root.endswith('managed_components'):  # idf-component-manager
+            LOGGER.debug('=> Skipping %s (managed components)', root)
             del dirs[:]
             continue
 

{idf_build_apps-1.0.0 → idf_build_apps-1.0.0.dev1}/idf_build_apps/main.py

@@ -2,12 +2,14 @@
 # SPDX-License-Identifier: Apache-2.0
 
 import argparse
+import io
 import json
 import os
 import re
 import shutil
 import sys
 import textwrap
+import warnings
 from pathlib import (
     Path,
 )
@@ -53,22 +55,22 @@ def _check_app_dependency(
     manifest_rootpath,  # type: str
     modified_components,  # type: list[str] | None
     modified_files,  # type: list[str] | None
-    ignore_app_dependencies_filepatterns,  # type: list[str] | None
+    ignore_component_dependencies_file_patterns,  # type: list[str] | None
 ):  # type: (...) -> bool
     # not check since modified_components and modified_files are not passed
     if modified_components is None and modified_files is None:
         return False
 
-    # not check since ignore_app_dependencies_filepatterns is passed and matched
+    # not check since ignore_component_dependencies_file_patterns is passed and matched
     if (
-        ignore_app_dependencies_filepatterns
+        ignore_component_dependencies_file_patterns
         and modified_files is not None
-        and files_matches_patterns(modified_files, ignore_app_dependencies_filepatterns, manifest_rootpath)
+        and files_matches_patterns(modified_files, ignore_component_dependencies_file_patterns, manifest_rootpath)
     ):
         LOGGER.debug(
             'Skipping check component dependencies for apps since files %s matches patterns: %s',
             ', '.join(modified_files),
-            ', '.join(ignore_app_dependencies_filepatterns),
+            ', '.join(ignore_component_dependencies_file_patterns),
         )
         return False
 
@@ -93,7 +95,7 @@ def find_apps(
     default_build_targets=None,  # type: list[str] | str | None
     modified_components=None,  # type: list[str] | str | None
     modified_files=None,  # type: list[str] | str | None
-    ignore_app_dependencies_filepatterns=None,  # type: list[str] | str | None
+    ignore_component_dependencies_file_patterns=None,  # type: list[str] | str | None
     sdkconfig_defaults=None,  # type: str | None
 ):  # type: (...) -> list[App]
     """
@@ -137,9 +139,9 @@ def find_apps(
     :type modified_components: list[str] | str | None
     :param modified_files: modified files
     :type modified_files: list[str] | str | None
-    :param ignore_app_dependencies_filepatterns: file patterns that used for ignoring checking the component
+    :param ignore_component_dependencies_file_patterns: file patterns that used for ignoring checking the component
         dependencies
-    :type ignore_app_dependencies_filepatterns: list[str] | str | None
+    :type ignore_component_dependencies_file_patterns: list[str] | str | None
     :param sdkconfig_defaults: semicolon-separated string, pass to idf.py -DSDKCONFIG_DEFAULTS if specified,
         also could be set via environment variables "SDKCONFIG_DEFAULTS"
     :type sdkconfig_defaults: str | None
@@ -164,7 +166,7 @@ def find_apps(
 
     modified_components = to_list(modified_components)
     modified_files = to_list(modified_files)
-    ignore_app_dependencies_filepatterns = to_list(ignore_app_dependencies_filepatterns)
+    ignore_component_dependencies_file_patterns = to_list(ignore_component_dependencies_file_patterns)
 
     apps = []
     if target == 'all':
@@ -193,7 +195,7 @@ def find_apps(
                         manifest_rootpath=manifest_rootpath,
                         modified_components=modified_components,
                         modified_files=modified_files,
-                        ignore_app_dependencies_filepatterns=ignore_app_dependencies_filepatterns,
+                        ignore_component_dependencies_file_patterns=ignore_component_dependencies_file_patterns,
                     ),
                     modified_components=modified_components,
                     modified_files=modified_files,
@@ -221,7 +223,7 @@ def build_apps(
     manifest_rootpath=None,  # type: str | None
     modified_components=None,  # type: list[str] | str | None
     modified_files=None,  # type: list[str] | str | None
-    ignore_app_dependencies_filepatterns=None,  # type: list[str] | str | None
+    ignore_component_dependencies_file_patterns=None,  # type: list[str] | str | None
 ):  # type: (...) -> (int, list[App]) | int
     """
     Build all the specified apps
@@ -256,9 +258,9 @@ def build_apps(
     :type modified_components: list[str] | str | None
     :param modified_files: modified files
     :type modified_files: list[str] | str | None
-    :param ignore_app_dependencies_filepatterns: file patterns that used for ignoring checking the component
+    :param ignore_component_dependencies_file_patterns: file patterns that used for ignoring checking the component
         dependencies
-    :type ignore_app_dependencies_filepatterns: list[str] | str | None
+    :type ignore_component_dependencies_file_patterns: list[str] | str | None
     :return: (exit_code, built_apps) if specified ``modified_components``
     :rtype: int, list[App]
     :return: exit_code if not specified ``modified_components``
@@ -267,7 +269,7 @@ def build_apps(
     apps = to_list(apps)  # type: list[App]
     modified_components = to_list(modified_components)
     modified_files = to_list(modified_files)
-    ignore_app_dependencies_filepatterns = to_list(ignore_app_dependencies_filepatterns)
+    ignore_component_dependencies_file_patterns = to_list(ignore_component_dependencies_file_patterns)
 
     ignore_warnings_regexes = []
     if ignore_warning_strs:
@@ -298,13 +300,27 @@ def build_apps(
         app.parallel_count = parallel_count
 
         if collect_app_info:
-            app._collect_app_info = collect_app_info
+            if isinstance(collect_app_info, io.TextIOWrapper):
+                warnings.warn(
+                    '"collect_app_info" does not support file stream in idf-build-apps 1.0.0, Please use str instead',
+                    DeprecationWarning,
+                )
+                app._collect_app_info = collect_app_info.name
+            else:
+                app._collect_app_info = collect_app_info
 
             if app.collect_app_info not in collect_files:
                 collect_files.append(app.collect_app_info)
 
         if collect_size_info:
-            app._collect_size_info = collect_size_info
+            if isinstance(collect_size_info, io.TextIOWrapper):
+                warnings.warn(
+                    '"collect_size_info" does not support file stream in idf-build-apps 1.0.0, Please use str instead',
+                    DeprecationWarning,
+                )
+                app._collect_size_info = collect_size_info.name
+            else:
+                app._collect_size_info = collect_size_info
 
             if app.collect_size_info not in collect_files:
                 collect_files.append(app.collect_size_info)
@@ -336,7 +352,7 @@ def build_apps(
                 modified_components=modified_components,
                 modified_files=modified_files,
                 check_app_dependencies=_check_app_dependency(
-                    manifest_rootpath, modified_components, modified_files, ignore_app_dependencies_filepatterns
+                    manifest_rootpath, modified_components, modified_files, ignore_component_dependencies_file_patterns
                 ),
             )
         except BuildError as e:
@@ -452,7 +468,15 @@ class IdfBuildAppsCliFormatter(argparse.HelpFormatter):
             else:
                 default_type = type(action.default)
 
-            if action.nargs in [argparse.ZERO_OR_MORE, argparse.ONE_OR_MORE]:
+            if isinstance(action, argparse._AppendAction):  # noqa
+                _help += (
+                    '. Could be specified for multiple times'
+                    '{} ! DeprecationWarning: will change to space-separated list in idf-build-apps 1.0.0 version'.format(
+                        self.LINE_SEP
+                    )
+                )
+                _type = 'list[{}]'.format(default_type.__name__)
+            elif action.nargs in [argparse.ZERO_OR_MORE, argparse.ONE_OR_MORE]:
                 _type = 'list[{}]'.format(default_type.__name__)
             else:
                 _type = default_type.__name__
@@ -498,7 +522,11 @@ def get_parser():  # type: () -> argparse.ArgumentParser
         action='store_true',
         help='Look for apps in the specified paths recursively',
     )
-    common_args.add_argument('--exclude', nargs='+', help='Ignore specified path (if --recursive is given)')
+    common_args.add_argument(
+        '--exclude',
+        action='append',
+        help='Ignore specified directory (if --recursive is given)',
+    )
     common_args.add_argument(
         '--work-dir',
         help='If set, the app is first copied into the specified directory, and then built. '
@@ -520,7 +548,7 @@ def get_parser():  # type: () -> argparse.ArgumentParser
     )
     common_args.add_argument(
         '--config',
-        nargs='+',
+        action='append',
         help='Adds configurations (sdkconfig file names) to build. '
         'This can either be FILENAME[=NAME] or FILEPATTERN. FILENAME is the name of the sdkconfig file, '
         'relative to the project directory, to be used. Optional NAME can be specified, '
@@ -550,7 +578,7 @@ def get_parser():  # type: () -> argparse.ArgumentParser
     )
     common_args.add_argument(
         '--manifest-file',
-        nargs='+',
+        action='append',
         help='Manifest files which specify the build test rules of the apps',
     )
     common_args.add_argument(
@@ -562,7 +590,10 @@ def get_parser():  # type: () -> argparse.ArgumentParser
         '--default-build-targets',
         nargs='+',
         help='space-separated list of supported targets. Targets supported in current ESP-IDF branch '
-        '(except preview ones) would be used if this option is not set.',
+        '(except preview ones) would be used if this option is not set.'
+        '{} ! DeprecationWarning: comma-separated list support will be removed in idf-build-apps 1.0.0 version'.format(
+            IdfBuildAppsCliFormatter.LINE_SEP
+        ),
     )
     common_args.add_argument(
         '--modified-components',
@@ -581,12 +612,13 @@ def get_parser():  # type: () -> argparse.ArgumentParser
     )
     common_args.add_argument(
         '-if',
-        '--ignore-app-dependencies-filepatterns',
+        '--ignore-component-dependencies-file-patterns',
         nargs='*',
         default=None,
-        help='space-separated list which specifies the file patterns used for ignoring checking the app dependencies. '
-        'The `depends_components` and `depends_filepatterns` set in the manifest files will be ignored when any of the '
-        'specified file patterns matches any of the modified files. Must be used together with --modified-files',
+        help='space-separated list which specifies the file patterns used for ignoring the component dependencies. '
+        'The `depends_components` and `depends_filepatterns` set in the manifest files will be ignored when any of '
+        'the specified file patterns matches any of the modified files. Must be used together with '
+        '--modified-files',
     )
 
     common_args.add_argument(
@@ -643,7 +675,7 @@ def get_parser():  # type: () -> argparse.ArgumentParser
     )
     build_parser.add_argument(
         '--ignore-warning-str',
-        nargs='+',
+        action='append',
         help='Ignore the warning string that match the specified regex in the build output',
     )
     build_parser.add_argument(
@@ -680,17 +712,20 @@ def validate_args(parser, args):  # type: (argparse.ArgumentParser, argparse.Nam
     default_build_targets = []
     if args.default_build_targets:
         for target in args.default_build_targets:
-            if target not in ALL_TARGETS:
-                raise InvalidCommand(
-                    'Unrecognizable target {} specified with "--default-build-targets". '
-                    'Current ESP-IDF available targets: {}'.format(target, ALL_TARGETS)
-                )
+            t_list = [_t.strip() for _t in target.split(',')] if ',' in target else [target.strip()]
+            for _t in t_list:
+                if _t not in ALL_TARGETS:
+                    raise InvalidCommand(
+                        'Unrecognizable target {} specified with "--default-build-targets". '
+                        'Current ESP-IDF available targets: {}'.format(_t, ALL_TARGETS)
+                    )
+
+                if _t not in default_build_targets:
+                    default_build_targets.append(_t)
 
-            if target not in default_build_targets:
-                default_build_targets.append(target)
     args.default_build_targets = default_build_targets
 
-    if args.ignore_app_dependencies_filepatterns:
+    if args.ignore_component_dependencies_file_patterns:
         if args.modified_files is None:
             raise InvalidCommand(
                 'Must specify "--ignore-component-dependencies-file-patterns" with "--modified-files", '
@@ -732,7 +767,7 @@ def main():
         default_build_targets=args.default_build_targets,
         modified_components=args.modified_components,
         modified_files=args.modified_files,
-        ignore_app_dependencies_filepatterns=args.ignore_app_dependencies_filepatterns,
+        ignore_component_dependencies_file_patterns=args.ignore_component_dependencies_file_patterns,
         sdkconfig_defaults=args.sdkconfig_defaults,
     )
 
@@ -766,7 +801,7 @@ def main():
         manifest_rootpath=args.manifest_rootpath,
         modified_components=args.modified_components,
         modified_files=args.modified_files,
-        ignore_app_dependencies_filepatterns=args.ignore_app_dependencies_filepatterns,
+        ignore_component_dependencies_file_patterns=args.ignore_component_dependencies_file_patterns,
     )
 
     if args.modified_components is not None:

{idf_build_apps-1.0.0 → idf_build_apps-1.0.0.dev1}/idf_build_apps/manifest/if_parser.py

@@ -1,7 +1,6 @@
 # SPDX-FileCopyrightText: 2022-2023 Espressif Systems (Shanghai) CO LTD
 # SPDX-License-Identifier: Apache-2.0
 
-import os
 from ast import (
     literal_eval,
 )
@@ -87,10 +86,6 @@ class ChipAttr(Stmt):
         if self.attr in SOC_HEADERS[target]:
             return SOC_HEADERS[target][self.attr]
 
-        # for non-keyword cap words, check if it is defined in the environment variables
-        if self.attr in os.environ:
-            return os.environ[self.attr]
-
         return 0  # default return 0 as false
 
 

{idf_build_apps-1.0.0 → idf_build_apps-1.0.0.dev1}/idf_build_apps/utils.py

@@ -213,7 +213,7 @@ def files_matches_patterns(
     patterns,  # type: list[str] | str
     rootpath=None,  # type: str
 ):  # type: (...) -> bool
-    # can't match an absolute pattern with a relative path
+    # can't match a absolute pattern with a relative path
     # change all to absolute paths
     files = [to_absolute_path(f, rootpath) for f in to_list(files)]
     patterns = [to_absolute_path(p, rootpath) for p in to_list(patterns)]

{idf_build_apps-1.0.0 → idf_build_apps-1.0.0.dev1}/setup.py

@@ -26,7 +26,7 @@ entry_points = \
 {'console_scripts': ['idf-build-apps = idf_build_apps:main.main']}
 
 setup(name='idf-build-apps',
-      version='1.0.0',
+      version='1.0.0.dev1',
       description='Tools for building ESP-IDF related apps.',
       author=None,
       author_email='Fu Hanxi <fuhanxi@espressif.com>',

{idf_build_apps-1.0.0 → idf_build_apps-1.0.0.dev1}/tests/test_build.py

@@ -4,9 +4,6 @@ import shutil
 
 import pytest
 
-from idf_build_apps import (
-    find_apps,
-)
 from idf_build_apps.app import (
     CMakeApp,
 )
@@ -77,20 +74,6 @@ class TestBuild:
 
         assert built == is_built
 
-    def test_build_without_modified_components_but_ignored_app_dependency_check(self):
-        test_dir = str(IDF_PATH / 'examples' / 'get-started' / 'hello_world')
-
-        apps = find_apps(
-            test_dir,
-            'esp32',
-            modified_components=[],
-            modified_files=['foo.c'],
-            ignore_app_dependencies_filepatterns=['foo.c'],
-        )
-
-        for app in apps:
-            assert app.build()
-
 
 @pytest.mark.skipif(not shutil.which('idf.py'), reason='idf.py not found')
 def test_idf_version_keywords_type():

{idf_build_apps-1.0.0 → idf_build_apps-1.0.0.dev1}/tests/test_finder.py

@@ -3,7 +3,6 @@
 
 import logging
 import os
-import tempfile
 from pathlib import (
     Path,
 )
@@ -418,84 +417,3 @@ class TestFindWithSdkconfigFiles:
         assert apps[0].sdkconfig_files == [
             str(tmp_path / 'test1' / 'sdkconfig.defaults'),
         ]
-
-    def test_env_var(self, tmp_path, monkeypatch):
-        create_project('test1', tmp_path)
-
-        (tmp_path / 'test1' / 'sdkconfig.ci.foo').touch()
-        (tmp_path / 'test1' / 'sdkconfig.ci.bar').touch()
-        (tmp_path / 'test1' / 'sdkconfig.ci.baz').touch()
-
-        yaml_file = tmp_path / 'test.yml'
-        yaml_file.write_text(
-            f'''
-{tmp_path}:
-  enable:
-    - if: CONFIG_NAME == "foo" and IDF_TARGET == "esp32"
-    - if: CONFIG_NAME == "bar" and IDF_TARGET == "esp32s2"
-    - if: TEST_ENV_VAR == "1"
-    - if: CONFIG_NAME == "baz" and TEST_ENV_VAR == 0
-''',
-            encoding='utf8',
-        )
-
-        # in case you set it...
-        monkeypatch.delenv('CONFIG_NAME', raising=False)
-        monkeypatch.delenv('TEST_ENV_VAR', raising=False)
-
-        # CONFIG_NAME should NOT be overridden by env var
-        monkeypatch.setenv('CONFIG_NAME', 'bar')
-        apps = find_apps(
-            str(tmp_path / 'test1'),
-            'esp32',
-            config_rules_str=['sdkconfig.ci=default', 'sdkconfig.ci.*='],
-            manifest_files=yaml_file,
-        )
-        assert len(apps) == 2
-        assert apps[0].sdkconfig_files == [
-            str(tmp_path / 'test1' / 'sdkconfig.ci.baz'),
-        ]
-        assert apps[1].sdkconfig_files == [
-            str(tmp_path / 'test1' / 'sdkconfig.ci.foo'),
-        ]
-        monkeypatch.delenv('CONFIG_NAME')
-
-        # env var should be expanded
-        monkeypatch.setenv('TEST_ENV_VAR', '1')
-        apps = find_apps(
-            str(tmp_path / 'test1'),
-            'esp32',
-            config_rules_str=['sdkconfig.ci=default', 'sdkconfig.ci.*='],
-            manifest_files=yaml_file,
-        )
-        assert len(apps) == 3
-        monkeypatch.delenv('TEST_ENV_VAR')
-
-
-@pytest.mark.parametrize(
-    'exclude_list, apps_count',
-    [
-        (['test1'], 3),  # not excluded
-        (['folder1/test2'], 2),
-        (['folder1'], 1),
-        (['folder2/test2'], 2),
-    ],
-)
-def test_find_apps_with_exclude(tmp_path, exclude_list, apps_count):
-    (tmp_path / 'folder1').mkdir()
-    (tmp_path / 'folder2').mkdir()
-
-    create_project('test1', tmp_path / 'folder1')
-    create_project('test2', tmp_path / 'folder1')
-    create_project('test2', tmp_path / 'folder2')
-
-    os.chdir(tmp_path)
-    apps = find_apps(str(tmp_path), 'esp32', recursive=True, exclude_list=exclude_list)
-    assert len(apps) == apps_count
-
-    # or with absolute_path
-    exclude_list = [os.path.abspath(x) for x in exclude_list]
-    tmp_path = os.path.abspath(tmp_path)
-    os.chdir(tempfile.tempdir)
-    apps = find_apps(str(tmp_path), 'esp32', recursive=True, exclude_list=exclude_list)
-    assert len(apps) == apps_count