test: Improve code coverage of mlia#13
Merged
Merged
Conversation
Resolves: MLIA-1416 Change-Id: I88e894987a0c2d2e786d59c76a31db0c99290317 Reviewed-on: https://eu-gerrit-2.euhpc.arm.com/c/ml/ecosystem/mlia/+/1146142 Tested-by: expkit <svc_expkit@arm.com> Reviewed-by: Wojciech Boncela <wojciech.boncela@arm.com> Reviewed-by: Isabella Gottardi <isabella.gottardi@arm.com> IP-review: Isabella Gottardi <isabella.gottardi@arm.com> Signed-off-by: Oluwatamilore Adebayo <oluwatamilore.adebayo@arm.com> Reviewed-on: https://eu-gerrit-2.euhpc.arm.com/c/ml/ecosystem/mlia/+/1156960
Resolves: MLIA-1428 tflite_compat.py:25: A line of top level code about handling imports tflite_convert.py:167: A __name__ == '__main__' entry point tflite_metrics.py:199: A branch that is only reached if an unaccounted for value of ReportClusterMode is given, of which there are none Change-Id: Ie399e0b69e93afcd5af30d15630eb2f6ff64b70f Reviewed-on: https://eu-gerrit-2.euhpc.arm.com/c/ml/ecosystem/mlia/+/1154593 Tested-by: expkit <svc_expkit@arm.com> Reviewed-by: Wojciech Boncela <wojciech.boncela@arm.com> IP-review: Isabella Gottardi <isabella.gottardi@arm.com> Signed-off-by: Oluwatamilore Adebayo <oluwatamilore.adebayo@arm.com> Reviewed-on: https://eu-gerrit-2.euhpc.arm.com/c/ml/ecosystem/mlia/+/1156967
Resolves: MLIA-1434 Change-Id: Ia3bfbcd270f6b90940334afd0f120eaa32219319 Reviewed-on: https://eu-gerrit-2.euhpc.arm.com/c/ml/ecosystem/mlia/+/1149054 Tested-by: expkit <svc_expkit@arm.com> Reviewed-by: Mike Kelly <mike.kelly@arm.com> IP-review: Mike Kelly <mike.kelly@arm.com> Signed-off-by: Oluwatamilore Adebayo <oluwatamilore.adebayo@arm.com> Reviewed-on: https://eu-gerrit-2.euhpc.arm.com/c/ml/ecosystem/mlia/+/1156986 Reviewed-by: Wojciech Boncela <wojciech.boncela@arm.com> IP-review: Isabella Gottardi <isabella.gottardi@arm.com>
Resolves: MLIA-1436 Change-Id: Ic2f235b565d1a7e9d71b8d4af5d3be976baf8803 Reviewed-on: https://eu-gerrit-2.euhpc.arm.com/c/ml/ecosystem/mlia/+/1147935 Reviewed-by: Isabella Gottardi <isabella.gottardi@arm.com> Tested-by: expkit <svc_expkit@arm.com> Reviewed-by: Mike Kelly <mike.kelly@arm.com> IP-review: Mike Kelly <mike.kelly@arm.com> Signed-off-by: Oluwatamilore Adebayo <oluwatamilore.adebayo@arm.com> Reviewed-on: https://eu-gerrit-2.euhpc.arm.com/c/ml/ecosystem/mlia/+/1157335 Reviewed-by: Wojciech Boncela <wojciech.boncela@arm.com> IP-review: Isabella Gottardi <isabella.gottardi@arm.com>
Signed-off-by: Maksims Svecovs <maksims.svecovs@arm.com> Change-Id: If931c24edf36007ccee3239e075b16386ea51837 Reviewed-on: https://eu-gerrit-2.euhpc.arm.com/c/ml/ecosystem/mlia/+/1149092 Reviewed-by: Isabella Gottardi <isabella.gottardi@arm.com> IP-review: Isabella Gottardi <isabella.gottardi@arm.com> Reviewed-on: https://eu-gerrit-2.euhpc.arm.com/c/ml/ecosystem/mlia/+/1158300 Tested-by: expkit <svc_expkit@arm.com>
oluwatamilore-arm
requested review from
IsabellaARM,
integratedintegral21 and
max-shvetsov
IsabellaARM
approved these changes
Dec 8, 2025
IsabellaARM
pushed a commit
that referenced
this pull request
May 19, 2026
* test: Migrate to shared pytest-native e2e framework Pull the generic MLIA end-to-end test framework out of mlia-ethos-u and integrate it into the core mlia repo. This replaces the older bespoke tests_e2e harness with a reusable pytest-native helper under src/mlia/testing/e2e.py and moves repo-facing e2e coverage into normal tests/ modules. - add src/mlia/testing/e2e.py with shared MLIA e2e helpers - add src/mlia/testing/__init__.py exports and README documentation - add tests/test_e2e.py as the thin repo-facing e2e suite - add tests/test_e2e_helpers.py for focused helper coverage - remove the legacy tests_e2e/ harness The new framework is driven by environment variables instead of a custom config-file runner: - MLIA_E2E_EXECUTIONS describes concrete CLI execution blocks in JSON - MLIA_E2E_ARTIFACTS points at the prepared artifact tree - MLIA_E2E_BACKENDS lists globally installed backends to prepare once - MLIA_E2E_SHARD_INDEX and MLIA_E2E_SHARD_COUNT support test sharding Each parametrized case now represents exactly one MLIA CLI invocation. Cases are expanded at collection time, staged into a temporary workdir, and executed independently. * fix: Make e2e helper validation lazy and secure artifact staging * fix: Format import test file * fix: Ruff ruff!
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Improve the code coverage of:
Other than top level code handling imports and entry points code coverage is at 100%.
Add a feature to use a custom tag suffix to the versioning if desired.