New idea for remote CC
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Anfängerfreundlichkeit
- 25/100
- Issue-Typ
- Feature
- Klarheit
- Muss geklärt werden
- Aktivitätsstatus
- Veraltet
- Tech-Stack
- php
- Bereich
- testing-qa
Rechercherichtung
Beginne mit der Durchsicht von Issue #85 und des bestehenden C3-Remote-Coverage-Ablaufs, einschließlich der Art und Weise, wie die Selenium-, REST- und Php-Browsermodule derzeit Anfragen identifizieren. Verfolge anschließend den Pfad des PHP CodeCoverage-Reports und der Serialisierung und lege vor der Implementierung gemeinsam mit den Maintainer:innen das Verhalten für Verbindung, Streaming, Bereinigung und parallele Tests fest.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
After reviewing #85 (good work keep it going!) I thought of a different approach to code coverage that might work cleanly as well.
The idea:
- Before a test starts, connect to C3, this is a "long lived" connection. C3 creates a file temp file with a fixed name and watches it for data, this data is streamed back to the coverage module.
- During the test each request to the server is profiled and the serialized result (php-codecoverage library supports serialization) is atomically written to the local temp file.
- After the test ends we disconnect the stream from 1 and this prompts the remote coverage file to be removed
This could simplify things:
- We no longer need to mark each request with a cookie, which needs to be set in the browser module (Selenium, REST, or Php)
- We no longer have edge cases where fetch requests might not send cookies for example
- We cannot support parallel tests since we won't be able to know to which test the coverage belongs
Streaming coverage
We can stream coverage for each request like this using the PHP report class (copied below since its short)
final class PHP
{
public function process(CodeCoverage $coverage, ?string $target = null): string
{
$coverage->clearCache();
$buffer = "<?php
return \unserialize(<<<'END_OF_COVERAGE_SERIALIZATION'" . PHP_EOL . serialize($coverage) . PHP_EOL . 'END_OF_COVERAGE_SERIALIZATION' . PHP_EOL . ');';
if ($target !== null) {
if (!strpos($target, '://') !== false) {
Filesystem::createDirectory(dirname($target));
}
if (@file_put_contents($target, $buffer) === false) {
throw new WriteOperationFailedException($target);
}
}
return $buffer;
}
}
- Capture it like normal
- Use the
SebastianBergmann\CodeCoverage\Report\PHPreport to process it to a string - Stream the result to the calling code
- The calling code splits it using the end marker.
- Each part is deserialized to an of
Coverage, which can be merged with the base coverage object via $baseCoverage->append($unserializedCoverage)`
Optionally we could do some automatic path detection which in most cases should actually be trivial.
Thoughts?
- Vorherrschende Sprache
- PHP
- Sterne
- 73
- Forks
- 46
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Entwicklungsumgebung
Dieses Projekt bietet weder Dev-Container noch Dockerfile noch Beitragsleitfaden – die Einrichtung liegt bei Ihnen. Beginnen Sie mit der README; die allgemeinen Schritte stehen in unserem Leitfaden für den ersten Beitrag.
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus Codeception/c3
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 25/100
Codeception/c3#76 · 1 Kommentar · 1 Reaktion ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 35/100
Codeception/c3#58 ·
-
Could not write error to log fileEvtl. vergeben @marcovtwout hat das vor 1972 Tagen übernommen. Offen
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 45/100
Codeception/c3#41 ·
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 25/100
Codeception/c3#40 · 5 Kommentare ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 45/100
Codeception/c3#37 · 3 Kommentare · 4 Reaktionen ·
Ähnliche Issues
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 84/100
WordPress/hosting-handbook#506 ·
Maintainer antworten meist innerhalb von 2 Tagen
-
Installing a plugin with composer_packages installs dev dependencies and breaks the autoloader (Error 500)Evtl. vergeben @Boy132 hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100
Maintainer antworten meist innerhalb von 1 Tag
-
nginx: static-file locations drop the security headers, and the dotfile deny is matched after themOffenbug Low Security
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 78/100
backdrop-contrib/commerce#60 ·