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.

Sign up for GitHub

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

Jump to bottom

feat(test): adding mocking tests for type and subtype #941

Merged
merged 92 commits into from
Dec 19, 2024
Merged

feat(test): adding mocking tests for type and subtype #941

merged 92 commits into from
Dec 19, 2024

Conversation

thetif
Copy link
Collaborator

@thetif thetif commented Dec 17, 2024 •
edited
Loading

🎫 Linked Ticket

Contributes to OY2-31568

💬 Description / Notes

Continuing to add mocking endpoints for tests. Still working to get all of the backend mocked out.

🛠 Changes

  • added mocking endpoint for searching for types and subtypes
  • updated structure of mock handlers to make it clearer what they are mocking

thetif added 30 commits November 20, 2024 18:20
@thetif
tests starting to work
0142962
@thetif
saving here, can't mock auth
c29c5b0
@thetif
some tests passing
8aecd24
@thetif
mid working through getting lib tests to use msw too
3049a45
@thetif
Merge branch 'main' of github.com:Enterprise-CMCS/macpro-mako into mo…
c8e9975 
…cking-tests
@thetif
?
b3aa61f
@thetif
most tests passing
09c1d32
@thetif
all tests passing now
aa73b14
@thetif
Merge branch 'main' of github.com:Enterprise-CMCS/macpro-mako into mo…
8023281 
…cking-tests
@thetif
added async to renders
6b3c1f4
@thetif
removed vi.mock for useGetCounties
ad6d901
@thetif
updated useReadOnlyUser to wait for user to load before checking if t…
e77037e 
…hey were read only
@thetif
Merge branch 'main' of github.com:Enterprise-CMCS/macpro-mako into mo…
dfcac19 
…cking-tests
@thetif
reverting these because they are only formatting changes
fd60d3c
@thetif
rolled back files with only formatting changes, removed console.logs
f2e2fc0
@thetif
attributed borrowed code
0045d36
@thetif
fixed todo
bed47aa
@thetif
attempting to add auth mocks back in
745201a
@thetif
added debug log
693fa99
@thetif
debug logs for auth
fb4762d
@thetif
attempting to fix never issue with opensearch.changelog.Document
ff8e8d0
@thetif
changes to Document
b523cc4
@thetif
Merge branch 'main' of github.com:Enterprise-CMCS/macpro-mako into mo…
569656f 
…cking-tests
@thetif
reusing types instead of recreating them; fixing issue with opensearc…
53c3a97 
…h types
@thetif
added comment
baaa498
@thetif
couple more updates to opensearch types
11e11f0
@thetif
working on typescript issues
194c791
@thetif
more fixes to typescript
79ba1d1
@thetif
Merge branch 'main' of github.com:Enterprise-CMCS/macpro-mako into mo…
3eda647 
…cking-tests
@thetif
more debug logs
6d34d39
@thetif thetif temporarily deployed to mocking-type-tests December 19, 2024 14:52 — with GitHub Actions Inactive
@thetif thetif temporarily deployed to mocking-type-tests December 19, 2024 14:52 — with GitHub Actions Inactive
@thetif thetif temporarily deployed to mocking-type-tests December 19, 2024 14:54 — with GitHub Actions Inactive
@thetif thetif temporarily deployed to mocking-type-tests-app December 19, 2024 14:54 — with GitHub Actions Inactive
@thetif thetif temporarily deployed to mocking-type-tests December 19, 2024 14:55 — with GitHub Actions Inactive
@thetif thetif temporarily deployed to mocking-type-tests-kibana December 19, 2024 14:55 — with GitHub Actions Inactive
@thetif thetif temporarily deployed to mocking-type-tests December 19, 2024 14:55 — with GitHub Actions Inactive
@thetif thetif temporarily deployed to mocking-type-tests December 19, 2024 15:53 — with GitHub Actions Inactive
@thetif thetif temporarily deployed to mocking-type-tests December 19, 2024 15:53 — with GitHub Actions Inactive
@thetif thetif temporarily deployed to mocking-type-tests December 19, 2024 15:56 — with GitHub Actions Inactive
@thetif thetif temporarily deployed to mocking-type-tests-kibana December 19, 2024 15:56 — with GitHub Actions Inactive
@thetif thetif temporarily deployed to mocking-type-tests-app December 19, 2024 15:56 — with GitHub Actions Inactive
@thetif thetif temporarily deployed to mocking-type-tests December 19, 2024 15:56 — with GitHub Actions Inactive
@thetif thetif temporarily deployed to mocking-type-tests December 19, 2024 15:56 — with GitHub Actions Inactive
@thetif thetif temporarily deployed to mocking-type-tests December 19, 2024 16:36 — with GitHub Actions Inactive
@thetif thetif temporarily deployed to mocking-type-tests December 19, 2024 16:36 — with GitHub Actions Inactive
@thetif thetif temporarily deployed to mocking-type-tests December 19, 2024 16:39 — with GitHub Actions Inactive
@thetif thetif temporarily deployed to mocking-type-tests December 19, 2024 16:39 — with GitHub Actions Inactive
@thetif thetif temporarily deployed to mocking-type-tests December 19, 2024 16:39 — with GitHub Actions Inactive
@thetif thetif requested a review from thwalker6 December 19, 2024 16:52
benjaminpaige
benjaminpaige approved these changes Dec 19, 2024
View reviewed changes
mocks/handlers/opensearch/index.ts
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Love these rollup files

thwalker6
thwalker6 approved these changes Dec 19, 2024
View reviewed changes
Copy link
Collaborator

@thwalker6 thwalker6 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@thetif thetif merged commit 966b23c into main Dec 19, 2024
17 checks passed
@github-actions GitHub Actions
Copy link
Contributor

🎉 This PR is included in version 1.5.0-val.99 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@thetif thetif deleted the mocking-type-tests branch January 2, 2025 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@thwalker6 thwalker6 thwalker6 approved these changes

@benjaminpaige benjaminpaige benjaminpaige approved these changes

@13bfrancis 13bfrancis Awaiting requested review from 13bfrancis

Assignees
No one assigned
Labels
released on @val
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@thetif @benjaminpaige @thwalker6