Fix PHPStan snapshot and UI screenshots

Author: tzi

phpstan-baseline.neon

@@ -4658,11 +4658,6 @@ parameters:
 			count: 1
 			path: plugins/ScheduledReports/ScheduledReports.php
 
-		-
-			message: "#^Negated boolean expression is always true\\.$#"
-			count: 1
-			path: plugins/SegmentEditor/API.php
-
 		-
 			message: "#^Parameter \\#2 \\$idSites of class Piwik\\\\Segment constructor expects array, int\\|null given\\.$#"
 			count: 1

plugins/SegmentEditor/tests/UI/SegmentSelectorEditor_spec.js

@@ -64,30 +64,31 @@ describe("SegmentSelectorEditorTest", function () {
     });
 
     it("should star all segments", async function() {
-        await page.click('.segmentList li:nth-child(2) .starSegment');
-        await page.click('.segmentList li:nth-child(3) .starSegment');
-        await page.click('.segmentList li:nth-child(4) .starSegment');
-        const firstSegment = await page.$('.segmentList li:nth-child(2)');
-        expect(firstSegment.className).to.contain('segmentStarred');
-        expect(firstSegment.find('.starSegment').attr('data-state')).to.equal('');
+        const firstSegment = await page.$('.segmentList li:nth-of-type(2)');
+        const firstSegmentStar = await page.$('.segmentList li:nth-of-type(2) .starSegment');
+        await firstSegmentStar.click('.segmentList li:nth-of-type(2) .starSegment');
+        await page.click('.segmentList li:nth-of-type(3) .starSegment');
+        await page.click('.segmentList li:nth-of-type(4) .starSegment');
+        expect(firstSegment.attr('class')).to.match(/segmentStarred/);
+        expect(firstSegmentStar.attr('data-state')).to.equal('');
         expect(await page.screenshotSelector(selectorsToCapture)).to.matchImage('1_selector_starred');
     });
 
     it("should unstar first segment", async function() {
-        await page.click('.segmentList li:nth-child(2) .starSegment');
-        const firstSegment = await page.$('.segmentList li:nth-child(2)');
-        expect(firstSegment.className).to.not.contain('segmentStarred');
-        expect(firstSegment.find('.starSegment').attr('data-state')).to.equal('');
+        const firstSegment = await page.$('.segmentList li:nth-of-type(2)');
+        const firstSegmentStar = await page.$('.segmentList li:nth-of-type(2) .starSegment');
+        await firstSegmentStar.click('.segmentList li:nth-of-type(2) .starSegment');
+        expect(firstSegment.attr('class')).to.not.match(/segmentStarred/);
+        expect(firstSegmentStar.attr('data-state')).to.equal('');
         expect(await page.screenshotSelector(selectorsToCapture)).to.matchImage('1b_selector_unstarred');
     });
 
     it("should have disabled star for anonymous users", async function() {
         await switchToAnonymousUser();
         await page.goto(url);
         await page.click('.segmentationContainer .title');
-        const firstSegment = await page.$('.segmentList li:nth-child(2)');
-        expect(firstSegment.className).to.contain('segmentStarred');
-        expect(firstSegment.find('.starSegment').attr('data-state')).to.equal('');
+        const firstSegmentStar = await page.$('.segmentList li:nth-of-type(2) .starSegment');
+        expect(firstSegmentStar.attr('data-state')).to.equal('disabled');
     });
 
     it("should open segment editor when edit link clicked for existing segment", async function() {