Problem

XssTest test class in views_ui/src/Tests lacks the name of the test in class annotation comments, therefore this test is never discovered and is never run by the testing system, not even Drupal QA testbot.

Proposed resolution

Ensure the MissingSummaryLineException is not caught during test discovery.

Remaining tasks

None.

User interface changes

None.

API changes

None.

CommentFileSizeAuthor
#9 2473907.9.patch38.08 KBalexpott
#9 5-9-interdiff.txt36.08 KBalexpott
#5 2473907.5.patch3.3 KBalexpott
#5 1-5-interdiff.txt2.92 KBalexpott
#5 2473907.5.test-only.patch879 bytesalexpott
#1 views_ui-xsstest-2473907.patch388 bytesmaijs

Comments

maijs’s picture

maijs commented
StatusFileSize
new views_ui-xsstest-2473907.patch388 bytes

Changing the attribution.

maijs’s picture

maijs commented
Status: Active » Needs review
maijs’s picture

maijs commented
Issue summary: View changes
jcnventura’s picture

jcnventura
Primary language English
commented
Status: Needs review » Reviewed & tested by the community

Good catch.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Title: XssTest test in views_ui is not discovered » Tests not being run by testbot due to missing summary line
Issue summary: View changes
Priority: Normal » Major
Status: Reviewed & tested by the community » Needs review
Related issues: +#2422019: Don't use reflection for parsing test annotations
StatusFileSize
new 2473907.5.test-only.patch879 bytes
new 1-5-interdiff.txt2.92 KB
new 2473907.5.patch3.3 KB

There are more tests affected by this. Bug introduced by #2422019: Don't use reflection for parsing test annotations

Fingers-crossed all the tests pass... alexpott--

The last submitted patch, 5: 2473907.5.test-only.patch, failed testing.

dawehner’s picture

dawehner
Primary language German
commented
Status: Needs review » Reviewed & tested by the community

Wow

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 5: 2473907.5.patch, failed testing.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Needs work » Needs review
StatusFileSize
new 5-9-interdiff.txt36.08 KB
new 2473907.9.patch38.08 KB

More hardening and fixing the failures.

dawehner’s picture

dawehner
Primary language German
commented
Status: Needs review » Reviewed & tested by the community

Perfect!

neclimdul’s picture

neclimdul
He/Him
commented

Before: PASSED: [[SimpleTest]]: [PHP 5.4 MySQL] 94,345 pass(es).
After: PASSED: [[SimpleTest]]: [PHP 5.4 MySQL] 94,436 pass(es).

No new tests. Good job guys!

catch’s picture

catch
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Ouch, nice find.

Committed/pushed to 8.0.x, thanks!

  • catch committed 5eb13a9 on 8.0.x 
    Issue #2473907 by alexpott, maijs: Tests not being run by testbot due to...

Status: Fixed » Closed (fixed)

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