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

add interview logic and part of offer #14

Merged
merged 1 commit into from
Sep 12, 2024
Merged

add interview logic and part of offer #14

merged 1 commit into from
Sep 12, 2024

Conversation

januschung
Copy link
Owner

For both interview and offer:

  • add create table statements to schema.sql
  • add mutation, type, query, input definition to schema.graphqls
  • add model class definitions

More for interview:

  • add controller, repository and service class
  • add unit tests

Other change:

  • refactor JobApplicationGraphQlTest to use mock instead of actual db (somehow it still rely on a in memory db which cause test data initialization and test data insertion problem. The fix is to generate a separate db per test script with random.uuid
  • add more test to JobApplicationGGraphQlTest
  • fix logging typo

@januschung
Copy link
Owner Author

@oseasmoran73 can you review when you have a chance?

@januschung januschung merged commit b21b698 into main Sep 12, 2024
1 check passed
@januschung januschung deleted the interview-offer branch September 12, 2024 17:47
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.

2 participants