Skip to content

[feature/BLAZ-2467] to dev - #479

Merged
smaneroiriusrisk merged 5 commits into
devfrom
feature/BLAZ-2467
Oct 28, 2025
Merged

smaneroiriusrisk merged 5 commits into
devfrom
feature/BLAZ-2467

Conversation

@abausac

@abausac abausac commented Oct 16, 2025

Copy link
Copy Markdown
Contributor

Draw.io tests for startleft

@smaneroiriusrisk smaneroiriusrisk left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I miss the test for these two PROCESSOR use cases:

UC2 A (400)
in TestDrawioValidator.test_diagram_file_invalid_extensions

UC2 B (400)
in TestDrawioValidator.test_diagram_file_invalid_extensions

Comment thread tests/integration/api/controllers/diagram/test_otm_controller_diagram.py Outdated
Comment thread tests/integration/api/controllers/diagram/test_otm_controller_diagram.py Outdated
Comment thread tests/integration/api/controllers/diagram/test_otm_controller_diagram.py Outdated
Comment thread slp_drawio/tests/integration/test_drawio_processor.py Outdated
…ize usage and improved assertions: use of built-in list, and added cases for 'id' and 'name' test cases, removed not possible API message checking.
@abausac

abausac commented Oct 23, 2025

Copy link
Copy Markdown
Contributor Author

I miss the test for these two PROCESSOR use cases:

UC2 A (400) in TestDrawioValidator.test_diagram_file_invalid_extensions

UC2 B (400) in TestDrawioValidator.test_diagram_file_invalid_extensions

About UC2 A & B for PROCESSOR Drawio, the test cases were wrong referenced, but they were already created.
The reference was fixed, and it's TestOTMControllerDiagramDrawio.test_diagram_file_invalid_extensions, there're those needed tests.
The rest of requested changes were done in the last commit.

@smaneroiriusrisk
smaneroiriusrisk merged commit a4b9ee2 into dev Oct 28, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants