• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In
Warning: This build has drifted.
The coverage report for this pull request build may be inaccurate because its base commit is no longer the HEAD of its target branch.
This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

    • Learn more: For more information on this, see Tracking coverage changes for pull request builds.
    • Fix now: For a quick fix, rebase this PR at GitHub. Your next report should be accurate.
    • Prevent going forward: To avoid this issue with future PRs, see these Recommended CI Configurations.
New Repo Setting:
INCLUDE COVERAGE % WITH WARNINGS ABOUT DRIFTED BUILDS?

Enabling this setting will include a (potentially inaccurate) coverage % with warning messages in status updates for drifted builds.

Adjust setting

vocdoni / saas-backend / 25042049101
60%
main: 60%

Build:
Build:
LAST BUILD BRANCH: f/autogroups-434
DEFAULT BRANCH: main
Ran 28 Apr 2026 08:25AM UTC
Jobs 1
Files 90
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

24 Apr 2026 10:36AM UTC coverage: 59.563% (-0.2%) from 59.808%
25042049101

Pull #435

github

elboletaire
test(db): add regression test for twoFa-only duplicate detection in census validation

Before the fix, CheckGroupMembersFields skipped the duplicate-detection
block entirely when authFields was empty, even when twoFaFields contained
the only identity fields (e.g. email+phone for a CSP census). All members
passed validation as valid, and the failure only surfaced much later as
E11000 duplicate-key errors during the census participant bulk-write.

The test creates three members: two sharing the same phone number (phone
uniqueness is not enforced by the orgMembers index, making same-phone
members a realistic data state) and one with a unique phone. It then calls
CheckGroupMembersFields with authFields=[] and twoFaFields=[phone] and
asserts that:
  - both same-phone members appear in Duplicates
  - neither the second-seen duplicate nor the unique member is
    misclassified
  - MissingData is empty

This test would have failed against the unfixed code and passes with the
one-line guard change (len(authFields) > 0 || len(twoFaFields) > 0).

refs #434
Pull Request #435: feat(groups): autogroup all members

169 of 248 new or added lines in 6 files covered. (68.15%)

249 existing lines in 3 files now uncovered.

7543 of 12664 relevant lines covered (59.56%)

39.24 hits per line

Uncovered Changes

Lines Coverage ∆ File
33
52.17
migrations/0011_create_all_members_auto_group.go
18
81.39
-2.13% db/org_members.go
18
88.18
-1.73% db/org_members_groups.go
5
71.56
1.3% api/organization_groups.go
5
55.28
-0.97% subscriptions/subscriptions.go

Coverage Regressions

Lines Coverage ∆ File
187
48.99
-9.99% csp/handlers/handlers.go
60
65.67
-0.16% csp/auth.go
2
88.18
-1.73% db/org_members_groups.go
Jobs
ID Job ID Ran Files Coverage
1 25042049101.1 28 Apr 2026 08:25AM UTC 90
59.56
GitHub Action Run
Source Files on build 25042049101
  • Tree
  • List 90
  • Changed 10
  • Source Changed 0
  • Coverage Changed 10
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Repo
  • Pull Request #435
  • PR Base - main (#22758966323)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc