Skip to content

Update the fixture files#80

Merged
gasperzgonec merged 1 commit intomainfrom
gasperz/ISS-291655
Apr 21, 2026
Merged

Update the fixture files#80
gasperzgonec merged 1 commit intomainfrom
gasperz/ISS-291655

Conversation

@gasperzgonec
Copy link
Copy Markdown
Contributor

Description

Connected Issues

Checklist

  • Tests added/updated and ran with npm run test OR no tests needed.
  • Code formatted and checked with npm run lint.
  • Added "How to test" section to the description OR this section is not needed.

Copilot AI review requested due to automatic review settings April 21, 2026 07:09
@github-actions
Copy link
Copy Markdown

⚠️ Heads-up: This repository will be blocked from any work other than patching.

File code/fixtures/start_extracting_data/event.json is not allowed to be modified in this patch.
The following vulnerability issues are past SLA:

Note that there is significant latency in updating this list. Please reach out on #antifragile if you are in a hurry or have an emergency.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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 updates the local test fixture event.json files to better match the AirdropEvent shape expected by the codebase (notably payload.event_type and payload.connection_data) while avoiding hard-coded API keys.

Changes:

  • Move connection_data (and event_context where applicable) under payload to align with runtime usage of event.payload.*.
  • Replace the hard-coded API key in the external-sync-units fixture with an environment-substituted placeholder (${TODO_API_KEY}).

Reviewed changes

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

File Description
code/fixtures/start_extracting_external_sync_units/event.json Nests connection_data under payload and switches the API key to ${TODO_API_KEY}.
code/fixtures/start_extracting_data/event.json Nests connection_data and event_context under payload to match expected event structure.

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

@gasperzgonec gasperzgonec merged commit 35c7915 into main Apr 21, 2026
7 of 8 checks passed
@gasperzgonec gasperzgonec deleted the gasperz/ISS-291655 branch April 21, 2026 10:24
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