-
Notifications
You must be signed in to change notification settings - Fork 2
Push Oxomium 3.7 #105
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
Open
pep-un
wants to merge
27
commits into
main
Choose a base branch
from
dev
base: main
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.
Open
Push Oxomium 3.7 #105
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
Will help for all requierement related business. Fix #88
Implementing Save&Next, Fix #100 Clean up tree update logic Display update
Clean up et reorganise signal in a dedicated signal.py Implement automatic archiving logics when action are closed. Fix #77
Bug detected by unit test !
Moved and split tests from tests.py into dedicated files under conformity/test/, including test_auth.py, test_middleware.py, and test_sigals.py. Renamed tests.py to tests_models.py and removed duplicated or redundant test classes. This improves test organization, coverage, and maintainability. Implement test on signal.py. Improve authentication test my making it automatic (no manual class listing)
Added source encoding, sources, and test paths. Refined exclusions and test inclusions for more accurate analysis and coverage reporting.
Moved all files from 'conformity/test/' to 'conformity/tests/' to follow standard naming conventions for test directories.
Simplifies the test for Attachment pre_save mime_type population by using SimpleUploadedFile instead of a temporary file. Adds explicit cleanup of both the database record and the underlying file from storage to ensure no residual files remain after the test.
Introduces test cases for HomeView, FindingIndexView, ConformityIndexView, and ConformityDetailIndexView. Includes a BaseDataMixin to set up minimal test data for organizations, frameworks, requirements, conformities, audits, findings, actions, controls, and control points.
Introduces minimal coverage for Conformity.get_related() in negative_only mode, verifying related actions and control points are surfaced. Adds a light sanity check for the Attachment model's string representation, ensuring filename suffix and base name are present.
Added .sonarlint/connectedMode.json to .gitignore to prevent local SonarLint configuration from being tracked in version control.
Introduces ConformitySaveNextTests to verify that the 'save_next' action in ConformityUpdateView correctly redirects to the next sibling conformity or falls back to the default success URL if no sibling exists. Also refactors test class names for consistency.
Introduces new test cases for Framework language choices, conformity relations and guards, audit and finding helpers, control and control point logic, and action model helpers. These tests improve coverage and verify model behaviors, propagation logic, status guards, URL generation, and side effects.
Introduces unit tests for Conformity model helpers, guards, and related model interactions. Tests cover tree navigation, action/control retrieval, status aggregation, responsible/applicable propagation, and guard logic for status updates.
|
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.