Skip to content

Maintenance: Break down event_source_data_classes tests #509

Closed
@heitorlessa

Description

@heitorlessa
Contributor

Issue: #504

Functional tests for event source data classes utility are now 1.3K LOC in a single file. This issue tracks resolving them into a test per event source to ease maintenance and contrib

Activity

michaelbrewer

michaelbrewer commented on Jul 23, 2021

@michaelbrewer
Contributor

@heitorlessa - i don't mind splitting this up :)

heitorlessa

heitorlessa commented on Jul 24, 2021

@heitorlessa
ContributorAuthor
michaelbrewer

michaelbrewer commented on Jan 7, 2022

@michaelbrewer
Contributor

@heitorlessa - do you think this is still necessary? These tests are extremely fast, so it is not for performance reasons. Going forward for new types of event sources, i will make sure to split out the tests. And if there is any major refactoring or changes to the existing ones.

added
tech-debtTechnical Debt tasks
and removed
internalMaintenance changes
on Apr 17, 2023
moved this from Triage to Backlog in Powertools for AWS Lambda (Python)on Jun 19, 2023

8 remaining items

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

event_sourcesEvent Source Data Class utilityhelp wantedCould use a second pair of eyes/handstech-debtTechnical Debt tasks

Type

No type

Projects

Status

Shipped

Milestone

No milestone

Relationships

None yet

    Participants

    @heitorlessa@leandrodamascena@michaelbrewer

    Issue actions

      Maintenance: Break down event_source_data_classes tests · Issue #509 · aws-powertools/powertools-lambda-python