Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Extend basically non existing unit tests for 100% coverage on all metrics #5

Open
torsknod2 opened this issue Feb 14, 2025 · 1 comment · May be fixed by #51
Open

Extend basically non existing unit tests for 100% coverage on all metrics #5

torsknod2 opened this issue Feb 14, 2025 · 1 comment · May be fixed by #51
Labels
aspice-relevant Reduces ASPICE gaps enhancement New feature or request good first issue Good for newcomers
Milestone
0.1.0

Comments

@torsknod2
Copy link
Owner

torsknod2 commented Feb 14, 2025 •
edited
Loading

Extend basically nearly non existing unit tests for 100% coverage on all metrics.

See e.g. https://app.codecov.io/gh/torsknod2/ieee1788-ada/blob/MergeWorkflowsAndEnablePushToMainTrigger/src%2Fieee1788.adb, we are not at 2.6%, which is not surprising.

For that, have one test suite for each function.
Have one function call per test case.
@torsknod2 torsknod2 added enhancement New feature or request good first issue Good for newcomers labels Feb 14, 2025
@codeautopilot CodeAutopilot
Copy link

codeautopilot bot commented Feb 15, 2025 •
edited
Loading

Your organization has reached the subscribed usage limit. You can upgrade your account by purchasing a subscription at Stripe payment link

This comment was generated by AI. Information provided may be incorrect.

Current plan usage: 100%

Have feedback or need help?
Documentation
support@codeautopilot.com

@torsknod2 torsknod2 changed the title Extend basically non existing unit test for 100% coverage on all metrics Extend basically non existing unit tests for 100% coverage on all metrics Feb 16, 2025
@torsknod2 torsknod2 added this to the 0.1.0 milestone Feb 18, 2025
@torsknod2 torsknod2 added the aspice-relevant Reduces ASPICE gaps label Feb 18, 2025
@torsknod2 torsknod2 mentioned this issue Feb 19, 2025
Merged
torsknod2 added a commit that referenced this issue Feb 19, 2025
@torsknod2
Enhance Project Automation, CI/CD, and Documentation (#50)
55e4236 
This PR introduces comprehensive improvements to streamline development
workflows and enhance project quality, addressing several open issues:

# Project Automation 🤖
- Add automatic project assignment workflow for issues and PRs (Closes
#37)
  - Automatically assigns new issues/PRs to project boards
  - Includes bulk assignment capability for existing items
  - Implements secure runner configuration with egress traffic control

# CI/CD and Quality Assurance ✅
- Enhance test result visibility and coverage reporting
  - Add AUnit XML test results publishing to Codecov (Closes #26)
  - Implement automated PR comments for test results (Closes #25)
  - Remove redundant Python setup in coverage workflow
- Implement robust version management
  - Add version tag validation against alire.toml
  - Add Python scripts for semantic version checking
- Ensure version consistency across the project (Partially addresses
#36)

# Documentation and Developer Experience 📚
- Improve project documentation
  - Add status badges for issue metrics (Closes #48)
  - Enhance API documentation section
  - Add links to test coverage and verification results
  - Update GitHub page information (Closes #27)
- Configure GitHub Copilot chat locale to English

# Technical Implementation
- New Workflows:
   - `.github/workflows/automatic-project-assignment.yaml`
   - `.github/workflows/ensure_tags_match_alire_toml_version.yaml`
- Enhanced Workflows:
   - Modified `.github/workflows/cicd.yaml` for test results

# Related Issues
This PR contributes to the following issues:
- Project Management (#38)
- Configuration Management (#23)
- Unit Test Coverage (#5)

Note: While this PR doesn't fully resolve all related issues, it lays
the groundwork for better project management, documentation, and test
coverage tracking.
@torsknod2 torsknod2 linked a pull request Feb 20, 2025 that will close this issue
Reach 100% STMT+MC/DC coverage #51
Draft
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
aspice-relevant Reduces ASPICE gaps enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Milestone
0.1.0
Development

Successfully merging a pull request may close this issue.

Reach 100% STMT+MC/DC coverage torsknod2/ieee1788-ada
1 participant
@torsknod2