Bug #16817
Emergency shutdown automated tests on Buster often fail to notice that memory wipe was completed
0%
Description
I see lots of “FindFailed: can not find MemoryWipeCompleted.png” while the video shows that the expected message was displayed (so I have a doubt wrt. the image update in commit:56f58e34d2bca109dc2f7d2467d2bc1364732355).
Subtasks
History
#1 Updated by intrigeri 2019-06-17 15:14:26
(I won’t file tickets for every Buster test suite problem you surely know about already, but I need to at least track those that block the first beta.)
#2 Updated by intrigeri 2019-06-17 15:14:34
- blocks Feature #16209: Core work: Foundations Team added
#3 Updated by intrigeri 2019-06-17 15:29:52
- Status changed from Confirmed to In Progress
- Assignee set to intrigeri
And indeed, https://jenkins.tails.boum.org/view/RM/job/test_Tails_ISO_feature-buster/212/artifact/build-artifacts/sikuli_candidates/MemoryWipeCompleted.png looks more like what we have on our 3.x branches. I’m starting to suspect that commit:56f58e34d2bca109dc2f7d2467d2bc1364732355 was only needed due to Bug #16743 and now breaks stuff.
#4 Updated by intrigeri 2019-06-17 15:33:14
- Feature Branch set to test/16817-emergency-shutdown
#5 Updated by intrigeri 2019-06-17 16:12:17
features/emergency_shutdown.feature
passes for me locally on this branch. Let’s see what Jenkins thinks and especially, check that sikuli does not need to do fuzzy matching to find the picture.
#6 Updated by intrigeri 2019-06-18 06:52:20
- Status changed from In Progress to Needs Validation
- Assignee deleted (
intrigeri)
intrigeri wrote:
> features/emergency_shutdown.feature
passes for me locally on this branch. Let’s see what Jenkins thinks
That feature passed twice on https://jenkins.tails.boum.org/view/Tails_ISO/job/test_Tails_ISO_test-16817-emergency-shutdown/.
> and especially, check that sikuli does not need to do fuzzy matching to find the picture.
No MemoryWipeCompleted.png
in https://jenkins.tails.boum.org/view/Tails_ISO/job/test_Tails_ISO_test-16817-emergency-shutdown/1/artifact/build-artifacts/sikuli_candidates/ nor https://jenkins.tails.boum.org/view/Tails_ISO/job/test_Tails_ISO_test-16817-emergency-shutdown/2/artifact/build-artifacts/sikuli_candidates/.
So I think my hypothesis above this correct and this branch is the right thing to do.
#7 Updated by hefee 2019-06-18 10:02:50
- Assignee set to hefee
#8 Updated by hefee 2019-06-18 10:17:41
- Status changed from Needs Validation to In Progress
- Assignee changed from hefee to intrigeri
Go ahaed, this sounds logic for me.
#9 Updated by intrigeri 2019-06-18 10:26:10
- Status changed from In Progress to Fix committed
- % Done changed from 0 to 100
Applied in changeset commit:tails|9858dc2e1e8b89162c6f360305d2202be61d6722.
#10 Updated by intrigeri 2019-06-18 10:26:53
- Status changed from Fix committed to Resolved
- % Done changed from 100 to 0