Added the dependency for test containers and logging format for console

[vikhyat187]

…le logs

Date: 2nd Dec

Developer Name: @vikhyat187

---

Issue Ticket Number

• Closes Setup database for integeration tests to run and GitHub actions #35

Description

This is PR for integration tests of /skills endpoint, and the setup for db in integration tests

Documentation Updated?

• Yes
• No

If your feature adds a new API, then documentation related to the feature like API-Contracts, and Data-models must be updated.

Breaking Changes

• Yes
• No

If your feature introduces breaking changes or if something is missing, please mention the related issue tickets.

Development Tested?

• Yes
• No

Confirm whether the changes have been tested locally during development.

Tested in Staging?

• Yes
• No

Indicate whether the changes have been tested in the staging environment.

Under Feature Flag

• Yes
• No

Specify if the changes are currently under a feature flag.

Database Changes

• Yes
• No

Indicate whether the changes include modifications to the database.

Note to developers

• This PR to be merged after this PR Adding the APIs for /skills route (Skills Entity) #34, as this has the code.

Test Coverage

Integration tests

[debanjanc01]

@vikhyat187 I tried checking out this branch on local, but it's unable to find the following

    private final UserService userService;
    private final SkillsService skillsService;
    private UserDTO user;
    private SkillDTO skill;

How are you running this in local without these classes?

[vikhyat187]

Hi @debanjanc01 they are two PRs which had to get merged before this one, they are not yet merged, in those PRs we have these classes, so I had to write test in those branch and check its working.

PR for skill service - #34
PR for user service - #22

[vikhyat187]

[heyrandhir]

Could we incorporate a test to verify the functionality of pagination. Additionally, I'd like to propose an adjustment in the endpoint structure. It seems that /skills/name/:name might be misleading, considering that name is not the primary resource identifier. I suggest using it as a query parameter instead of part of the path parameter.

[vikhyat187]

The name route we discussed in discord earlier if you can remember

[heyrandhir]

Maybe I missed that. Can you please let me know the summary of the discussion.

[vikhyat187]

https://discord.com/channels/673083527624916993/1114589148901294150/1173873152707801088
This one @heyrandhir

One change for exception handling, I'll take in a seperate ticket, as it might require a lot of changes.