The following conventions have been applied:
* Rule: (sizes) LineLength

The following conventions have been applied:
* Fix indentation levels
  Rules: (indentation) Indentation, (regexp) RegexpSinglelineJava
* Always use operators at the beginning of a new line if there is a line
  break.
  Rule: (whitespace) OperatorWrap
* Always use empty lines between method declarations.
  Rule: (whitespace) EmptyLineSeparator
* Always use new line between annotation and field/method declaration.
  Rule: (extension) AnnotationLocationMostCases
* Additional rules:
    * (whitespace) ParenPad
    * (whitespace) GenericWhitespace
    * (blocks) LeftCurly
    * (extension) RightCurlyAlone

The following conventions have been applied:
* Import order: static packages -> other packages -> ARSnova packages
  Rule: (imports) CustomImportOrder
* New line between import package groups
  Rule: (imports) CustomImportOrder
* All packages are imported explicitly (no wildcard imports)
  Rule: (imports) AvoidStarImport
* New line between license comment and package line
  Rule: (whitespace) EmptyLineSeparator

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

This also fixes the read permission room owners. They now have read
access to closed rooms.

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

The `/room/find` endpoint has been extended to support querying rooms by
moderator user ID. It can be used in combination with the `ownerId`
property.

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

Moderators can assist a room owner in managing the room. This commit
adds API and persistence support for moderators.

An error was logged before destroying UserServiceImpl class without any
reason.

An error was logged before destroying UserService class without any
reason.

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`.

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.

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)

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

add queue naming conventions to doc

add management interface into to rabbit file

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

remove unneccessary imports
use correct Nullable from checkerframework

remove unused imports

move ws handler to websocket package

add WebSocketMessage entity structure
add basic feedback logic
add tests for Feedback messages

These tests exercise the happy path. If needed, negative tests could
be added in the future.

Ektorp's default implementation expects purge_seq to be of type int.
CouchDB 2.x purge_seq is of type String.

Reverts 8ad7cd84

This fixes issue #33. However, it is still unclear why different
beans are shared between test and production code.

Both the RoomRepository as well as the CacheManager beans were
not shared between test and production classes. Hence, when we
set up some mocks (or caches), the mocked values were not
present in the production classes.

We explicitly force instance sharing by applying the Singleton pattern.

* Update MathJax URL to use version 2.7.5
* Piwik was renamed to Matomo

Only OIDC providers that support configuration discovery are supported.