[AAE-3209]Fix e2e about attach file from local

[cristinaj]

Please check if the PR fulfills these requirements

• The commit message follows our guidelines
• Tests for the changes have been added (for bug fixes / features)
• Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (check one with "x")

• Bugfix
• Feature
• Code style update (formatting, local variables)
• Refactoring (no functional changes, no api changes)
• Build related changes
• Documentation
• Other... Please describe:

What is the current behaviour? (You can also link to an open issue here)

What is the new behaviour?

Does this PR introduce a breaking change? (check one with "x")

• Yes
• No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...

Other information:

[codecov-commenter]

Codecov Report

Merging #5944 into develop will increase coverage by 2.62%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop    #5944      +/-   ##
===========================================
+ Coverage    75.53%   78.16%   +2.62%     
===========================================
  Files          701      142     -559     
  Lines        14847     3920   -10927     
  Branches      2911      780    -2131     
===========================================
- Hits         11215     3064    -8151     
+ Misses        2775      603    -2172     
+ Partials       857      253     -604     

Impacted Files	Coverage Δ
lib/core/mock/identity-user.service.mock.ts
...d-view/components/card-view/card-view.component.ts
...ns/diagram-icon-alfresco-publish-task.component.ts
...ces-cloud/src/lib/pipes/process-name-cloud.pipe.ts
...rocess/start-process/start-process-cloud.module.ts
...s-services/src/lib/app-list/apps-list.component.ts
lib/core/utils/moment-date-formats.model.ts
lib/core/directives/highlight.directive.ts
...ghts/src/lib/mock/diagram/diagram-boundary.mock.ts
...lib/process/directives/process-directive.module.ts
... and 833 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1dec2ba...e3371cd. Read the comment docs.

[eromano]

why do we have this 0 at the beginning?

[cristinaj]

This test uses hruser and there are a lot of folders created[by other people, other tests, i don't know]. attachFileFromContentNode() method is checking if the content[folder in this case] is displayed, without taking in consideration to click on 'Show more' in case is displayed. So if the folder is not displayed in the first ones, the test will fail. The easiest way for me was to fix it like this. Other options would be to use a new created user instead of hruser or modify the attachFileFromContentNode() method to take in consideration cases where multiple folders are created.

[eromano]

I think is better to use a clean user

[cristinaj]

It seems i can't use clean user without changing the process definition. The user task has hruser as assignee. We can raise separate ticket for this.