Manage support dev#253
Merged
Merged
Conversation
There are still multiple things to do but this is stable
E2e Test results - Vcast 2025sp0 - Group manage15 tests 15 ✅ 7m 3s ⏱️ Results for commit 39a4709. |
E2e Test results - Vcast 2025sp1 - Group manage15 tests 15 ✅ 6m 24s ⏱️ Results for commit 39a4709. ♻️ This comment has been updated with latest results. |
E2e Test results - Vcast 2025sp1 - Group manage_server15 tests 15 ✅ 6m 22s ⏱️ Results for commit 39a4709. ♻️ This comment has been updated with latest results. |
E2e Test results - Vcast 2025sp1 - Group basic_user_interactions_server 9 files ±0 9 suites ±0 10m 40s ⏱️ -21s Results for commit 39a4709. ± Comparison against base commit f14ee05. ♻️ This comment has been updated with latest results. |
E2e Test results - Vcast 2025sp0 - Group manage_server15 tests 15 ✅ 6m 24s ⏱️ Results for commit 39a4709. ♻️ This comment has been updated with latest results. |
Zbigor
approved these changes
May 19, 2025
Collaborator
Zbigor
left a comment
Zbigor
left a comment
There was a problem hiding this comment.
All comments have been addressed and all tests are passing.
SalvatoreZagaria
added a commit
that referenced
this pull request
Jul 1, 2025
Merged main including manage support into autoreq. I had to do some changes regarding the env ID from the testing pane. * Add libxcb dependencies to Ubuntu24 CI Dockerfile (#254) * Add libxkb dependencies to Ubuntu24 CI Dockerfile (#256) * Added fix for MCDC coverage & import_coded_tests for 2025+ (#255) Fixes the MCDC & import_coded_tests issues we have on 2025sp1 * Manage support dev (#253) ## Overview This PR adds the functionality and a suite of automated tests for **Manage Support**. ## Tests & Functionality Included - **Tree Structure**: Test the correctness of the project tree structure. - **Add Existing Environment**: Add an existing environment to a project node. - **Create Compiler from CFG File**: Validate creating a compiler from a CFG configuration. - **Delete Compiler**: Test removing a compiler from the project. - **Build/Execute Incremental**: Validate incremental builds and executions. - **Create Testsuite**: Test creating a new testsuite. - **Delete Testsuite**: Verify testsuite deletion functionality. - **Delete Project Environment**: Ensure environments can be properly deleted from a project. - **Build Single Project Environment**: Test building a single environment separately. - **Remove Environment from Testsuite**: Validate unlinking an environment from a testsuite. - **Create Environment from Source Files**: Test creating an environment directly from source files. - **Change Update Project Setting & Update project manually** ## Additional Notes: - Manage-only code is placed in the folder `src/manage` - `tests/internal/e2e/test` now include a `manage` folder, including files important for tests - Because PCT wanted to load a test script by saving the file and not by (only) pressing "Load Test Script into Environment", I had to change all tests that had a `tab.save()`. Because this would trigger the loading test script which led to errors. - For many tests I added additional log awaitings because otherwise CI was too quick. - Some async issues had to be resolved in the source code, as we ran into race condition problems on CI --------- Co-authored-by: John Paliotta <john.paliotta@vector.com> Co-authored-by: SalvatoreZagaria <salvatore.zagaria@vector.com> * Changed CHANGELOG date (#257) * E2e import coded test (#258) E2E for importing coded test files. Resolves issue #114 * Remove developer vars from environment in release mode (#260) * Add debug flag to get debug info from cli outputs * Autocompletion fix and unit test for uut_prototype_stubs (#261) * Merge adjustments for requirements * Deleted unused var * Reverted commented code for server state * Fixed vscodeignore format --------- Co-authored-by: SalvatoreZagaria <37420335+SalvatoreZagaria@users.noreply.github.com> Co-authored-by: Denis Moslavac <44575706+Den1552@users.noreply.github.com> Co-authored-by: John Paliotta <john.paliotta@vector.com> Co-authored-by: SalvatoreZagaria <salvatore.zagaria@vector.com> Co-authored-by: Patrick Bareiss <Patrick.Bareiss@vector.com>
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Overview
This PR adds the functionality and a suite of automated tests for Manage Support.
Tests & Functionality Included
Additional Notes:
src/managetests/internal/e2e/testnow include amanagefolder, including files important for teststab.save(). Because this would trigger the loading test script which led to errors.