Skip to content

testing the exe of main.py#176

Open
DagaBhai wants to merge 8 commits into
pyfenn:mainfrom
DagaBhai:temptesting
Open

testing the exe of main.py#176
DagaBhai wants to merge 8 commits into
pyfenn:mainfrom
DagaBhai:temptesting

Conversation

@DagaBhai

@DagaBhai DagaBhai commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

for #136
test_check_all_templates_dir
Verifies that all expected template directories are present in the templates folder and that no unexpected template directories exist
test_check_for_main
Ensures every template contains a main.py file, which serves as the template's entry point
test_check_for_requirement
Ensures every template contains a requirements.txt file listing its Python dependencies
test_main
Validates template executability by installing the template's dependencies, running its main.py script, and then uninstalling the dependencies before proceeding to the next template

Refactor template directory path handling in tests.
@DagaBhai DagaBhai marked this pull request as ready for review July 4, 2026 05:25
DagaBhai added 4 commits July 4, 2026 11:04
Updated TEMPLATE_DIR path to reflect new directory structure and added directory check in the list comprehension.
@DagaBhai

DagaBhai commented Jul 4, 2026

Copy link
Copy Markdown
Contributor Author

@blkdmr ig its working, the job is failing because multiple templates are missing requirements.txt files or their main.py
also the fenn repository (the parent repository) is currently pointing to an older commit of the templates submodule.

DagaBhai added 2 commits July 4, 2026 11:36
Updated subprocess.run to capture output and handle errors.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant