Skip to content

Add complex object unification tests #133

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
iransamarasekara wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Add complex object unification tests #133

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
ff9fb25
Initial plan
Copilot Nov 27, 2025
ec69bd2
Add complex unification test scenarios
Copilot Nov 27, 2025
f3b7e1e
Finalize complex unification tests with code review and security checks
Copilot Nov 27, 2025
6f46d2e
Add Scenario 19: Two masters with existing hierarchies merge
iransamarasekara Dec 16, 2025
a7e1346
update child count assertions
iransamarasekara Dec 16, 2025
3ccf3c7
Add details about additional complex unification test scenarios to ma…
iransamarasekara Dec 16, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion go.mod
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ require (
github.com/google/uuid v1.6.0
github.com/joho/godotenv v1.5.1
github.com/lib/pq v1.10.9
github.com/pkg/errors v0.9.1
github.com/stretchr/testify v1.10.0
github.com/testcontainers/testcontainers-go v0.37.0
go.mongodb.org/mongo-driver v1.17.3
Expand Down Expand Up @@ -44,7 +45,6 @@ require (
github.com/morikuni/aec v1.0.0 // indirect
github.com/opencontainers/go-digest v1.0.0 // indirect
github.com/opencontainers/image-spec v1.1.1 // indirect
github.com/pkg/errors v0.9.1 // indirect
github.com/pmezard/go-difflib v1.0.0 // indirect
github.com/power-devops/perfstat v0.0.0-20210106213030-5aafc221ea8c // indirect
github.com/shirou/gopsutil/v4 v4.25.1 // indirect
Expand Down
10 changes: 10 additions & 0 deletions test/Unification_Scenarios.MD
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -56,3 +56,13 @@ It simulates multiple real-world merging situations across temporary and permane
| **7** | Rule Inactivation | Email rule temporarily disabled. | No merge occurs until rule is re-enabled. |
| **8** | Rule Change Stability | Existing merged profiles remain stable after rule changes. | Unification remains intact; hierarchy unchanged. |
| **9** | Cross-Tenant Isolation | Two profiles from different tenants share same identifiers. | Profiles remain separate; no cross-tenant unification. |
| **10** | Multiple Cascading Merges | Four profiles chain together - P1, P2 by email, P2, P3 by phone, P3, P4 by email. | All four merge into one master with combined data. |
| **11** | Deep Hierarchy Unification | Two profiles merge first, then a third joins via phone match. | All three end up in same hierarchy with all interests combined. |
| **12** | Application Data Merge | Profiles with different application data merge together. | App data from both profiles gets combined properly. |
| **13** | Update Triggers Reunification | Two separate profiles exist, one gets updated to add matching email. | After update, both profiles merge together. |
| **14** | Multiple Attributes Match | Two profiles match on both email AND phone at the same time. | They merge once using higher priority rule (email). |
| **15** | Four Profile Chain | Chain merge A to B to C to D through email and phone links. | All four merge with all preferences combined. |
| **16** | Permanent with Multiple Temps | One permanent profile, three temps join via email and phone. | All temps merge under the permanent profile. |
| **17** | Different Trait Types Merge | Profiles have arrays (interests) and scalars (score). | Arrays combine, scalars use last value. |
| **18** | Rule Reactivation | Rule disabled, profiles created, rule re-enabled, new profile added. | New profile triggers merge after rule reactivation. |
| **19** | Two Hierarchies Merge | Two separate master hierarchies (3 children + 2 children) connected by new profile. | All profiles end up in single unified hierarchy. |
Loading