Skip to content

[TILES-V2-11]: add unit and integration tests for tiles v2 #1033

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.

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
wants to merge 6 commits into
base: tiles-v2/load-tests
Choose a base branch
from

Conversation

pregnantboy
Copy link
Contributor

@pregnantboy pregnantboy commented Jun 3, 2025

Tests for Tiles v2

Changes

  1. Added a PostgreSQL container for integration tests
  2. Extending existing mutation tests to work with both DynamoDB and PostgreSQL
  3. Adding unit tests for PostgreSQL table functions
  4. Implementing proper database type switching in mutations

@pregnantboy pregnantboy changed the title chore: add pg container for tiles [TILES-V2-11]: add unit and integration tests for tiles v2 Jun 3, 2025
@pregnantboy pregnantboy marked this pull request as ready for review June 3, 2025 09:24
@pregnantboy pregnantboy requested a review from a team as a code owner June 3, 2025 09:24
@datadog-opengovsg
Copy link

datadog-opengovsg bot commented Jun 3, 2025

Datadog Report

Branch report: tiles-v2/unit-tests
Commit report: 623258c
Test service: plumber

✅ 0 Failed, 902 Passed, 0 Skipped, 3m 5.75s Total Time
⬆️ Test Sessions change in coverage: 1 increased (+1.34%)

cursor-com[bot]

This comment was marked as outdated.

@pregnantboy pregnantboy force-pushed the tiles-v2/unit-tests branch from c8bddb0 to c714718 Compare June 3, 2025 10:37
@pregnantboy pregnantboy force-pushed the tiles-v2/load-tests branch from cee0f04 to 86f6844 Compare June 3, 2025 10:37
cursor-com[bot]

This comment was marked as outdated.

cursor-com[bot]

This comment was marked as outdated.

@pregnantboy pregnantboy force-pushed the tiles-v2/unit-tests branch 2 times, most recently from 6fd3c52 to ab2090f Compare June 3, 2025 10:52
cursor-com[bot]

This comment was marked as outdated.

@pregnantboy pregnantboy force-pushed the tiles-v2/load-tests branch from 86f6844 to 8a0169b Compare June 3, 2025 11:02
@pregnantboy pregnantboy force-pushed the tiles-v2/unit-tests branch from ab2090f to 0b2acfd Compare June 3, 2025 11:02
cursor-com[bot]

This comment was marked as outdated.

cursor-com[bot]

This comment was marked as outdated.

cursor-com[bot]

This comment was marked as outdated.

@pregnantboy pregnantboy force-pushed the tiles-v2/load-tests branch from 8a0169b to 6707e1a Compare June 5, 2025 07:31
@pregnantboy pregnantboy force-pushed the tiles-v2/unit-tests branch from fe60198 to 63b1a17 Compare June 5, 2025 07:31
cursor-com[bot]

This comment was marked as outdated.

@pregnantboy pregnantboy force-pushed the tiles-v2/load-tests branch from 6707e1a to a960b53 Compare June 6, 2025 07:20
@pregnantboy pregnantboy force-pushed the tiles-v2/unit-tests branch from 63b1a17 to 623258c Compare June 6, 2025 07:21
Copy link

@cursor-com cursor-com bot left a comment

Choose a reason for hiding this comment

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

✅ BugBot reviewed your changes and found no bugs!


BugBot free trial expires on June 10, 2025
You have used $0.00 of your $1.00 spend limit so far. Manage your spend limit in the Cursor dashboard.

Was this report helpful? Give feedback by reacting with 👍 or 👎

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.

1 participant