fluxxus-nl / ATDD.TestScriptor.VSCodeExtension

VSCode extension for Acceptance Test-Driven Development test scriptor

MIT License

18 stars 3 forks source link

Use Case: Removing Scenario #39

Closed lvanvugt closed 3 years ago

lvanvugt commented 4 years ago

The following flowchart shows the use case:

Use Case (#39) - Removing Scenario

Exceptions

Settings

Make setting possible that user can disable the confirm on removing helper function with two flavors (see #28)
1. Do not show and do not remove (this scenario is not (yet) in flowchart)
2. Do not show, but do remove
And similar setting regarding disabling confirm on removing test function

Notes

The garbage can icon to remove a scenario has been replaced by a red '- Scenario' button

Test Scenarios

No.	Feature	Scenario
0056	Removing Scenario	Removing Scenario
0057	Removing Scenario	Removing Scenario step 2a
0058	Removing Scenario	Removing Scenario step 3a
0059	Removing Scenario	Removing Scenario step 2b
0060	Removing Scenario	Removing Scenario step 3b
0068	Removing Scenario	Removing Scenario with Initialize
0069	Removing Scenario	Removing Scenario with UI Handler
0070	Removing Scenario	Removing Scenario with UI Handler 2
0071	Removing Scenario	Removal Mode "No confirmation, but removal"
0072	Removing Scenario	Removal Mode "No confirmation & no removal"

in atdd-test-scenarios-and-results.xlsx

DavidFeldhoff commented 4 years ago

If the Scenario has handler functions these should be deleted (after a confirmation of the user) as well, I guess? I'll implement it like this at first.

lvanvugt commented 4 years ago

Agree.

lvanvugt commented 4 years ago

@DavidFeldhoff no commits related to this issue?

DavidFeldhoff commented 4 years ago

Sorry, haven't linked that appropriately. Will fix that later

DavidFeldhoff commented 4 years ago

fixed in commit https://github.com/fluxxus-nl/ATDD.TestScriptor.VSCodeExtension/commit/4bd3fbfc90b63246f5c66e7e2de9f89b1fe44088

DavidFeldhoff commented 4 years ago

..Forgot the removal of HandlerFunctions - I hope I can add that this evening

DavidFeldhoff commented 4 years ago

hm.. already implemented that as well in the commit above, but I'm using there the gotodefinition provider of my al-codeactions there, so I added that as dependency. @lvanvugt The stage is yours for testing :)

lvanvugt commented 4 years ago

@DavidFeldhoff tested and OK

lvanvugt commented 4 years ago

@DavidFeldhoff: reopened as the settings have not been added yet.

DavidFeldhoff commented 3 years ago

Hi Luc, feel free to test this one, it's ready and works as well with the settings.

lvanvugt commented 3 years ago

Tested and added some extra scenarios -> OK

lvanvugt commented 3 years ago

Reopened due to #80.

lvanvugt commented 3 years ago

Closed as #80 cannot be reproduced.

lvanvugt commented 3 years ago

86 and #88 fixed, so closed