Skip to content

test: cover member organization affiliations (CM-1180)#4327

Merged
skwowet merged 4 commits into
mainfrom
tests/CM-1180
Jul 13, 2026
Merged

test: cover member organization affiliations (CM-1180)#4327
skwowet merged 4 commits into
mainfrom
tests/CM-1180

Conversation

@skwowet

@skwowet skwowet commented Jul 12, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@skwowet skwowet self-assigned this Jul 12, 2026
Copilot AI review requested due to automatic review settings July 12, 2026 19:03

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds integration coverage for member organization affiliation timeline selection and activity refresh behavior.

Changes:

  • Tests timeline fallbacks, overrides, priorities, manual affiliations, and email domains.
  • Tests activity relation updates across timeline scenarios.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI review requested due to automatic review settings July 13, 2026 07:05

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.

Copilot AI review requested due to automatic review settings July 13, 2026 08:10

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

Comment thread services/libs/data-access-layer/src/member-organization-affiliation/index.test.ts Outdated
Comment thread services/libs/data-access-layer/src/member-organization-affiliation/index.test.ts Outdated
Copilot AI review requested due to automatic review settings July 13, 2026 09:51

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

Comment thread services/libs/data-access-layer/src/member-organization-affiliation/index.test.ts Outdated
Comment thread docs/adr/0008-how-we-write-unit-tests.md
Copilot AI review requested due to automatic review settings July 13, 2026 10:19
@skwowet skwowet changed the title test: cover member organization affiliation refresh and timeline (CM-1180) test: cover member organization affiliations (CM-1180) Jul 13, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.

Comment thread services/libs/data-access-layer/src/member-organization-affiliation/index.test.ts Outdated
Comment thread services/libs/data-access-layer/src/member-organization-affiliation/index.test.ts Outdated
Comment thread services/libs/data-access-layer/src/member-organization-affiliation/index.test.ts Outdated
Comment thread services/libs/data-access-layer/src/member-organization-affiliation/index.test.ts Outdated
Copilot AI review requested due to automatic review settings July 13, 2026 11:04

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

Comment thread services/libs/data-access-layer/src/member-organization-affiliation/index.test.ts Outdated
Comment thread services/libs/data-access-layer/src/member-organization-affiliation/index.test.ts Outdated
Copilot AI review requested due to automatic review settings July 13, 2026 11:15

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.

Comment thread services/libs/data-access-layer/src/member-organization-affiliation/index.test.ts Outdated
Comment thread services/libs/data-access-layer/src/member-organization-affiliation/index.test.ts Outdated
Comment thread services/libs/data-access-layer/src/member-organization-affiliation/index.test.ts Outdated
Comment thread services/libs/data-access-layer/src/member-organization-affiliation/index.test.ts Outdated
Base automatically changed from test-kit-primitives to main July 13, 2026 16:23
skwowet and others added 2 commits July 13, 2026 22:00
Signed-off-by: Yeganathan S <63534555+skwowet@users.noreply.github.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
…1180)

Signed-off-by: Yeganathan S <63534555+skwowet@users.noreply.github.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Copilot AI review requested due to automatic review settings July 13, 2026 16:32

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

Comment thread services/libs/data-access-layer/src/member-organization-affiliation/index.test.ts Outdated
Comment thread services/libs/data-access-layer/src/member-organization-affiliation/index.test.ts Outdated
Signed-off-by: Yeganathan S <63534555+skwowet@users.noreply.github.com>
Copilot AI review requested due to automatic review settings July 13, 2026 16:40
@skwowet skwowet marked this pull request as ready for review July 13, 2026 16:40

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

Signed-off-by: Yeganathan S <63534555+skwowet@users.noreply.github.com>
Copilot AI review requested due to automatic review settings July 13, 2026 16:52
@skwowet skwowet merged commit 8ec9eca into main Jul 13, 2026
12 checks passed
@skwowet skwowet deleted the tests/CM-1180 branch July 13, 2026 16:53

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

Comment on lines +1261 to +1273
...generateActivityRelations({
memberId: member.id,
segmentId: kubernetes.id,
identities,
count: 30,
timestamp: { from: '2022-01-01T00:00:00.000Z', to: '2022-06-01T00:00:00.000Z' },
}),
])

await refreshMemberOrganizationAffiliations(qx, member.id)

expect(await countByOrg(qx, member.id, kubernetes.id, sponsor.id)).toBe(40)
expect(await countByOrg(qx, member.id, kubernetes.id, employer.id)).toBe(30)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants