#417 Fix #404 no more dup message on combined validators

Merged Bruno Rocha rochacbruno

No flags found

Use flags to group coverage reports by test type, project and/or folders.
Then setup custom commit statuses and notifications for each flag.

e.g., #unittest #integration

#production #enterprise

#frontend #backend

Learn more about Codecov Flags here.

Showing 1 of 3 files from the diff.

@@ -262,9 +262,13 @@
Loading
262 262
                continue
263 263
            else:
264 264
                return
265 +
265 266
        raise ValidationError(
266 267
            self.messages["combined"].format(
267 -
                errors=" or ".join(str(e) for e in errors)
268 +
                errors=" or ".join(
269 +
                    str(e).replace("combined validators failed ", "")
270 +
                    for e in errors
271 +
                )
268 272
            )
269 273
        )
270 274
@@ -281,10 +285,14 @@
Loading
281 285
            except ValidationError as e:
282 286
                errors.append(e)
283 287
                continue
288 +
284 289
        if errors:
285 290
            raise ValidationError(
286 291
                self.messages["combined"].format(
287 -
                    errors=" and ".join(str(e) for e in errors)
292 +
                    errors=" and ".join(
293 +
                        str(e).replace("combined validators failed ", "")
294 +
                        for e in errors
295 +
                    )
288 296
                )
289 297
            )
290 298

Everything is accounted for!

No changes detected that need to be reviewed.
What changes does Codecov check for?
Lines, not adjusted in diff, that have changed coverage data.
Files that introduced coverage data that had none before.
Files that have missing coverage data that once were tracked.
Files Coverage
dynaconf 100.00%
Project Totals (17 files) 100.00%
Loading