Skip to content

test(global): created extraction and generation tests, and created ci#10

Merged
AndreAugusto11 merged 1 commit into
mainfrom
add-tests-and-ci
Jun 13, 2025
Merged

test(global): created extraction and generation tests, and created ci#10
AndreAugusto11 merged 1 commit into
mainfrom
add-tests-and-ci

Conversation

@AndreAugusto11

Copy link
Copy Markdown
Owner

No description provided.

@AndreAugusto11 AndreAugusto11 self-assigned this Jun 13, 2025
@AndreAugusto11 AndreAugusto11 force-pushed the add-tests-and-ci branch 9 times, most recently from df1cd42 to 0470ba0 Compare June 13, 2025 18:05
@AndreAugusto11 AndreAugusto11 requested a review from Copilot June 13, 2025 18:05

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull Request Overview

This PR introduces extraction and generation tests along with CI configuration while refactoring various repository and handler modules to update SQLAlchemy query usage and standardize handler constructor signatures. Key changes include:

  • Refactoring database access from session.query(...).get() to session.get(...).
  • Updating constructors for handlers and the Extractor to include a blockchains parameter.
  • Adjusting block range chunk sizes and adding a new commit-check GitHub workflow.

Reviewed Changes

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

Show a summary per file
File Description
repository/debridge/repository.py Updated SQLAlchemy query method usage.
repository/database.py Changed import of declarative_base to use sqlalchemy.orm.
repository/cctp/repository.py Updated SQLAlchemy query method usage.
repository/ccip/repository.py Updated SQLAlchemy query method usage.
repository/base.py Added a new get_all method for fetching records.
repository/across/repository.py Updated SQLAlchemy query method usage.
generator/omnibridge/generator.py Removed table clearance call and updated token mapping comments.
extractor/stargate/handler.py Updated init signature to include blockchains parameter.
extractor/ronin/handler.py Updated init signature to include blockchains parameter.
extractor/polygon/handler.py Updated init signature to include blockchains parameter.
extractor/omnibridge/handler.py Updated init signature to include blockchains parameter.
extractor/extractor.py Modified constructor and block range chunk calculation; enhanced docstring.
extractor/debridge/handler.py Updated init signature and delegated blockchain name conversion.
extractor/cctp/handler.py Updated init signature and altered blockchain name logic.
extractor/ccip/handler.py Updated init signature to include blockchains parameter.
extractor/base_handler.py Changed constructor to accept blockchains and added mapping logic.
extractor/across/handler.py Updated init signature and removed debug print.
config/rpcs_base_config.yaml Updated RPC endpoints and added new gnosis and BSC RPC entries.
cli/cli.py Updated instantiation of Extractor to pass blockchains parameter.
.github/workflows/commit-check.yml Added a new workflow to enforce commit conventions and sign-offs.
Comments suppressed due to low confidence (2)

extractor/extractor.py:174

  • [nitpick] Consider revising the comment for clarity, e.g. 'validate blockchain mapping', to better convey its intent.
# validate and blockchain mapping

generator/omnibridge/generator.py:224

  • Ensure that the removal of the empty_table call is intentional, as it might lead to residual data not being cleared and affect subsequent operations.
self.xdai_cross_chain_transactions.empty_table()

Signed-off-by: André Augusto <andre.augusto@tecnico.ulisboa.pt>
@AndreAugusto11 AndreAugusto11 merged commit 00005ff into main Jun 13, 2025
4 checks passed
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.

2 participants