Use Case: Adding Scenario

[lvanvugt]

The following flowchart shows the use case:

Exceptions

• Non-alphabetical or non-numerical characters should not be ported to test function name
• Scenario description should not have unlimited length as AL function names have a max length of 120 characters
  • I suggest to build logic that checks allowed length in Scenario description based on text with "to be removed" non-alphabetical or non-numerical characters

Settings

• Make setting possible that user can set max length to Scenario description himself

Update 20201005

In the previous version of Add Scenario the Initialize helper function was not added. This has been updated on the flowchart. The basic structure of the Initialize helper function should be:

local procedure Initialize()
var
    LibraryTestInitialize: Codeunit "Library - Test Initialize";
begin
    LibraryTestInitialize.OnTestInitialize(Codeunit::<name of test codeunit>);

    if IsInitialized then
        exit;

    LibraryTestInitialize.OnBeforeTestSuiteInitialize(Codeunit::<name of test codeunit>);

    IsInitialized := true;
    Commit();

    LibraryTestInitialize.OnAfterTestSuiteInitialize(Codeunit::<name of test codeunit>);
end;

The isInitialize variable is a global Boolean.

[lvanvugt]

@DavidFeldhoff no commits related to this issue?

[DavidFeldhoff]

Fixed in
4bd3fbf#diff-f267418653192133529f152c33dd6450R21
I'll assign it to you for testing Luc, as my tasks are finished so far. If you find some bugs: Keep them :)
No, if you find them, assign them to me, but I think the major part here is done, so it's better assigned to you, I think

[lvanvugt]

@DavidFeldhoff as mentioned on #46 this scenario needs to be extended as the Initialize helper functions needs to be added if it does not exist yet in the test codeunit. I will update the initial description of this post above (including the flowchart).

[DavidFeldhoff]

@martonsagi

• Use setting to limit scenario description in UI

[DavidFeldhoff]

Short note: This one is nearly ready for testing. After #44 is solved, it should work just fine.

[lvanvugt]

Tested and OK.