Daniel Gerhardt authored 5 years ago and Daniel Gerhardt committed 5 years ago

Authenticated users can read `loginId` and `authProvider` properties.
A FindQueryService has been added to lookup UserProfiles by `loginId`.

Resolve "Tests should not use env specific arsnova.properties"

Closes #41

See merge request arsnova/arsnova-backend!132

Add tests for JsonViewControllerAdvice

See merge request !129

Christoph Thelen authored 5 years ago and Daniel Gerhardt committed 5 years ago

Two key changes:

  1. Add a factory method to create the EntityService with the
     DummyEntity, so that all other test cases can initialize
     this Bean.  This solves the IllegalStateException reported
     in this ticket.
  2. Use @Autowire instead of @MockBean to use the new factory
     method.  That resolves the known issue that the caching test
     case does not work because of different RoomRepositories being
     created for the PermissionEvaluatorAdvice and the EntityService.

This makes all tests runnable again.  My current theory is that due to
the fact that we have at least two EntityServices (one created by
the DefaultEntityServiceImplTest, and now one by this test).  The
@MockBean annotation could be throwing out the previous EntityService,
since according to documentation, it replaces existing Beans with the
same type.  This could lead to a situation where, initially, the
PermissionEvaluatorAdvice and the DefaultEntityService share the same
RoomRepository.  But after the @MockBean replacement occurs, the
EntityService is recreated with a different RoomRepository, while the
Advice keeps its initial RoomRepository, leading to testing failures
in the testCaching test case.

Christoph Thelen authored 5 years ago and Daniel Gerhardt committed 5 years ago

This solves the IllegalStateExceptions occurring for most of the other
test cases.

However, we now have the old problem that different Repositories are
injected into objects. In my case, the testCaching case of the
DefaultEntityServiceImplTest stops working, because it gets a different
RoomRepository than the ApplicationPermissionEvaluator.

This behavior is caused by the introduction of the additional profile.

Daniel Gerhardt authored 5 years ago and Daniel Gerhardt committed 5 years ago

See commit f620b9e7.

Add nullpointer check for passwordResetTime

Closes #40

See merge request arsnova/arsnova-backend!127

Add a 'view' parameter for all entity endpoints

See merge request !128

A JsonView for serialization is applied based on the 'view' query
parameter.

The following values are currently supported:
* admin (View.Admin.class)
* owner (View.Owner.class)
* <unset> (Public.Admin.class)

Return 403 Forbidden on duplicated username when registering

Closes #38

See merge request arsnova/arsnova-backend!126

The RegEx unintentionally required a minimum length of 2 for IDs.

Enable rabbitmq

See merge request arsnova/arsnova-backend!122

add queue naming conventions to doc

add management interface into to rabbit file

Add auth logic for stomp clients over ws

See merge request arsnova/arsnova-backend!123

Tom Käsler authored 5 years ago and Daniel Gerhardt committed 5 years ago

remove unneccessary imports
use correct Nullable from checkerframework

Fix feedback messaging

See merge request arsnova/arsnova-backend!121

The version set for Pac4j was previously ineffective because it was set
by spring-security-pac4j first.

The Spring Platform BOM has reached EOL, so we need to replace it for
future updates. This should not change any dependency versions.

https://spring.io/blog/2018/04/09/spring-io-platform-end-of-life-announcement

See GH-65.

Add websocket handler to component scan

See merge request !119

add WebSockets with STOMP on top

See merge request !112

remove unused imports