[MGNLTEST-34] Clean-up packages for int.- and ui-tests on CE, dx-core Created: 15/Jan/20 Updated: 16/Mar/21 Resolved: 16/Mar/21 |
|
| Status: | Closed |
| Project: | Magnolia Test Framework |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Task | Priority: | Neutral |
| Reporter: | Christoph Meier | Assignee: | Unassigned |
| Resolution: | Fixed | Votes: | 1 |
| Labels: | QA&Testing | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||
| Template: |
|
||||||||||||||||||||
| Acceptance criteria: |
Empty
|
||||||||||||||||||||
| Task DoR: |
Empty
|
||||||||||||||||||||
| Epic Link: | core-TF-features-bugs-improvements | ||||||||||||||||||||
| Description |
|
This ticket has been created to "save" some valuable comments given on a PR. The main idea is to make it easy to distinguish:
I add the contents of the comments to the comments section. Otherwise this description get too big. But here are the links to the comments: (Not sure whether they will survive)
Note that this is highly related to / with
|
| Comments |
| Comment by Christoph Meier [ 15/Jan/20 ] |
|
This comment is a copy of a comment from a PR on bitbucket.
TL;DR Preliminary suggestion: Thanks for explaining this thoroughly. Bad names may not seem important here if you have been navigating the packages and modules for a long time - you have a clear mental mapping of what is what and know how to move around those. It may subjectively feel like a non-issue since the mental mapping works fine for you. But for me (and possibly anyone new to this) it's a source of confusion and it requires digging, asking and creating the mental mapping afterwards for one self. Which fades if the packages are not sth one interacts with on daily basis. We want to avoid everyone asking the same questions like:
The naming should be refined now and made clear, ideally by those, who have a good understanding of what makes them distinct. Source: https://git.magnolia-cms.com/projects/PLATFORM/repos/ce/pull-requests/206/overview?commentId=44564 (from sdemocko) |