Skip to content

Add pytest and CI workflow#20

Merged
thomas-bc merged 21 commits intofprime-community:develfrom
rmzmrnn:create-pytest
Sep 11, 2025
Merged

Add pytest and CI workflow#20
thomas-bc merged 21 commits intofprime-community:develfrom
rmzmrnn:create-pytest

Conversation

@rmzmrnn
Copy link
Contributor

@rmzmrnn rmzmrnn commented Jun 20, 2025

Related Issue(s) fprime #2551
Has Unit Tests (y/n) y
Documentation Included (y/n) n

Change Description

fprime-bootstrap-ci.yml

  • Added a CI workflow for testing fprime-bootstrap.

.gitignore

  • Ignored the tmp/ folder used during tests.

pyproject.toml

  • Registered custom pytest markers.

test_bootstrap.py

  • Implemented initial pytest test suite for fprime-bootstrap.

Rationale

Fixes Add tests to fprime-bootstrap repo #2551

Testing/Review Recommendations

To run all tests:

pytest test/ -v

To run specific tests:

  • All tests except clone:

    pytest test/ -m bootstrap -v
    
  • Only project command:

    pytest test/ -m project -v
    
  • Only clone command:

    pytest test/ -m clone -v
    
  • Check for no *-template files:

    pytest test/ -m template -v
    
  • Check if all {{FPRIME_PROJECT_NAME}} placeholders are replaced:

    pytest test/ -m replace -v
    

Future Work

None

Appendix

❌ Failure example: test_no_template_files()

image

❌ Failure example: test_fprime_project_name_replace()

image

@rmzmrnn
Copy link
Contributor Author

rmzmrnn commented Jun 20, 2025

Hi @thomas-bc 👋
When you have time, I'd appreciate a review on this PR. Let me know if anything needs to be changed or improved. Thanks!

@LeStarch LeStarch requested a review from thomas-bc September 3, 2025 22:36
@thomas-bc
Copy link
Member

Very sorry for the delay here. Closing and re-opening to trigger CI

@thomas-bc thomas-bc closed this Sep 11, 2025
@thomas-bc thomas-bc reopened this Sep 11, 2025
thomas-bc and others added 5 commits September 11, 2025 14:24
* Updating fprime-bootstrap to add namespace folders

* Adding requirements.txt helper

* Add different repo-name and NamespaceName

* formatting

---------

Co-authored-by: M Starch <LeStarch@googlemail.com>
Copy link
Member

@thomas-bc thomas-bc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I went ahead and updated the tests to work with the latest state of this repo.
Thanks a lot for contributing!

@thomas-bc thomas-bc merged commit 1306a0b into fprime-community:devel Sep 11, 2025
8 checks passed
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.

Add tests to fprime-bootstrap repo

3 participants