Skip to content

Refactor: add API versioning prefix, adjust test DB, update scripts#81

Open
Robinsonchiziterem wants to merge 4 commits into
VertexChainLabs:mainfrom
Robinsonchiziterem:api-versioning
Open

Refactor: add API versioning prefix, adjust test DB, update scripts#81
Robinsonchiziterem wants to merge 4 commits into
VertexChainLabs:mainfrom
Robinsonchiziterem:api-versioning

Conversation

@Robinsonchiziterem

Copy link
Copy Markdown

#23 Pull Request # Pull Request

Add API versioning prefix, SQLite test configuration, and update E2E test script

Description

  • Added global API version prefix (v1) to all backend routes while keeping /health unprefixed.
  • Configured SQLite in‑memory database for the test environment in DatabaseModule to allow fast, isolated E2E tests.
  • Updated E2E test script to set NODE_ENV=test on Windows (set NODE_ENV=test && jest …).
  • Adjusted E2E test files to use the new /v1 routes.
  • Fixed package scripts to work correctly on Windows platforms.

Related issue: #23

Checklist

  • Tests added or updated
  • Documentation updated (README, docs, comments)
  • Linting passes
  • Type checking passes (if applicable)
  • Relevant issue linked
  • Changelog updated (if applicable)

Security

No security‑sensitive changes are included. No secrets, authentication logic, or data exposure modifications were made.

Notes for reviewers

  • Verify that the global prefix (/v1) is applied to all routes except /health.

  • Run the E2E test suite locally with:

    npm run test:e2e
    
    

Add API versioning prefix, SQLite test configuration, and update E2E test script

Description

  • Added global API version prefix (v1) to all backend routes while keeping /health unprefixed.
  • Configured SQLite in‑memory database for the test environment in DatabaseModule to allow fast, isolated E2E tests.
  • Updated E2E test script to set NODE_ENV=test on Windows (set NODE_ENV=test && jest …).
  • Adjusted E2E test files to use the new /v1 routes.
  • Fixed package scripts to work correctly on Windows platforms.

Related issue: #23

Checklist

  • Tests added or updated
  • Documentation updated (README, docs, comments)
  • Linting passes
  • Type checking passes (if applicable)
  • Relevant issue linked
  • Changelog updated (if applicable)

Security

No security‑sensitive changes are included. No secrets, authentication logic, or data exposure modifications were made.

Notes for reviewers

  • Verify that the global prefix (/v1) is applied to all routes except /health.

  • Run the E2E test suite locally with:

    npm run test:e2e

Closes #23

Copy link
Copy Markdown
Contributor

Hey @Robinsonchiziterem 👋 Thanks for the refactor — the /v1 prefix and the SQLite-backed test DB are solid improvements. The Backend (NestJS) CI check is currently failing on this PR though. Could you investigate the failure and push a fix? Once everything is green I’ll be ready to merge this to close #23.

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.

refactor: add API versioning prefix to backend routes

2 participants