Project Risks
diff --git a/Detectors/index.html b/Detectors/index.html index 9a5a1bb..6b4e161 100644 --- a/Detectors/index.html +++ b/Detectors/index.html @@ -208,7 +208,7 @@Legend
High: 90% to 100%- Generated by php-code-coverage 10.1.10 using PHP 8.2.14 and PHPUnit 10.5.5 at Sun Dec 31 21:41:09 UTC 2023. + Generated by php-code-coverage 10.1.10 using PHP 8.2.14 and PHPUnit 10.5.5 at Sun Dec 31 21:48:12 UTC 2023.
diff --git a/FacelessLogger.php.html b/FacelessLogger.php.html index eab7fb8..9ddc4d2 100644 --- a/FacelessLogger.php.html +++ b/FacelessLogger.php.html @@ -42,193 +42,193 @@
-
-
-
-
-
-
-
-
Legend
Covered by small (and larger) testsCovered by medium (and large) testsCovered by large tests (and tests of unknown size)Not coveredNot coverable
- Generated by php-code-coverage 10.1.10 using PHP 8.2.14 and PHPUnit 10.5.5 at Sun Dec 31 21:41:09 UTC 2023. + Generated by php-code-coverage 10.1.10 using PHP 8.2.14 and PHPUnit 10.5.5 at Sun Dec 31 21:48:12 UTC 2023.
diff --git a/coverage.xml b/coverage.xml index a842fad..f37975b 100644 --- a/coverage.xml +++ b/coverage.xml @@ -1,6 +1,6 @@ -Insufficient Coverage
Project Risks
-Insufficient Coverage
Project Risks
-Project Risks
@@ -167,7 +157,7 @@Project Risks
.yAxis.tickFormat(d3.format('d')); d3.select('#classCoverageDistribution svg') - .datum(getCoverageDistributionData([2,0,0,0,0,0,0,0,0,0,0,8], "Class Coverage")) + .datum(getCoverageDistributionData([1,0,0,0,0,0,0,0,0,0,0,9], "Class Coverage")) .transition().duration(500).call(chart); nv.utils.windowResize(chart.update); @@ -185,7 +175,7 @@Project Risks
.yAxis.tickFormat(d3.format('d')); d3.select('#methodCoverageDistribution svg') - .datum(getCoverageDistributionData([8,0,0,0,0,0,0,0,0,0,0,8], "Method Coverage")) + .datum(getCoverageDistributionData([2,0,0,0,0,0,0,0,0,0,0,14], "Method Coverage")) .transition().duration(500).call(chart); nv.utils.windowResize(chart.update); @@ -235,7 +225,7 @@Project Risks
chart.yAxis.axisLabel('Cyclomatic Complexity'); d3.select('#classComplexity svg') - .datum(getComplexityData([[100,1,"FacelessLog\\Anonymizers\\BirthDateAnonymizer<\/a>"],[100,1,"FacelessLog\\Anonymizers\\CreditCardAnonymizer<\/a>"],[100,1,"FacelessLog\\Anonymizers\\EmailAnonymizer<\/a>"],[100,1,"FacelessLog\\Anonymizers\\PhoneAnonymizer<\/a>"],[0,2,"FacelessLog\\Aspect\\LogAnonymizerAspect<\/a>"],[100,2,"FacelessLog\\Detectors\\BirthDateDetector<\/a>"],[100,1,"FacelessLog\\Detectors\\CreditCardDetector<\/a>"],[100,1,"FacelessLog\\Detectors\\EmailDetector<\/a>"],[100,1,"FacelessLog\\Detectors\\PhoneDetector<\/a>"],[0,9,"FacelessLog\\FacelessLogger<\/a>"]], 'Class Complexity')) + .datum(getComplexityData([[100,1,"FacelessLog\\Anonymizers\\BirthDateAnonymizer<\/a>"],[100,1,"FacelessLog\\Anonymizers\\CreditCardAnonymizer<\/a>"],[100,1,"FacelessLog\\Anonymizers\\EmailAnonymizer<\/a>"],[100,1,"FacelessLog\\Anonymizers\\PhoneAnonymizer<\/a>"],[0,2,"FacelessLog\\Aspect\\LogAnonymizerAspect<\/a>"],[100,2,"FacelessLog\\Detectors\\BirthDateDetector<\/a>"],[100,1,"FacelessLog\\Detectors\\CreditCardDetector<\/a>"],[100,1,"FacelessLog\\Detectors\\EmailDetector<\/a>"],[100,1,"FacelessLog\\Detectors\\PhoneDetector<\/a>"],[100,9,"FacelessLog\\FacelessLogger<\/a>"]], 'Class Complexity')) .transition() .duration(500) .call(chart); @@ -259,7 +249,7 @@Project Risks
chart.yAxis.axisLabel('Method Complexity'); d3.select('#methodComplexity svg') - .datum(getComplexityData([[100,1,"FacelessLog\\Anonymizers\\BirthDateAnonymizer::anonymize<\/a>"],[100,1,"FacelessLog\\Anonymizers\\CreditCardAnonymizer::anonymize<\/a>"],[100,1,"FacelessLog\\Anonymizers\\EmailAnonymizer::anonymize<\/a>"],[100,1,"FacelessLog\\Anonymizers\\PhoneAnonymizer::anonymize<\/a>"],[0,1,"FacelessLog\\Aspect\\LogAnonymizerAspect::__construct<\/a>"],[0,1,"FacelessLog\\Aspect\\LogAnonymizerAspect::process<\/a>"],[100,2,"FacelessLog\\Detectors\\BirthDateDetector::detect<\/a>"],[100,1,"FacelessLog\\Detectors\\CreditCardDetector::detect<\/a>"],[100,1,"FacelessLog\\Detectors\\EmailDetector::detect<\/a>"],[100,1,"FacelessLog\\Detectors\\PhoneDetector::detect<\/a>"],[0,1,"FacelessLog\\FacelessLogger::__construct<\/a>"],[0,2,"FacelessLog\\FacelessLogger::getInstance<\/a>"],[0,1,"FacelessLog\\FacelessLogger::addDetector<\/a>"],[0,1,"FacelessLog\\FacelessLogger::clearDefaultDetectors<\/a>"],[0,1,"FacelessLog\\FacelessLogger::clearDefaultAnonymizers<\/a>"],[0,3,"FacelessLog\\FacelessLogger::processMessage<\/a>"]], 'Method Complexity')) + .datum(getComplexityData([[100,1,"FacelessLog\\Anonymizers\\BirthDateAnonymizer::anonymize<\/a>"],[100,1,"FacelessLog\\Anonymizers\\CreditCardAnonymizer::anonymize<\/a>"],[100,1,"FacelessLog\\Anonymizers\\EmailAnonymizer::anonymize<\/a>"],[100,1,"FacelessLog\\Anonymizers\\PhoneAnonymizer::anonymize<\/a>"],[0,1,"FacelessLog\\Aspect\\LogAnonymizerAspect::__construct<\/a>"],[0,1,"FacelessLog\\Aspect\\LogAnonymizerAspect::process<\/a>"],[100,2,"FacelessLog\\Detectors\\BirthDateDetector::detect<\/a>"],[100,1,"FacelessLog\\Detectors\\CreditCardDetector::detect<\/a>"],[100,1,"FacelessLog\\Detectors\\EmailDetector::detect<\/a>"],[100,1,"FacelessLog\\Detectors\\PhoneDetector::detect<\/a>"],[100,1,"FacelessLog\\FacelessLogger::__construct<\/a>"],[100,2,"FacelessLog\\FacelessLogger::getInstance<\/a>"],[100,1,"FacelessLog\\FacelessLogger::addDetector<\/a>"],[100,1,"FacelessLog\\FacelessLogger::clearDefaultDetectors<\/a>"],[100,1,"FacelessLog\\FacelessLogger::clearDefaultAnonymizers<\/a>"],[100,3,"FacelessLog\\FacelessLogger::processMessage<\/a>"]], 'Method Complexity')) .transition() .duration(500) .call(chart); diff --git a/index.html b/index.html index adfc224..afb133c 100644 --- a/index.html +++ b/index.html @@ -42,31 +42,31 @@
-
FacelessLogger.php
-
Legend
High: 90% to 100%- Generated by php-code-coverage 10.1.10 using PHP 8.2.14 and PHPUnit 10.5.5 at Sun Dec 31 21:41:09 UTC 2023. + Generated by php-code-coverage 10.1.10 using PHP 8.2.14 and PHPUnit 10.5.5 at Sun Dec 31 21:48:12 UTC 2023.
diff --git a/phpunit-report.html b/phpunit-report.html index 6eb2500..d95a6ac 100644 --- a/phpunit-report.html +++ b/phpunit-report.html @@ -93,6 +93,14 @@Email Detector (Unit\Detectors\EmailDetector)
Faceless Logger (Unit\FacelessLogger)
+-
+
- Singleton instance +
- Add and process detector anonymizer +
- Clear default detectors +
- Clear default anonymizers +
- Process message with no detectors +
Phone Anonymizer (Unit\Anonymizers\PhoneAnonymizer)
- Anonymize phone number diff --git a/src/FacelessLogger.php b/src/FacelessLogger.php index 537b20a..bf46faf 100644 --- a/src/FacelessLogger.php +++ b/src/FacelessLogger.php @@ -15,7 +15,7 @@ class FacelessLogger { - private static FacelessLogger $instance; + private static ?FacelessLogger $instance = null; private $detectors = []; private $anonymizers = []; @@ -27,10 +27,10 @@ public function __construct() $this->addDetector(new BirthDateDetector(), new BirthDateAnonymizer()); } - public static function getInstance() + public static function getInstance(): FacelessLogger { if (self::$instance === null) { - self::$instance = new FacelessLogger(); + self::$instance = new self(); } return self::$instance; } diff --git a/tests/Unit/FacelessLoggerTest.php b/tests/Unit/FacelessLoggerTest.php new file mode 100644 index 0000000..a85da93 --- /dev/null +++ b/tests/Unit/FacelessLoggerTest.php @@ -0,0 +1,75 @@ +logger = FacelessLogger::getInstance(); + } + + public function testSingletonInstance() + { + $logger1 = FacelessLogger::getInstance(); + $logger2 = FacelessLogger::getInstance(); + $this->assertSame($logger1, $logger2); + } + + public function testAddAndProcessDetectorAnonymizer() + { + $email = "test@example.com"; + $anonymizedEmail = "t***@example.com"; + + $this->logger->addDetector(new EmailDetector(), new EmailAnonymizer()); + $result = $this->logger->processMessage($email); + + $this->assertEquals($anonymizedEmail, $result); + } + + public function testClearDefaultDetectors() + { + $this->logger->addDetector(new EmailDetector(), new EmailAnonymizer()); + + $this->logger->clearDefaultDetectors(); + + $detectors = $this->getPrivateProperty($this->logger, 'detectors'); + $this->assertEmpty($detectors); + } + + public function testClearDefaultAnonymizers() + { + $this->logger->addDetector(new EmailDetector(), new EmailAnonymizer()); + + $this->logger->clearDefaultAnonymizers(); + + $anonymizers = $this->getPrivateProperty($this->logger, 'anonymizers'); + $this->assertEmpty($anonymizers); + } + + public function testProcessMessageWithNoDetectors() + { + $this->logger->clearDefaultDetectors(); + + $message = "test@example.com"; + $processedMessage = $this->logger->processMessage($message); + + $this->assertEquals($message, $processedMessage); + } + + private function getPrivateProperty($object, $propertyName) + { + $reflector = new \ReflectionClass($object); + $property = $reflector->getProperty($propertyName); + $property->setAccessible(true); + + return $property->getValue($object); + } +}