-
Notifications
You must be signed in to change notification settings - Fork 0
Add automated tests and CI integration with fs_tester support #11
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Co-authored-by: JohnAmadis <17320783+JohnAmadis@users.noreply.github.com>
Co-authored-by: JohnAmadis <17320783+JohnAmadis@users.noreply.github.com>
…lank lines Co-authored-by: JohnAmadis <17320783+JohnAmadis@users.noreply.github.com>
Co-authored-by: JohnAmadis <17320783+JohnAmadis@users.noreply.github.com>
Co-authored-by: JohnAmadis <17320783+JohnAmadis@users.noreply.github.com>
Co-authored-by: JohnAmadis <17320783+JohnAmadis@users.noreply.github.com>
|
@copilot Jeden z testów failuje na CI - napraw to proszę nawet jeśli to jest faktyczny błąd niezwiązany z tym PR Test command: /usr/bin/cmake "-E" "copy_if_different" "/__w/dmdevfs/dmdevfs/build_tests/dmf/dmdevfs.dmf" "/__w/dmdevfs/dmdevfs/build_tests/dmf/dmdevfs.dmf.test" |
Co-authored-by: JohnAmadis <17320783+JohnAmadis@users.noreply.github.com>
Fixed in commit 2783a15. The test was failing because the CI workflow wasn't building the module in the test directory before running tests. The workflow now runs |
Completed: Add Automatic Tests and CI Integration
Summary of Changes
This PR successfully implements automatic tests and CI integration for dmdevfs as requested in issue #10.
Latest Fix:
Fixed CI test failure by ensuring the module is built in the test directory before running tests. The test was failing because it tried to verify the existence of
dmdevfs.dmfwhich wasn't built in thebuild_testsdirectory.Files Modified/Added:
.github/workflows/ci.yml- Added test building and execution steps, fixed to build before testingCMakeLists.txt- Added DMDEVFS_BUILD_TESTS optionREADME.md- Added testing sectiontests/CMakeLists.txt- Build verification test suitetests/README.md- Comprehensive testing documentationtests/run_integration_tests.sh- Script for fs_tester integrationTest Coverage:
Automated CI Tests (running on every push/PR):
fs_tester Integration (documented for manual use):
Security:
Original prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.