Comments

claudiu.cristea created an issue. See original summary.

Mixologic’s picture

Category: Bug report » Feature request
Priority: Critical » Major

Unless Im mistaken, this is not how the current PIFT/PIFR bots behave, and therefore not a blocker to shutting them off.

This is something drupalCI should do, and we'll add it, but not before we shut off the old bots.

claudiu.cristea’s picture

If you shut them off before fixing this, it will make possible to commit code with failed tests in core. Anyway many of reviewers/committers ar looking only to DrupalCI results because the other one gives random failures. So, I think, these issues are both critical and blockers.

Mixologic’s picture

We definitely need to fix the fatal error that is related to this issue, but Strict Standards warnings are not indicative of a failed test, nor did they fail before on the old testbots.

claudiu.cristea’s picture

but Strict Standards warnings are not indicative of a failed test

This I don't know. IMO such patches should be reworked to fix the warning. But, who knows...

fabianx’s picture

Category: Feature request » Task
Priority: Major » Critical

Yes, they failed on the old bots - hence the other core critical.

That is why this is critical and hence blocking turning off of the old bots.

Mixologic’s picture

Category: Task » Feature request
Priority: Critical » Major

I just went and looked deeper,

The old testbots did *not* run PHP Strict Standards at all, so this would not be a regression from current functionality.

Whether or not tests *should* fail on Strict Standards warnings is another matter entirely. It may be that we want to emit debug messages, or something else to indicate a warning, but not necessarily fail a whole test for it.

fabianx’s picture

Title: Tests must fail on "PHP Strict Standards" warnings » Tests should maybe fail on "PHP Strict Standards" warnings
Priority: Major » Normal

Sorry, I mixed things up with #2580293: Patch having test with "PHP Fatal error" is marked as PASSED, which is the critical part.

Mixologic’s picture

Title: Tests should maybe fail on "PHP Strict Standards" warnings » [Policy] Should tests fail based on strict standards warnings
Component: Code » Policy
Related issues: +#2779167: Enable PHP Strict Standards and provide results.

This issue will be for discussing failures of strict standards. Added a related issue to at least provide that information, regardless of whether or not it fails the tests.

Mixologic’s picture