Branch tests broken [#2784703]

Comment	File	Size	Author
#48	2784703-48.patch	1.4 KB	mondrake
#46	2784703-46.patch	8.05 KB	mondrake
#44	2784703-44.patch	8.02 KB	mondrake
#42	2784703-42.patch	5.96 KB	mondrake
#40	2784703-40.patch	4.92 KB	mondrake
#38	2784703-38.patch	4.87 KB	mondrake
#36	2784703-36.patch	4.87 KB	mondrake
#34	2784703-34.patch	1.11 KB	mondrake
#32	2784703-31.patch	4.06 KB	mondrake
#30	2784703-30.patch	51.45 KB	mondrake
#29	2784703-29.patch	51.45 KB	mondrake
#27	2784703-27.patch	52.28 KB	mondrake
#25	2784703-25.patch	52.27 KB	mondrake
#23	2784703-23.patch	52.02 KB	mondrake
#22	2784703-22.patch	52.26 KB	mondrake
#20	2784703-20.patch	51.72 KB	mondrake
#17	2784703-17.patch	2.8 KB	mondrake
#16	2784703-16.patch	2.53 KB	mondrake
#14	2784703-14.patch	1.53 KB	mondrake
#12	2784703-12.patch	1.96 KB	mondrake
#10	2784703-10.patch	723 bytes	mondrake
#8	2784703-8.patch	828 bytes	mondrake
#6	2784703-6.patch	500 bytes	mondrake
#4	2784703-4.patch	1.39 KB	mondrake
#2	2784703-2.patch	1.32 KB	mondrake

Comment #1

16 August 2016 at 08:14

mondrake created an issue. See original summary.

Log in or register to post comments

Comment #2

mondrake

🇮🇹

commented 16 August 2016 at 21:55

Priority:	Major	» Critical
Status:	Active	» Needs review

Status	File	Size
new	2784703-2.patch	1.32 KB

Locally tests pass, so let's do some debugging to see where this fails on CI

Log in or register to post comments

Comment #3

16 August 2016 at 21:59

Status:

Needs review

» Needs work

The last submitted patch, 2: 2784703-2.patch, failed testing.

Log in or register to post comments

Comment #4

mondrake

🇮🇹

commented 16 August 2016 at 22:04

Status:

Needs work

» Needs review

Status	File	Size
new	2784703-4.patch	1.39 KB

Log in or register to post comments

Comment #5

16 August 2016 at 22:08

Status:

Needs review

» Needs work

The last submitted patch, 4: 2784703-4.patch, failed testing.

Log in or register to post comments

Comment #6

mondrake

🇮🇹

commented 17 August 2016 at 06:25

Status:

Needs work

» Needs review

Status	File	Size
new	2784703-6.patch	500 bytes

Log in or register to post comments

Comment #7

17 August 2016 at 06:38

Status:

Needs review

» Needs work

The last submitted patch, 6: 2784703-6.patch, failed testing.

Log in or register to post comments

Comment #8

mondrake

🇮🇹

commented 17 August 2016 at 06:41

Status:

Needs work

» Needs review

Status	File	Size
new	2784703-8.patch	828 bytes

Log in or register to post comments

Comment #9

17 August 2016 at 06:47

Status:

Needs review

» Needs work

The last submitted patch, 8: 2784703-8.patch, failed testing.

Log in or register to post comments

Comment #10

mondrake

🇮🇹

commented 17 August 2016 at 06:51

Status:

Needs work

» Needs review

Status	File	Size
new	2784703-10.patch	723 bytes

4 files were hidden/shown/deleted

Status	File	Size
hidden	2784703-2.patch	1.32 KB
hidden	2784703-4.patch	1.39 KB
hidden	2784703-6.patch	500 bytes
hidden	2784703-8.patch	828 bytes

Log in or register to post comments

Comment #11

17 August 2016 at 06:54

Status:

Needs review

» Needs work

The last submitted patch, 10: 2784703-10.patch, failed testing.

Log in or register to post comments

Comment #12

mondrake

🇮🇹

commented 17 August 2016 at 06:58

Status:

Needs work

» Needs review

Status	File	Size
new	2784703-12.patch	1.96 KB

Log in or register to post comments

Comment #13

17 August 2016 at 07:01

Status:

Needs review

» Needs work

The last submitted patch, 12: 2784703-12.patch, failed testing.

Log in or register to post comments

Comment #14

mondrake

🇮🇹

commented 18 August 2016 at 17:37

Status:

Needs work

» Needs review

Status	File	Size
new	2784703-14.patch	1.53 KB

2 files were hidden/shown/deleted

Status	File	Size
hidden	2784703-10.patch	723 bytes
hidden	2784703-12.patch	1.96 KB

Log in or register to post comments

Comment #15

18 August 2016 at 17:56

Status:

Needs review

» Needs work

The last submitted patch, 14: 2784703-14.patch, failed testing.

Log in or register to post comments

Comment #16

mondrake

🇮🇹

commented 18 August 2016 at 18:08

Status:

Needs work

» Needs review

Status	File	Size
new	2784703-16.patch	2.53 KB

1 file was hidden/shown/deleted

Status	File	Size
hidden	2784703-14.patch	1.53 KB

Log in or register to post comments

Comment #17

mondrake

🇮🇹

commented 18 August 2016 at 18:09

Status	File	Size
new	2784703-17.patch	2.8 KB

Log in or register to post comments

Comment #18

18 August 2016 at 21:24

The last submitted patch, 16: 2784703-16.patch, failed testing.

Log in or register to post comments

Comment #19

18 August 2016 at 21:24

Status:

Needs review

» Needs work

The last submitted patch, 17: 2784703-17.patch, failed testing.

Log in or register to post comments

Comment #20

mondrake

🇮🇹

commented 19 August 2016 at 07:30

Status:

Needs work

» Needs review

Status	File	Size
new	2784703-20.patch	51.72 KB

2 files were hidden/shown/deleted

Status	File	Size
hidden	2784703-16.patch	2.53 KB
hidden	2784703-17.patch	2.8 KB

Log in or register to post comments

Comment #21

19 August 2016 at 07:41

Status:

Needs review

» Needs work

The last submitted patch, 20: 2784703-20.patch, failed testing.

Log in or register to post comments

Comment #22

mondrake

🇮🇹

commented 19 August 2016 at 07:56

Status:

Needs work

» Needs review

Status	File	Size
new	2784703-22.patch	52.26 KB

1 file was hidden/shown/deleted

Status	File	Size
hidden	2784703-20.patch	51.72 KB

Log in or register to post comments

Comment #23

mondrake

🇮🇹

commented 19 August 2016 at 08:09

Status	File	Size
new	2784703-23.patch	52.02 KB

Log in or register to post comments

Comment #24

19 August 2016 at 08:12

Status:

Needs review

» Needs work

The last submitted patch, 23: 2784703-23.patch, failed testing.

Log in or register to post comments

Comment #25

mondrake

🇮🇹

commented 21 August 2016 at 21:11

Status:

Needs work

» Needs review

Status	File	Size
new	2784703-25.patch	52.27 KB

2 files were hidden/shown/deleted

Status	File	Size
hidden	2784703-22.patch	52.26 KB
hidden	2784703-23.patch	52.02 KB

Log in or register to post comments

Comment #26

21 August 2016 at 21:14

Status:

Needs review

» Needs work

The last submitted patch, 25: 2784703-25.patch, failed testing.

Log in or register to post comments

Comment #27

mondrake

🇮🇹

commented 21 August 2016 at 21:17

Status:

Needs work

» Needs review

Status	File	Size
new	2784703-27.patch	52.28 KB

1 file was hidden/shown/deleted

Status	File	Size
hidden	2784703-25.patch	52.27 KB

Log in or register to post comments

Comment #28

21 August 2016 at 21:21

Status:

Needs review

» Needs work

The last submitted patch, 27: 2784703-27.patch, failed testing.

Log in or register to post comments

Comment #29

mondrake

🇮🇹

commented 21 August 2016 at 21:26

Status:

Needs work

» Needs review

Status	File	Size
new	2784703-29.patch	51.45 KB

1 file was hidden/shown/deleted

Status	File	Size
hidden	2784703-27.patch	52.28 KB

Log in or register to post comments

Comment #30

mondrake

🇮🇹

commented 21 August 2016 at 21:53

Status	File	Size
new	2784703-30.patch	51.45 KB

Log in or register to post comments

Comment #31

21 August 2016 at 22:11

The last submitted patch, 29: 2784703-29.patch, failed testing.

Log in or register to post comments

Comment #32

mondrake

🇮🇹

commented 21 August 2016 at 22:11

Status	File	Size
new	2784703-31.patch	4.06 KB

2 files were hidden/shown/deleted

Status	File	Size
hidden	2784703-29.patch	51.45 KB
hidden	2784703-30.patch	51.45 KB

Log in or register to post comments

Comment #33

21 August 2016 at 22:16

Status:

Needs review

» Needs work

The last submitted patch, 32: 2784703-31.patch, failed testing.

Log in or register to post comments

Comment #34

mondrake

🇮🇹

commented 21 August 2016 at 22:22

Status:

Needs work

» Needs review

Status	File	Size
new	2784703-34.patch	1.11 KB

Log in or register to post comments

Comment #35

21 August 2016 at 22:26

Status:

Needs review

» Needs work

The last submitted patch, 34: 2784703-34.patch, failed testing.

Log in or register to post comments

Comment #36

mondrake

🇮🇹

commented 21 August 2016 at 22:40

Status:

Needs work

» Needs review

Status	File	Size
new	2784703-36.patch	4.87 KB

2 files were hidden/shown/deleted

Status	File	Size
hidden	2784703-31.patch	4.06 KB
hidden	2784703-34.patch	1.11 KB

Log in or register to post comments

Comment #37

21 August 2016 at 22:43

Status:

Needs review

» Needs work

The last submitted patch, 36: 2784703-36.patch, failed testing.

Log in or register to post comments

Comment #38

mondrake

🇮🇹

commented 21 August 2016 at 22:45

Status:

Needs work

» Needs review

Status	File	Size
new	2784703-38.patch	4.87 KB

Log in or register to post comments

Comment #39

21 August 2016 at 22:49

Status:

Needs review

» Needs work

The last submitted patch, 38: 2784703-38.patch, failed testing.

Log in or register to post comments

Comment #40

mondrake

🇮🇹

commented 22 August 2016 at 05:31

Status:

Needs work

» Needs review

Status	File	Size
new	2784703-40.patch	4.92 KB

2 files were hidden/shown/deleted

Status	File	Size
hidden	2784703-36.patch	4.87 KB
hidden	2784703-38.patch	4.87 KB

Log in or register to post comments

Comment #41

22 August 2016 at 05:39

Status:

Needs review

» Needs work

The last submitted patch, 40: 2784703-40.patch, failed testing.

Log in or register to post comments

Comment #42

mondrake

🇮🇹

commented 22 August 2016 at 07:29

Status:

Needs work

» Needs review

Status	File	Size
new	2784703-42.patch	5.96 KB

1 file was hidden/shown/deleted

Status	File	Size
hidden	2784703-40.patch	4.92 KB

Log in or register to post comments

Comment #43

22 August 2016 at 07:35

Status:

Needs review

» Needs work

The last submitted patch, 42: 2784703-42.patch, failed testing.

Log in or register to post comments

Comment #44

mondrake

🇮🇹

commented 22 August 2016 at 08:29

Status:

Needs work

» Needs review

Status	File	Size
new	2784703-44.patch	8.02 KB

1 file was hidden/shown/deleted

Status	File	Size
hidden	2784703-42.patch	5.96 KB

Log in or register to post comments

Comment #45

22 August 2016 at 08:46

Status:

Needs review

» Needs work

The last submitted patch, 44: 2784703-44.patch, failed testing.

Log in or register to post comments

Comment #46

mondrake

🇮🇹

commented 22 August 2016 at 09:12

Status:

Needs work

» Needs review

Status	File	Size
new	2784703-46.patch	8.05 KB

1 file was hidden/shown/deleted

Status	File	Size
hidden	2784703-44.patch	8.02 KB

Log in or register to post comments

Comment #47

22 August 2016 at 10:34

Status:

Needs review

» Needs work

The last submitted patch, 46: 2784703-46.patch, failed testing.

Log in or register to post comments

Comment #48

mondrake

🇮🇹

commented 22 August 2016 at 11:43

Status:

Needs work

» Needs review

Status	File	Size
new	2784703-48.patch	1.4 KB

1 file was hidden/shown/deleted

Status	File	Size
hidden	2784703-46.patch	8.05 KB

In the end, it looks like it was sufficient to add a test_dependency to 'imagemagick' in the .info.yml file.

See http://cgit.drupalcode.org/image_effects/commit/?id=c6ee646.

Some of the work in #46 could be useful if converting the test suite to KernelTests.

Patch here would just fix a couple of minor things.

Log in or register to post comments

Comment #49

22 August 2016 at 11:48

mondrake committed efa2101 on 8.x-1.x

Issue #2784703 by mondrake: Branch tests broken

Log in or register to post comments

Comment #50

mondrake

🇮🇹

commented 22 August 2016 at 11:48

Status:

Needs review

» Fixed

Log in or register to post comments

Comment #51

5 September 2016 at 11:54

Status:

Fixed

» Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

Log in or register to post comments

Branch tests broken

Comments