Skip to content
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

Jump to bottom

Defnll/vax 230 introduce e2e with satellite #70

Merged
merged 1 commit into from
Nov 28, 2022
Merged

Defnll/vax 230 introduce e2e with satellite #70

merged 1 commit into from
Nov 28, 2022

Conversation

define-null
Copy link
Contributor

No description provided.

@linear
Copy link

linear bot commented Nov 24, 2022

VAX-230 Add e2e tests Satelite <> Electric

Introduce end to end tests for Satelite <> Vaxine. In order to do that, we need to figure out how to package and run Satelite as a standalone service in our test environment (docker + nodejs + necessary configuration + interface to access SQlite from the tests to start transactions)

@github-actions
Copy link
Contributor

LCOV of commit 20ea24e during CI #412

Summary coverage rate:
  lines......: 40.9% (1259 of 3075 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate:
                                                                   |Lines       |Functions  |Branches    
  Filename                                                         |Rate     Num|Rate    Num|Rate     Num
  =======================================================================================================
  lib/electric/migration_utils.ex                                  | 0.0%      8|    -     0|    -      0
  lib/electric/postgres/postgres_manager.ex                        | 0.0%     87|    -     0|    -      0
  lib/electric/replication/vaxine/log_consumer.ex                  |63.4%     41|    -     0|    -      0
  lib/electric/satellite/satellite_protocol.ex                     |78.0%    118|    -     0|    -      0

@github-actions
Copy link
Contributor

LCOV of commit a4dc378 during CI #416

Summary coverage rate:
  lines......: 41.0% (1262 of 3075 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate:
                                                                   |Lines       |Functions  |Branches    
  Filename                                                         |Rate     Num|Rate    Num|Rate     Num
  =======================================================================================================
  lib/electric/migration_utils.ex                                  | 0.0%      8|    -     0|    -      0
  lib/electric/postgres/postgres_manager.ex                        | 0.0%     87|    -     0|    -      0
  lib/electric/replication/vaxine/log_consumer.ex                  |63.4%     41|    -     0|    -      0
  lib/electric/satellite/satellite_protocol.ex                     |78.0%    118|    -     0|    -      0

@github-actions
Copy link
Contributor

LCOV of commit cce98d7 during CI #417

Summary coverage rate:
  lines......: 41.0% (1262 of 3075 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate:
                                                                   |Lines       |Functions  |Branches    
  Filename                                                         |Rate     Num|Rate    Num|Rate     Num
  =======================================================================================================
  lib/electric/migration_utils.ex                                  | 0.0%      8|    -     0|    -      0
  lib/electric/postgres/postgres_manager.ex                        | 0.0%     87|    -     0|    -      0
  lib/electric/replication/vaxine/log_consumer.ex                  |63.4%     41|    -     0|    -      0
  lib/electric/satellite/satellite_protocol.ex                     |78.0%    118|    -     0|    -      0

@define-null define-null force-pushed the defnll/vax-230-introduce-e2e-with-satellite branch from cce98d7 to 20ea24e Compare November 24, 2022 19:27
@github-actions
Copy link
Contributor

LCOV of commit 20ea24e during CI #418

Summary coverage rate:
  lines......: 41.0% (1262 of 3075 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate:
                                                                   |Lines       |Functions  |Branches    
  Filename                                                         |Rate     Num|Rate    Num|Rate     Num
  =======================================================================================================
  lib/electric/migration_utils.ex                                  | 0.0%      8|    -     0|    -      0
  lib/electric/postgres/postgres_manager.ex                        | 0.0%     87|    -     0|    -      0
  lib/electric/replication/vaxine/log_consumer.ex                  |63.4%     41|    -     0|    -      0
  lib/electric/satellite/satellite_protocol.ex                     |78.0%    118|    -     0|    -      0

@github-actions
Copy link
Contributor

LCOV of commit 9b3885f during CI #423

Summary coverage rate:
  lines......: 41.2% (1278 of 3100 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate:
                                                                   |Lines       |Functions  |Branches    
  Filename                                                         |Rate     Num|Rate    Num|Rate     Num
  =======================================================================================================
  lib/electric/migration_utils.ex                                  | 0.0%      8|    -     0|    -      0
  lib/electric/postgres/postgres_manager.ex                        | 0.0%     87|    -     0|    -      0
  lib/electric/replication/vaxine/log_consumer.ex                  |62.8%     43|    -     0|    -      0
  lib/electric/satellite/satellite_protocol.ex                     |78.5%    121|    -     0|    -      0

@github-actions
Copy link
Contributor

LCOV of commit e5e2348 during CI #424

Summary coverage rate:
  lines......: 41.4% (1282 of 3100 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate:
                                                                   |Lines       |Functions  |Branches    
  Filename                                                         |Rate     Num|Rate    Num|Rate     Num
  =======================================================================================================
  lib/electric/migration_utils.ex                                  | 0.0%      8|    -     0|    -      0
  lib/electric/postgres/postgres_manager.ex                        | 0.0%     87|    -     0|    -      0
  lib/electric/replication/vaxine/log_consumer.ex                  |62.8%     43|    -     0|    -      0
  lib/electric/satellite/satellite_protocol.ex                     |78.5%    121|    -     0|    -      0

@github-actions
Copy link
Contributor

LCOV of commit b99b612 during CI #425

Summary coverage rate:
  lines......: 41.2% (1277 of 3100 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate:
                                                                   |Lines       |Functions  |Branches    
  Filename                                                         |Rate     Num|Rate    Num|Rate     Num
  =======================================================================================================
  lib/electric/migration_utils.ex                                  | 0.0%      8|    -     0|    -      0
  lib/electric/postgres/postgres_manager.ex                        | 0.0%     87|    -     0|    -      0
  lib/electric/replication/vaxine/log_consumer.ex                  |62.8%     43|    -     0|    -      0
  lib/electric/satellite/satellite_protocol.ex                     |77.7%    121|    -     0|    -      0

@github-actions
Copy link
Contributor

LCOV of commit 3d7194b during CI #427

Summary coverage rate:
  lines......: 41.3% (1281 of 3100 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate:
                                                                   |Lines       |Functions  |Branches    
  Filename                                                         |Rate     Num|Rate    Num|Rate     Num
  =======================================================================================================
  lib/electric/migration_utils.ex                                  | 0.0%      8|    -     0|    -      0
  lib/electric/postgres/postgres_manager.ex                        | 0.0%     87|    -     0|    -      0
  lib/electric/replication/vaxine/log_consumer.ex                  |62.8%     43|    -     0|    -      0
  lib/electric/satellite/satellite_protocol.ex                     |77.7%    121|    -     0|    -      0

@github-actions
Copy link
Contributor

LCOV of commit 6f7481b during CI #428

Summary coverage rate:
  lines......: 41.3% (1281 of 3100 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate:
                                                                   |Lines       |Functions  |Branches    
  Filename                                                         |Rate     Num|Rate    Num|Rate     Num
  =======================================================================================================
  lib/electric/migration_utils.ex                                  | 0.0%      8|    -     0|    -      0
  lib/electric/postgres/postgres_manager.ex                        | 0.0%     87|    -     0|    -      0
  lib/electric/replication/vaxine/log_consumer.ex                  |62.8%     43|    -     0|    -      0
  lib/electric/satellite/satellite_protocol.ex                     |77.7%    121|    -     0|    -      0

@define-null define-null force-pushed the defnll/vax-230-introduce-e2e-with-satellite branch from 76072ec to ef91bad Compare November 28, 2022 13:56
@github-actions
Copy link
Contributor

LCOV of commit 76072ec during CI #429

Summary coverage rate:
  lines......: 41.4% (1282 of 3100 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate:
                                                                   |Lines       |Functions  |Branches    
  Filename                                                         |Rate     Num|Rate    Num|Rate     Num
  =======================================================================================================
  lib/electric/migration_utils.ex                                  | 0.0%      8|    -     0|    -      0
  lib/electric/postgres/postgres_manager.ex                        | 0.0%     87|    -     0|    -      0
  lib/electric/replication/vaxine/log_consumer.ex                  |62.8%     43|    -     0|    -      0
  lib/electric/satellite/satellite_protocol.ex                     |78.5%    121|    -     0|    -      0

@github-actions
Copy link
Contributor

LCOV of commit ef91bad during CI #430

Summary coverage rate:
  lines......: 41.2% (1278 of 3100 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate:
                                                                   |Lines       |Functions  |Branches    
  Filename                                                         |Rate     Num|Rate    Num|Rate     Num
  =======================================================================================================
  lib/electric/migration_utils.ex                                  | 0.0%      8|    -     0|    -      0
  lib/electric/postgres/postgres_manager.ex                        | 0.0%     87|    -     0|    -      0
  lib/electric/replication/vaxine/log_consumer.ex                  |62.8%     43|    -     0|    -      0
  lib/electric/satellite/satellite_protocol.ex                     |78.5%    121|    -     0|    -      0

@define-null define-null force-pushed the defnll/vax-230-introduce-e2e-with-satellite branch from ef91bad to 4dcb365 Compare November 28, 2022 14:04
@github-actions
Copy link
Contributor

LCOV of commit 4dcb365 during CI #431

Summary coverage rate:
  lines......: 41.3% (1279 of 3100 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate:
                                                                   |Lines       |Functions  |Branches    
  Filename                                                         |Rate     Num|Rate    Num|Rate     Num
  =======================================================================================================
  lib/electric/migration_utils.ex                                  | 0.0%      8|    -     0|    -      0
  lib/electric/postgres/postgres_manager.ex                        | 0.0%     87|    -     0|    -      0
  lib/electric/replication/vaxine/log_consumer.ex                  |62.8%     43|    -     0|    -      0
  lib/electric/satellite/satellite_protocol.ex                     |77.7%    121|    -     0|    -      0

@github-actions
Copy link
Contributor

LCOV of commit 652fe79 during CI #432

Summary coverage rate:
  lines......: 41.2% (1279 of 3101 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate:
                                                                   |Lines       |Functions  |Branches    
  Filename                                                         |Rate     Num|Rate    Num|Rate     Num
  =======================================================================================================
  lib/electric/application.ex                                      |83.3%     12|    -     0|    -      0
  lib/electric/migration_utils.ex                                  | 0.0%      8|    -     0|    -      0
  lib/electric/postgres/postgres_manager.ex                        | 0.0%     87|    -     0|    -      0
  lib/electric/replication/vaxine/log_consumer.ex                  |62.8%     43|    -     0|    -      0
  lib/electric/satellite/satellite_protocol.ex                     |78.5%    121|    -     0|    -      0

@define-null define-null force-pushed the defnll/vax-230-introduce-e2e-with-satellite branch from aa66daf to a501683 Compare November 28, 2022 19:10
@github-actions
Copy link
Contributor

LCOV of commit a501683 during CI #441

Summary coverage rate:
  lines......: 41.2% (1277 of 3100 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate:
                                                                   |Lines       |Functions  |Branches    
  Filename                                                         |Rate     Num|Rate    Num|Rate     Num
  =======================================================================================================
  lib/electric/migration_utils.ex                                  | 0.0%      8|    -     0|    -      0
  lib/electric/postgres/postgres_manager.ex                        | 0.0%     87|    -     0|    -      0
  lib/electric/replication/vaxine/log_consumer.ex                  |62.8%     43|    -     0|    -      0
  lib/electric/satellite/satellite_protocol.ex                     |77.7%    121|    -     0|    -      0

@github-actions
Copy link
Contributor

LCOV of commit c2442b6 during CI #442

Summary coverage rate:
  lines......: 41.4% (1282 of 3100 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate:
                                                                   |Lines       |Functions  |Branches    
  Filename                                                         |Rate     Num|Rate    Num|Rate     Num
  =======================================================================================================
  lib/electric/migration_utils.ex                                  | 0.0%      8|    -     0|    -      0
  lib/electric/postgres/postgres_manager.ex                        | 0.0%     87|    -     0|    -      0
  lib/electric/replication/vaxine/log_consumer.ex                  |62.8%     43|    -     0|    -      0
  lib/electric/satellite/satellite_protocol.ex                     |78.5%    121|    -     0|    -      0

@define-null define-null force-pushed the defnll/vax-230-introduce-e2e-with-satellite branch from f02c0b2 to 64a77a1 Compare November 28, 2022 20:08
@github-actions
Copy link
Contributor

LCOV of commit 64a77a1 during CI #444

Summary coverage rate:
  lines......: 41.2% (1278 of 3100 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate:
                                                                   |Lines       |Functions  |Branches    
  Filename                                                         |Rate     Num|Rate    Num|Rate     Num
  =======================================================================================================
  lib/electric/migration_utils.ex                                  | 0.0%      8|    -     0|    -      0
  lib/electric/postgres/postgres_manager.ex                        | 0.0%     87|    -     0|    -      0
  lib/electric/replication/vaxine/log_consumer.ex                  |62.8%     43|    -     0|    -      0
  lib/electric/satellite/satellite_protocol.ex                     |78.5%    121|    -     0|    -      0

@github-actions
Copy link
Contributor

LCOV of commit 70471dd during CI #445

Summary coverage rate:
  lines......: 41.4% (1282 of 3100 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate:
                                                                   |Lines       |Functions  |Branches    
  Filename                                                         |Rate     Num|Rate    Num|Rate     Num
  =======================================================================================================
  lib/electric/migration_utils.ex                                  | 0.0%      8|    -     0|    -      0
  lib/electric/postgres/postgres_manager.ex                        | 0.0%     87|    -     0|    -      0
  lib/electric/replication/vaxine/log_consumer.ex                  |62.8%     43|    -     0|    -      0
  lib/electric/satellite/satellite_protocol.ex                     |78.5%    121|    -     0|    -      0

@github-actions
Copy link
Contributor

LCOV of commit 773d807 during CI #446

Summary coverage rate:
  lines......: 41.2% (1278 of 3100 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate:
                                                                   |Lines       |Functions  |Branches    
  Filename                                                         |Rate     Num|Rate    Num|Rate     Num
  =======================================================================================================
  lib/electric/migration_utils.ex                                  | 0.0%      8|    -     0|    -      0
  lib/electric/postgres/postgres_manager.ex                        | 0.0%     87|    -     0|    -      0
  lib/electric/replication/vaxine/log_consumer.ex                  |62.8%     43|    -     0|    -      0
  lib/electric/satellite/satellite_protocol.ex                     |78.5%    121|    -     0|    -      0

@github-actions
Copy link
Contributor

LCOV of commit 915dd74 during CI #447

Summary coverage rate:
  lines......: 41.2% (1278 of 3100 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate:
                                                                   |Lines       |Functions  |Branches    
  Filename                                                         |Rate     Num|Rate    Num|Rate     Num
  =======================================================================================================
  lib/electric/migration_utils.ex                                  | 0.0%      8|    -     0|    -      0
  lib/electric/postgres/postgres_manager.ex                        | 0.0%     87|    -     0|    -      0
  lib/electric/replication/vaxine/log_consumer.ex                  |62.8%     43|    -     0|    -      0
  lib/electric/satellite/satellite_protocol.ex                     |78.5%    121|    -     0|    -      0

@github-actions
Copy link
Contributor

LCOV of commit ab61c4a during CI #448

Summary coverage rate:
  lines......: 41.4% (1282 of 3100 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate:
                                                                   |Lines       |Functions  |Branches    
  Filename                                                         |Rate     Num|Rate    Num|Rate     Num
  =======================================================================================================
  lib/electric/migration_utils.ex                                  | 0.0%      8|    -     0|    -      0
  lib/electric/postgres/postgres_manager.ex                        | 0.0%     87|    -     0|    -      0
  lib/electric/replication/vaxine/log_consumer.ex                  |62.8%     43|    -     0|    -      0
  lib/electric/satellite/satellite_protocol.ex                     |78.5%    121|    -     0|    -      0

@github-actions
Copy link
Contributor

LCOV of commit d9c531f during CI #449

Summary coverage rate:
  lines......: 41.4% (1282 of 3100 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate:
                                                                   |Lines       |Functions  |Branches    
  Filename                                                         |Rate     Num|Rate    Num|Rate     Num
  =======================================================================================================
  lib/electric/migration_utils.ex                                  | 0.0%      8|    -     0|    -      0
  lib/electric/postgres/postgres_manager.ex                        | 0.0%     87|    -     0|    -      0
  lib/electric/replication/vaxine/log_consumer.ex                  |62.8%     43|    -     0|    -      0
  lib/electric/satellite/satellite_protocol.ex                     |78.5%    121|    -     0|    -      0

@github-actions
Copy link
Contributor

LCOV of commit 67059b7 during CI #450

Summary coverage rate:
  lines......: 41.4% (1282 of 3100 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate:
                                                                   |Lines       |Functions  |Branches    
  Filename                                                         |Rate     Num|Rate    Num|Rate     Num
  =======================================================================================================
  lib/electric/migration_utils.ex                                  | 0.0%      8|    -     0|    -      0
  lib/electric/postgres/postgres_manager.ex                        | 0.0%     87|    -     0|    -      0
  lib/electric/replication/vaxine/log_consumer.ex                  |62.8%     43|    -     0|    -      0
  lib/electric/satellite/satellite_protocol.ex                     |78.5%    121|    -     0|    -      0

@github-actions
Copy link
Contributor

LCOV of commit 92943f1 during CI #453

Summary coverage rate:
  lines......: 40.8% (1246 of 3054 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate:
                                                                   |Lines       |Functions  |Branches    
  Filename                                                         |Rate     Num|Rate    Num|Rate     Num
  =======================================================================================================
  lib/electric/migration_utils.ex                                  | 0.0%      8|    -     0|    -      0
  lib/electric/postgres/postgres_manager.ex                        | 0.0%     87|    -     0|    -      0
  lib/electric/replication/vaxine/log_consumer.ex                  |63.4%     41|    -     0|    -      0
  lib/electric/satellite/satellite_protocol.ex                     |77.1%    118|    -     0|    -      0

@github-actions
Copy link
Contributor

LCOV of commit 033dcfe during CI #454

Summary coverage rate:
  lines......: 40.8% (1245 of 3054 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate:
                                                                   |Lines       |Functions  |Branches    
  Filename                                                         |Rate     Num|Rate    Num|Rate     Num
  =======================================================================================================
  lib/electric/migration_utils.ex                                  | 0.0%      8|    -     0|    -      0
  lib/electric/postgres/postgres_manager.ex                        | 0.0%     87|    -     0|    -      0
  lib/electric/replication/vaxine/log_consumer.ex                  |63.4%     41|    -     0|    -      0
  lib/electric/satellite/satellite_protocol.ex                     |77.1%    118|    -     0|    -      0

@define-null define-null force-pushed the defnll/vax-230-introduce-e2e-with-satellite branch from 033dcfe to af94598 Compare November 28, 2022 22:59
@define-null
[vax-230] Introduce Satellite E2E tests.
e1992c0 
Fix: github CI to use actual branch commit instead the merge on
top of main. (actions/checkout#881)

Fix: caching for docker and CI to make it more strict. We no longer
pick local-build image on CI for integration tests

Fix: migration tests that previously were skipped on CI

Fix: postgres manager is handling multiple requests originated from
the schema.

Fix: satellite protocol code would previously crash on start/stop/start
replication flow.
@define-null define-null force-pushed the defnll/vax-230-introduce-e2e-with-satellite branch from af94598 to e1992c0 Compare November 28, 2022 23:03
@github-actions
Copy link
Contributor

LCOV of commit af94598 during CI #455

Summary coverage rate:
  lines......: 41.2% (1278 of 3100 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate:
                                                                   |Lines       |Functions  |Branches    
  Filename                                                         |Rate     Num|Rate    Num|Rate     Num
  =======================================================================================================
  lib/electric/migration_utils.ex                                  | 0.0%      8|    -     0|    -      0
  lib/electric/postgres/postgres_manager.ex                        | 0.0%     87|    -     0|    -      0
  lib/electric/replication/vaxine/log_consumer.ex                  |62.8%     43|    -     0|    -      0
  lib/electric/satellite/satellite_protocol.ex                     |78.5%    121|    -     0|    -      0

@github-actions
Copy link
Contributor

LCOV of commit e1992c0 during CI #456

Summary coverage rate:
  lines......: 41.4% (1282 of 3100 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate:
                                                                   |Lines       |Functions  |Branches    
  Filename                                                         |Rate     Num|Rate    Num|Rate     Num
  =======================================================================================================
  lib/electric/migration_utils.ex                                  | 0.0%      8|    -     0|    -      0
  lib/electric/postgres/postgres_manager.ex                        | 0.0%     87|    -     0|    -      0
  lib/electric/replication/vaxine/log_consumer.ex                  |62.8%     43|    -     0|    -      0
  lib/electric/satellite/satellite_protocol.ex                     |78.5%    121|    -     0|    -      0

@define-null define-null marked this pull request as ready for review November 28, 2022 23:26
@define-null define-null merged commit ba3d79e into main Nov 28, 2022
@define-null define-null deleted the defnll/vax-230-introduce-e2e-with-satellite branch November 28, 2022 23:26
sekmet pushed a commit to sekmet/electric-sql that referenced this pull request Aug 15, 2024
@kevin-dp
docs: add where clause to API spec (electric-sql#70)
55f4bf1 
This PR documents the `where` query parameter in the API specification.
Closes electric-sql/archived-electric-next#47
KyleAMathews pushed a commit that referenced this pull request Nov 1, 2024
@kevin-dp
docs: add where clause to API spec (#70)
5ac0b4d 
This PR documents the `where` query parameter in the API specification.
Closes electric-sql/archived-electric-next#47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@icehaunter icehaunter Awaiting requested review from icehaunter

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@define-null