Development: Unify json usage

[basak-akan]

Checklist

General

• This is a small issue that I tested locally and was confirmed by another developer on a test server.
• Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
• I chose a title conforming to the naming conventions for pull requests.

Server

• Important: I implemented the changes with a very good performance and prevented too many (unnecessary) database calls.
• I followed the coding and design guidelines.

Motivation and Context

Currently, different JSON implementations are used.

Description

Unified Json implementations to use gson library. Also some of the JsonParser implementations are replaced with ObjectMapper. Added an architectural test to prevent the use of other user libraries in the future.

Steps for Testing

Prerequisites:

• Please use test server 11 where LTI is enabled

1. Log in to https://moodle.ase.in.tum.de/ using username: student1 and password: as same password with artemis_test_user_1
2. Navigate to LTI Testing course and select TS11 Authentication Test Exercise.
3. Login to Artemis using artemis_test_user_1 credentials.
4. Verify you can see the exercise and participate in the exercise.

Testserver States

Note

These badges show the state of the test servers.
Green = Currently available, Red = Currently locked

Review Progress

Code Review

• Code Review 1
• Code Review 2

Manual Tests

• Test 1
• Test 2

Test Coverage

no change in test coverage since only replaced Json implementations.

Screenshots

[tobias-lippert]

Code LGTM

[julian-christl]

Looks good

[Strohgelaender]

server tests currently fail with a NullPointerException.

failing tests

server tests fixed

[Strohgelaender]

code

[tobias-lippert]

LGTM

[krusche]

Code looks good 👍 thank you