Skip to content
This repository has been archived by the owner on Dec 27, 2024. It is now read-only.

test: initial tests for persistence (context, announcements, categories) #21

Merged
merged 3 commits into from
Oct 12, 2023

Conversation

kurtaking
Copy link
Member

@kurtaking kurtaking commented Oct 12, 2023

The CatagoriesDatabase and persistenceContext are now reporting 100% test coverage. There is more work to be done for the AnnouncementsDatabase, but this PR includes a test to get us started.

image

Checklist:

  • I have updated the necessary documentation
  • I have signed off all my commits as required by DCO
  • My build is green

@kurtaking kurtaking self-assigned this Oct 12, 2023
@kurtaking kurtaking marked this pull request as ready for review October 12, 2023 14:03
@kurtaking kurtaking requested a review from a team as a code owner October 12, 2023 14:03
@kurtaking kurtaking requested a review from jmeridth October 12, 2023 14:05
@kurtaking kurtaking changed the title test: initial tests for persistence context and categories database test: initial tests for persistence (context, announcements, categories) Oct 12, 2023
@kurtaking kurtaking enabled auto-merge (squash) October 12, 2023 14:21
@kurtaking kurtaking disabled auto-merge October 12, 2023 15:02
@kurtaking
Copy link
Member Author

Feeling good about 🚢 ing this.

@kurtaking kurtaking merged commit a96ffcb into main Oct 12, 2023
@kurtaking kurtaking deleted the test-persistence branch October 12, 2023 15:03
cursedcoder pushed a commit to cursedcoder/backstage-plugin-announcements that referenced this pull request Oct 27, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant