-
Notifications
You must be signed in to change notification settings - Fork 73
Increase coverage #518
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
Draft
allnes
wants to merge
52
commits into
learning-process:master
Choose a base branch
from
allnes:an/increase-coverage-1
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Increase coverage #518
Conversation
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
…_programming_course # Conflicts: # cmake/gtest.cmake # cmake/json.cmake # cmake/libenvpp.cmake # cmake/mpi.cmake # cmake/onetbb.cmake # cmake/openmp.cmake # cmake/stb.cmake
…ormance measurement logic, and add utility for result string conversion
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #518 +/- ##
==========================================
- Coverage 94.72% 93.66% -1.06%
==========================================
Files 11 18 +7
Lines 360 505 +145
Branches 122 58 -64
==========================================
+ Hits 341 473 +132
- Misses 5 12 +7
- Partials 14 20 +6 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
…_programming_course into an/increase-coverage-1 # Conflicts: # modules/performance/include/performance.hpp
…rallel_programming_course into an/increase-coverage-1
…ormatting for better readability
…on, and enhance namespace usage
…k`, and refine `perf` module tests
…nMP validation test
…ance clarity and readability
…ance clarity and readability
…ases to enhance clarity and readability
…st cases to enhance clarity and readability
…ses for enhanced clarity and readability
…hance clarity with updated comments in OpenMP thread control tests
…ter test coverage and clarity
…improved readability
…tom terminate handler to enhance test flexibility
… simplify test logic
…ation and test comments for consistency and formatting improvement
…de and improve docstring formatting for clarity
…ved readability and maintainability
…or enhanced clarity and structure
… simplify coverage report generation
328e530
to
bdb4ef2
Compare
…olidating file searches and removing redundant path logic
…py and add gcov error ignoring for coverage generation
…un logic for compatibility and flexibility
…increase-coverage-1
…rings, update workflow logic for coverage generation consistency
… adherence to conventions
…st logic and update GitHub Actions for environment consistency
…file checks, and updated paths
…s for consistency and simplify file generation logic
…ecks, enhanced debug outputs, and refined command logic
…proved code clarity
…k-specific files and ensure unique entries
This commit introduces tests for various edge cases, including handling of built-in types, anonymous structs, deep namespace chains, special mangling cases, complex templates, and simulated MSVC behavior on non-Windows platforms. It ensures robust coverage for demangling logic and namespace extraction across diverse scenarios.
This commit introduces tests for various scenarios, including anonymous namespaces, global types, function pointers, arrays, nested templates, reference and pointer types, `std` types, const/volatile qualifiers, and lambda types. It ensures thorough validation of namespace extraction and demangling logic.
Adjusted tests for extremely long type names and `std::nullptr_t` to account for potential compiler differences in namespace resolution. Ensures compatibility with varying demangling behaviors.
Refactored tests to improve clarity and robustness. Updated type definitions to modern equivalents (`std::array`, `enum` with explicit width, etc.) and replaced namespace definitions with inline syntax. Expanded test cases for built-in, unsigned, floating-point, and character types to ensure comprehensive coverage of edge scenarios.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
No description provided.