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

Feature/plugin DI #63

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Feature/plugin DI #63

wants to merge 3 commits into from

Conversation

mateikki
Copy link
Member

@mateikki mateikki commented Oct 8, 2024

  • DI container added to the plugin
  • all services except controllers, helpers and base plugin class are now managed by the container
  • added unit tests that use mocked dependencies

@mateikki
- DI container added, dependency injection is now in the plugin
98f7b5f 
- tests fixed, now we have unit tests (with mocked dependencies) for the user application service as well as integration tests
@mateikki
- is callable check added to be consistent :D
ebbc5be
@mateikki
- test exception added to be used in mocked tests
7927df4 
- warnings removed
@mateikki mateikki added the feature New feature or request label Oct 8, 2024
@mateikki mateikki added this to the Starter V2 milestone Oct 8, 2024
@mateikki mateikki self-assigned this Oct 8, 2024
@fmarencic
Copy link
Member

I pao je build jer nema .env datoteke

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

@fmarencic fmarencic Awaiting requested review from fmarencic

@draganba draganba Awaiting requested review from draganba

At least 1 approving review is required to merge this pull request.

Assignees

@mateikki mateikki

Labels
feature New feature or request
Projects
None yet
Milestone
Starter V2
Development

Successfully merging this pull request may close these issues.
2 participants
@mateikki @fmarencic