Project

General

Profile

Actions

action #57266

closed

Forbid test suite name changes if used in job groups

Added by pcervinka over 4 years ago. Updated over 4 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Feature requests
Target version:
Start date:
2019-09-24
Due date:
% Done:

0%

Estimated time:

Description

When is test suite renamed or deleted, changes are not propagated to job group configuration in YAML.
It is behavior change when comparing to old style of job group definition.

Reproduction scenario:

  1. Rename or delete test suite
  2. Check job group definition in YAML, test suite is still present or has old name

Note: Sorry, if I made duplicate case(didn't find).

Actions #1

Updated by pcervinka over 4 years ago

  • Description updated (diff)
Actions #2

Updated by coolo over 4 years ago

I assume this is about test suites that are only used in one job group? We want to eliminate those anyway

Actions #3

Updated by pcervinka over 4 years ago

Could be related, but it is behavior change and unaware(used to old things) user can be surprised.
For example I was adding new test suite to YAML schedule and editor complained about other not existing test suite.
It was easy to fix(just removed it), but it means that users don't expect it (to manually go over all job group definitions and delete/rename test suite).

Actions #4

Updated by coolo over 4 years ago

The yaml config file is user input and we won't touch it. But what we can do is forbid deleting/renaming test suites if they are used.

Actions #5

Updated by coolo over 4 years ago

  • Subject changed from [YAML][tools] Test suite changes are not propagated to YAML to Forbid test suite changes if used in job groups
  • Category set to Feature requests
  • Target version set to Ready
  • Difficulty set to medium
Actions #6

Updated by livdywan over 4 years ago

coolo wrote:

The yaml config file is user input and we won't touch it. But what we can do is forbid deleting/renaming test suites if they are used.

We can point out the job group(s) the respective test suite is being used in. And as a side effect users will be more aware of where a test suite is being used rather than silently changing all affected job groups.

Actions #7

Updated by coolo over 4 years ago

  • Subject changed from Forbid test suite changes if used in job groups to Forbid test suite name changes if used in job groups

We're talking about the name of the test suite only - I clarified the title

Actions #8

Updated by pcervinka over 4 years ago

coolo wrote:

The yaml config file is user input and we won't touch it. But what we can do is forbid deleting/renaming test suites if they are used.

If I understand well, it means manually remove test suite from all YAML job groups, rename and add back again with new name.
Quite difference in action when you compare to non-yaml solution(almost one click action).

Actions #9

Updated by coolo over 4 years ago

As I mentioned: the trend is to eliminate the need for frequent test suite changes

Actions #10

Updated by pvorel over 4 years ago

coolo wrote:

As I mentioned: the trend is to eliminate the need for frequent test suite changes

Despite the trend, sometimes changes are needed (sometimes even complex changes).
I should have checked the code & doc for this feature better before. If I had known this feature, I wouldn't have voluntarily changed it to yaml.

Is this incompatibility change documented somewhere?

Actions #12

Updated by livdywan over 4 years ago

pvorel wrote:

Is this incompatibility change documented somewhere?

This actually needs to be documented explicitly. The Job group editor section only mentions configuring a group in YAML, but it doesn't mention how this affects test suites. I would indeed suggest to cover this in this ticket as well.

Actions #13

Updated by livdywan over 4 years ago

  • Status changed from New to In Progress
  • Assignee set to livdywan
Actions #14

Updated by livdywan over 4 years ago

Actions #15

Updated by livdywan over 4 years ago

  • Status changed from In Progress to Feedback
Actions #16

Updated by livdywan over 4 years ago

  • Status changed from Feedback to Resolved
  • Target version changed from Ready to Current Sprint
Actions #17

Updated by mkittler over 4 years ago

  • Target version changed from Current Sprint to Done
Actions

Also available in: Atom PDF