[MGNLUI-4658] Reenable two bypassed unit tests Created: 27/Sep/18 Updated: 28/Sep/18 Resolved: 28/Sep/18 |
|
| Status: | Closed |
| Project: | Magnolia UI |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Task | Priority: | Neutral |
| Reporter: | Maxime Michel | Assignee: | Aleksandr Pchelintcev |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Template: |
|
| Acceptance criteria: |
Empty
|
| Task DoR: |
Empty
|
| Description |
|
However, this was at the expense of two tests, which started to fail and were ignored because they couldn't be investigated: https://git.magnolia-cms.com/projects/PLATFORM/repos/ui/commits/85d5b488b425676cc83bc93acde24204f33a10d0 The problem has been pinpointed to lost weak references. The fix starts by making UI and VaadinSession class fields. |