GoogleCloudPlatform / dlp-dataflow-deidentification

Multi Cloud Data Tokenization Solution By Using Dataflow and Cloud DLP
Apache License 2.0
89 stars 53 forks source link

Inspect classify #199

Open dup05 opened 5 months ago

dup05 commented 5 months ago

Summary (Short summary of what is being done) :


Description (Describe in detail the fix made) :


Bug ID (if any) :


Public Documentation (if any) :


TESTED (Test Cases with scenario and description - must have 1 positive and 1 negative scenario) :

codecov[bot] commented 5 months ago

Codecov Report

Attention: Patch coverage is 0% with 292 lines in your changes are missing coverage. Please review.

Project coverage is 12.02%. Comparing base (1d32b05) to head (b41d6c9).

Files Patch % Lines
...le/swarm/tokenization/transforms/ProcessFiles.java 0.00% 67 Missing :warning:
.../swarm/tokenization/pipelines/InspectClassify.java 0.00% 54 Missing :warning:
...ogle/swarm/tokenization/transforms/ProcessCSV.java 0.00% 25 Missing :warning:
...le/swarm/tokenization/transforms/ProcessJsonl.java 0.00% 23 Missing :warning:
...arm/tokenization/classification/ClassifyFiles.java 0.00% 21 Missing :warning:
...gle/swarm/tokenization/transforms/ProcessAvro.java 0.00% 20 Missing :warning:
.../swarm/tokenization/transforms/ProcessParquet.java 0.00% 20 Missing :warning:
...kenization/classification/CreatePubSubMessage.java 0.00% 19 Missing :warning:
...ion/classification/ProcessInspectFindingsDoFn.java 0.00% 16 Missing :warning:
...ation/transforms/ReadFilesWithGivenExtensions.java 0.00% 14 Missing :warning:
... and 3 more
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #199 +/- ## ============================================ - Coverage 13.35% 12.02% -1.34% Complexity 67 67 ============================================ Files 53 64 +11 Lines 2531 2811 +280 Branches 213 220 +7 ============================================ Hits 338 338 - Misses 2173 2453 +280 Partials 20 20 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.