strictdoc 0.0.49a2__tar.gz → 0.0.49a3__tar.gz

{strictdoc-0.0.49a2 → strictdoc-0.0.49a3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: strictdoc
-Version: 0.0.49a2
+Version: 0.0.49a3
 Summary: StrictDoc is open-source software for technical documentation and requirements management.
 Project-URL: Changelog, https://github.com/strictdoc-project/strictdoc/blob/main/CHANGELOG.md
 Project-URL: Homepage, https://strictdoc.readthedocs.io/en/stable/

{strictdoc-0.0.49a2 → strictdoc-0.0.49a3}/mypy.ini

@@ -22,8 +22,12 @@ ignore_missing_imports = True
 ignore_missing_imports = True
 
 # We ignore build folder but mypy still follows into the Python packages,
-# in this case starlette. This StackOverflow answers suggests a solution.
+# in this case pydantic, starlette, selenium.
+# This StackOverflow answers suggests a solution:
 # https://stackoverflow.com/a/70367929/598057
+[mypy-pydantic.*]
+follow_imports = skip
+
 [mypy-starlette.*]
 follow_imports = skip
 

{strictdoc-0.0.49a2 → strictdoc-0.0.49a3}/requirements.check.txt

@@ -20,7 +20,7 @@ pytidylib>=0.3.2
 openpyxl>=3.0.5
 
 # End2end tests
-seleniumbase; python_version<="3.7"
+seleniumbase
 
 # psutil is needed to reap Uvicorn's zombie processes when running end2end
 # tests. One day someone finds a better solution.

{strictdoc-0.0.49a2 → strictdoc-0.0.49a3}/strictdoc/__init__.py

@@ -1,6 +1,6 @@
 from strictdoc.core.environment import SDocRuntimeEnvironment
 
-__version__ = "0.0.49a2"
+__version__ = "0.0.49a3"
 
 
 environment = SDocRuntimeEnvironment(__file__)

{strictdoc-0.0.49a2 → strictdoc-0.0.49a3}/strictdoc/backend/sdoc/error_handling.py

@@ -5,6 +5,7 @@ from strictdoc.backend.sdoc.models.document_grammar import (
     GrammarElement,
     GrammarElementField,
 )
+from strictdoc.backend.sdoc.models.document_view import ViewElement
 from strictdoc.backend.sdoc.models.reference import Reference
 from strictdoc.backend.sdoc.models.requirement import (
     Requirement,
@@ -291,6 +292,71 @@ class StrictDocSemanticError(Exception):
             filename=filename,
         )
 
+    @staticmethod
+    def view_references_nonexisting_grammar_element(
+        view_element: ViewElement,
+        object_type: str,
+        line=None,
+        col=None,
+        filename=None,
+    ):
+        return StrictDocSemanticError(
+            title=(
+                f"View element '{view_element.view_id}' references a non-existing"
+                f" grammar element '{object_type}'."
+            ),
+            hint=(
+                "Make sure that each View element references an existing "
+                "object in the grammar or the default REQUIREMENT object."
+            ),
+            example=None,
+            line=line,
+            col=col,
+            filename=filename,
+        )
+
+    @staticmethod
+    def view_references_nonexisting_field(
+        view_element: ViewElement,
+        object_type: str,
+        field_name: str,
+        line=None,
+        col=None,
+        filename=None,
+    ):
+        return StrictDocSemanticError(
+            title=(
+                f"View element '{view_element.view_id}' references a non-existing"
+                f" field '{field_name}' for grammar element '{object_type}'."
+            ),
+            hint=(
+                "Make sure that each View element references an existing "
+                "field in the grammar for the given grammar element."
+            ),
+            example=None,
+            line=line,
+            col=col,
+            filename=filename,
+        )
+
+    @staticmethod
+    def default_view_doesnt_exist(
+        default_view: str, line=None, col=None, filename=None
+    ):
+        return StrictDocSemanticError(
+            title=(
+                f"Default view '{default_view}' does not exist in the document."
+            ),
+            hint=(
+                "Make sure that the specified default view is created in the "
+                "VIEWS configuration."
+            ),
+            example=None,
+            line=line,
+            col=col,
+            filename=filename,
+        )
+
     def to_print_message(self):
         message = ""
         message += f"error: could not parse file: {self.file_path}.\n"

{strictdoc-0.0.49a2 → strictdoc-0.0.49a3}/strictdoc/backend/sdoc/models/constants.py

@@ -8,6 +8,10 @@ from strictdoc.backend.sdoc.models.document_grammar import (
     DocumentGrammar,
     GrammarElement,
 )
+from strictdoc.backend.sdoc.models.document_view import (
+    DocumentView,
+    ViewElement,
+)
 from strictdoc.backend.sdoc.models.fragment import Fragment
 from strictdoc.backend.sdoc.models.fragment_from_file import FragmentFromFile
 from strictdoc.backend.sdoc.models.inline_link import InlineLink
@@ -37,6 +41,9 @@ from strictdoc.backend.sdoc.models.type_system import (
     GrammarElementRelationChild,
     GrammarElementRelationFile,
     GrammarElementRelationParent,
+    ViewElementField,
+    ViewElementHiddenTag,
+    ViewElementTags,
 )
 
 SECTION_MODELS = [
@@ -62,6 +69,7 @@ DOCUMENT_MODELS = [
     DocumentConfig,
     Document,
     DocumentGrammar,
+    DocumentView,
     DocumentBibliography,
     GrammarElement,
     GrammarElementFieldString,
@@ -73,6 +81,10 @@ DOCUMENT_MODELS = [
     GrammarElementRelationParent,
     GrammarElementRelationChild,
     GrammarElementRelationFile,
+    ViewElement,
+    ViewElementField,
+    ViewElementTags,
+    ViewElementHiddenTag,
 ]
 DOCUMENT_MODELS.extend(SECTION_MODELS)
 

{strictdoc-0.0.49a2 → strictdoc-0.0.49a3}/strictdoc/backend/sdoc/models/document_view.py

@@ -11,11 +11,13 @@ from strictdoc.helpers.auto_described import auto_described
 class ViewElement:
     def __init__(
         self,
+        parent,
         view_id: str,
         tags: List[ViewElementTags],
         hidden_tags: Optional[List[ViewElementHiddenTag]],
         name: Optional[str],
     ):
+        self.parent = parent
         self.view_id: str = view_id
         self.tags: List[ViewElementTags] = tags
         self.hidden_tags: Optional[List[ViewElementHiddenTag]] = hidden_tags
@@ -23,5 +25,6 @@ class ViewElement:
 
 
 class DocumentView:
-    def __init__(self, views: List[ViewElement]):
+    def __init__(self, parent, views: List[ViewElement]):
+        self.parent = parent
         self.views: List[ViewElement] = views

{strictdoc-0.0.49a2 → strictdoc-0.0.49a3}/strictdoc/backend/sdoc/models/requirement.py

@@ -549,6 +549,9 @@ class Requirement(Node):  # pylint: disable=too-many-instance-attributes, too-ma
             self.requirement_type
         ]
         grammar_field_titles = list(map(lambda f: f.title, element.fields))
+        # FIXME: This will go away very soon when the RELATIONS become a
+        #        separate field in SDoc REQUIREMENT's grammar.
+        grammar_field_titles.append("REFS")
         field_index = grammar_field_titles.index(field_name)
         title_field_index = grammar_field_titles.index(
             RequirementFieldName.TITLE

{strictdoc-0.0.49a2 → strictdoc-0.0.49a3}/strictdoc/backend/sdoc/models/type_system.py

@@ -372,7 +372,8 @@ class GrammarElementFieldReference(GrammarElementField):
 
 @auto_described
 class ViewElementField:
-    def __init__(self, name: str, placement: Optional[str]):
+    def __init__(self, parent, name: str, placement: Optional[str]):
+        self.parent = parent
         self.name: str = name
         self.placement: Optional[str] = placement
 
@@ -380,13 +381,15 @@ class ViewElementField:
 @auto_described
 class ViewElementTags:
     def __init__(
-        self, object_type: str, visibile_fields: List[ViewElementField]
+        self, parent, object_type: str, visible_fields: List[ViewElementField]
     ):
+        self.parent = parent
         self.object_type: str = object_type
-        self.visible_fields: List[ViewElementField] = visibile_fields
+        self.visible_fields: List[ViewElementField] = visible_fields
 
 
 @auto_described
 class ViewElementHiddenTag:
-    def __init__(self, hidden_tag: str):
+    def __init__(self, parent, hidden_tag: str):
+        self.parent = parent
         self.hidden_tag: str = hidden_tag

{strictdoc-0.0.49a2 → strictdoc-0.0.49a3}/strictdoc/backend/sdoc/processor.py

@@ -7,6 +7,7 @@ from strictdoc.backend.sdoc.error_handling import StrictDocSemanticError
 from strictdoc.backend.sdoc.models.document import Document
 from strictdoc.backend.sdoc.models.document_config import DocumentConfig
 from strictdoc.backend.sdoc.models.document_grammar import DocumentGrammar
+from strictdoc.backend.sdoc.models.document_view import DocumentView
 from strictdoc.backend.sdoc.models.fragment import Fragment
 from strictdoc.backend.sdoc.models.fragment_from_file import FragmentFromFile
 from strictdoc.backend.sdoc.models.requirement import (
@@ -25,6 +26,7 @@ class ParseContext:
         self.document_grammar: DocumentGrammar = DocumentGrammar.create_default(
             None
         )
+        self.document_view: Optional[DocumentView] = None
         self.current_include_parent = None
         self.uses_old_refs_field: bool = False
         self.at_least_one_relations_field: bool = False
@@ -43,6 +45,7 @@ class SDocParsingProcessor:
             "Document": self.process_document,
             "DocumentConfig": self.process_document_config,
             "DocumentGrammar": self.process_document_grammar,
+            "DocumentView": self.process_document_view,
             "Section": self.process_section,
             "FragmentFromFile": self.process_include,
             "CompositeRequirement": self.process_composite_requirement,
@@ -63,6 +66,32 @@ class SDocParsingProcessor:
                 )
         self.parse_context.document_grammar = document_grammar
 
+    def process_document_view(self, document_view: DocumentView):
+        for view in document_view.views:
+            for tag in view.tags:
+                if (
+                    tag.object_type
+                    not in self.parse_context.document_grammar.registered_elements
+                ):
+                    raise StrictDocSemanticError.view_references_nonexisting_grammar_element(
+                        view,
+                        tag.object_type,
+                        **get_location(document_view),
+                    )
+                for field in tag.visible_fields:
+                    for (
+                        grammar_element
+                    ) in self.parse_context.document_grammar.elements:
+                        if field.name not in grammar_element.fields_map:
+                            raise StrictDocSemanticError.view_references_nonexisting_field(
+                                view,
+                                tag.object_type,
+                                field.name,
+                                **get_location(document_view),
+                            )
+
+        self.parse_context.document_view = document_view
+
     def process_section(self, section: Section):
         section.ng_document_reference = self.parse_context.document_reference
 

{strictdoc-0.0.49a2 → strictdoc-0.0.49a3}/strictdoc/backend/sdoc/reader.py

@@ -14,6 +14,7 @@ from strictdoc.backend.sdoc.models.constants import DOCUMENT_MODELS
 from strictdoc.backend.sdoc.models.document import Document
 from strictdoc.backend.sdoc.models.fragment import Fragment
 from strictdoc.backend.sdoc.processor import ParseContext, SDocParsingProcessor
+from strictdoc.backend.sdoc.validations.sdoc_validator import SDocValidator
 from strictdoc.helpers.cast import assert_cast
 from strictdoc.helpers.exception import StrictDocException
 from strictdoc.helpers.md5 import get_file_md5
@@ -65,17 +66,19 @@ class SDReader:
                 "  VALUE: REQ-1"
             )
 
+        return document, parse_context
+
+    @staticmethod
+    def read(input_string, file_path=None):
+        document, _ = SDReader._read(input_string, file_path)
+        SDocValidator.validate_document(document)
+
         # HACK:
         # ProcessPoolExecutor doesn't work because of non-picklable parts
         # of textx. The offending fields are stripped down because they
         # are not used anyway.
         drop_textx_meta(document)
 
-        return document, parse_context
-
-    @staticmethod
-    def read(input_string, file_path=None):
-        document, _ = SDReader._read(input_string, file_path)
         return document
 
     def read_from_file(self, file_path: str) -> Document:

strictdoc-0.0.49a3/strictdoc/backend/sdoc/validations/sdoc_validator.py

@@ -0,0 +1,38 @@
+from textx import get_location
+
+from strictdoc.backend.sdoc.error_handling import StrictDocSemanticError
+from strictdoc.backend.sdoc.models.document import Document
+from strictdoc.backend.sdoc.models.document_config import DocumentConfig
+
+
+class SDocValidator:
+    """
+    This helper class is used for validating SDoc documents right after the
+    textX parsing and processing steps have finished.
+    FIXME: In the future, all processing validation steps could be moved to this
+           class as well. This would remove the need in the ParseContext class
+           and simplify several other implementation details.
+    """
+
+    @staticmethod
+    def validate_document(document: Document):
+        SDocValidator._validate_document_config(document)
+
+    @staticmethod
+    def _validate_document_config(document: Document):
+        document_config: DocumentConfig = document.config
+        if document_config.default_view is not None:
+            if document.view is None:
+                raise StrictDocSemanticError.default_view_doesnt_exist(
+                    document_config.default_view,
+                    **get_location(document_config),
+                )
+            else:
+                view_names = map(
+                    lambda view_: view_.view_id, document.view.views
+                )
+                if document_config.default_view not in view_names:
+                    raise StrictDocSemanticError.default_view_doesnt_exist(
+                        document_config.default_view,
+                        **get_location(document_config),
+                    )