Project

General

Profile

Feature #3838

Examine JUnit test cases for migration needs prior to launching them

Added by Alexander Diewald 12 months ago. Updated 5 months ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
-
Target version:
Start date:
04/09/2020
Due date:
04/09/2020
% Done:

100%

Estimated time:
1.00 h

Description

Current situation

The migration service is not applied to models that are programmatically loaded. Instead, it is launched when projects are discovered in the runtime workspace. This applies to existing models and imported ones that are copied to this workspace.

Proposed resolution

Add an extra utility method for model loading from test cases. It shall check whether migration is needed and fail the test if a model update is needed. These updates can only be done manually, so it is better to fail early than failing test cases when a model migrator is removed.


Related issues

Follows Feature #3993: Import capability of AF3 Projects by CLI switchesClosed04/08/2020

History

#1 Updated by Alexander Diewald 6 months ago

  • Target version changed from AF3 2.17 (Feature Freeze) to Backlog

#2 Updated by Alexander Diewald 6 months ago

  • Status changed from New to Resolved
  • Assignee changed from Alexander Diewald to Tiziano Munaro
  • % Done changed from 0 to 100

This one overlaps with the refactoring done in #3993.

If you have too much to do, please reassign to me :)

For testing, revert the test model update of StateAutomataConstraintsCheckerTestData and execute the lauch cfg "AllTests". One of it should fail, asking for manual model migration.

MRs:
AF3: https://git.fortiss.org/af3/af3/-/merge_requests/330
Kernel: https://git.fortiss.org/af3/kernel/-/merge_requests/112

#3 Updated by Alexander Diewald 6 months ago

  • Due date set to 04/09/2020
  • Start date changed from 10/08/2019 to 04/09/2020
  • Follows Feature #3993: Import capability of AF3 Projects by CLI switches added

#4 Updated by Tiziano Munaro 5 months ago

  • Assignee changed from Tiziano Munaro to Alexander Diewald

GREEN! Can be merged!

#5 Updated by Alexander Diewald 5 months ago

  • Assignee changed from Alexander Diewald to Johannes Eder

Merged. Please close :)

#6 Updated by Johannes Eder 5 months ago

  • Status changed from Resolved to Closed
  • Assignee deleted (Johannes Eder)

Also available in: Atom PDF