Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: remove old enrollments api integrations tests AP-1346 #282

Merged
merged 1 commit into from
Sep 26, 2024

Conversation

BryanttV
Copy link
Contributor

@BryanttV BryanttV commented Sep 24, 2024

Description

This PR removes the old integration test file for the enrollment API.

This PR is part of a set of PRs to add integration tests to the enrollment API. It must be separated into different PRs because doing it in one PR exceeds the maximum size allowed by the pr-size-labeler action.

Jira Issue

@BryanttV BryanttV force-pushed the bav/remove-old-enrollments-api-integration-tests branch from 6348ccb to 6cb5f42 Compare September 24, 2024 16:47
@BryanttV BryanttV requested a review from a team September 24, 2024 16:48
magajh
magajh previously approved these changes Sep 25, 2024
mariajgrimaldi
mariajgrimaldi previously approved these changes Sep 25, 2024
@BryanttV BryanttV changed the base branch from bav/add-integration-tests-to-enrollment-api to master September 26, 2024 13:22
@BryanttV BryanttV dismissed stale reviews from mariajgrimaldi and magajh September 26, 2024 13:22

The base branch was changed.

@BryanttV BryanttV force-pushed the bav/remove-old-enrollments-api-integration-tests branch from 6cb5f42 to 86456d2 Compare September 26, 2024 13:23
@BryanttV BryanttV force-pushed the bav/remove-old-enrollments-api-integration-tests branch from 86456d2 to c143803 Compare September 26, 2024 13:25
@github-actions github-actions bot added size/l and removed size/xl labels Sep 26, 2024
@BryanttV
Copy link
Contributor Author

Hi @magajh @mariajgrimaldi. I changed the merge to the master branch. Can you please approve the PR again? Thanks!

@BryanttV BryanttV force-pushed the bav/remove-old-enrollments-api-integration-tests branch from c143803 to 18ef276 Compare September 26, 2024 21:34
@BryanttV BryanttV merged commit 3de097e into master Sep 26, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants