Project

General

Profile

Actions

bug #9612

closed

ROLE_USER_MANAGER is missing as a group in FirstDataImporter

Added by Andreas Müller over 1 year ago. Updated over 1 year ago.

Status:
Closed
Priority:
Highest
Category:
cdmlib
Target version:
Start date:
Due date:
% Done:

100%

Estimated time:
Severity:
normal
Found in Version:
Tags:

Description

... for details see #9051

Or is it not wanted to have a group for this as default similar to Group_Project_Manager?


Related issues

Related to EDIT - bug #9051: Role Project manager should not have right to edit users and creating a new user with missing rights results in two message-boxesNewKatja Luther

Actions
Related to EDIT - feature request #9613: Use fixed uuids for roles project manager, publish and user managerClosedAndreas Müller

Actions
Actions #1

Updated by Andreas Müller over 1 year ago

  • Subject changed from ROLE_USER_MANAGER is missing in FirstDataImporter to ROLE_USER_MANAGER is missing as a group in FirstDataImporter
  • Description updated (diff)
Actions #2

Updated by Andreas Müller over 1 year ago

  • Status changed from New to In Progress
  • Assignee changed from Andreas Kohlbecker to Andreas Müller
  • Priority changed from New to Highest
  • % Done changed from 0 to 20
Actions #3

Updated by Andreas Müller over 1 year ago

  • Related to bug #9051: Role Project manager should not have right to edit users and creating a new user with missing rights results in two message-boxes added
Actions #4

Updated by Andreas Müller over 1 year ago

  • Related to feature request #9613: Use fixed uuids for roles project manager, publish and user manager added
Actions #5

Updated by Andreas Müller over 1 year ago

  • Status changed from In Progress to Resolved
  • % Done changed from 20 to 50
Actions #6

Updated by Andreas Müller over 1 year ago

  • Assignee changed from Andreas Müller to Andreas Kohlbecker

This is fixed and can be reviewed.

Actions #7

Updated by Andreas Kohlbecker over 1 year ago

  • Status changed from Resolved to In Progress
  • Assignee changed from Andreas Kohlbecker to Andreas Müller
  • % Done changed from 50 to 40

implementation ok, the FirstDataInserterTest however looks like it has never been implemented, seems like a copy of some other test.
Since you have changed some core code, I suggest to implement at least one test that covers addMissingAuthority()

Actions #8

Updated by Andreas Kohlbecker over 1 year ago

Andreas Kohlbecker wrote:

implementation ok, the FirstDataInserterTest however looks like it has never been implemented, seems like a copy of some other test.
Since you have changed some core code, I suggest to implement at least one test that covers addMissingAuthority()

I now see that there is another ticket (#9613) to which this rather refers to, please decide yourself in which context you want to deal with this.

Actions #9

Updated by Andreas Müller over 1 year ago

  • Status changed from In Progress to Resolved
  • Assignee changed from Andreas Müller to Andreas Kohlbecker

The addMissingAuthority method is a private method that is explicitly tested in testBlankDB by the result dataset "FirstDataInserterTest.testBlankDB-result.xml". This is why I extended the result dataset. With this result dataset I don't think there is a need for testing the method explicitly. Before there was a need because the adding of the authority was tested nowhere.

Actions #10

Updated by Andreas Müller over 1 year ago

  • % Done changed from 40 to 50
Actions #11

Updated by Andreas Müller over 1 year ago

But I also do not understand what the testInsertData() test method is for. It does not seem to be related to first data inserting. Should we remove it?

Actions #12

Updated by Andreas Kohlbecker over 1 year ago

  • Status changed from Resolved to Closed
  • Assignee changed from Andreas Kohlbecker to Andreas Müller
  • % Done changed from 50 to 100

Ja, die sollte weg, dass die hier keinen Sinn macht, meinte ich auch mit "seems like a copy of some other test."

Den Sinn der anderen Test-Methode habe ich nicht gleich erkannt da ich den Methodennamen falsch interpretiert hatte. Ich hab den leicht geändert und auch noch einen java doc Kommentar beigefügt.

Ich denke wir können das Ticket nun schließen

Actions #13

Updated by Andreas Müller over 1 year ago

  • Target version changed from Release 5.25 to Release 5.23
Actions

Also available in: Atom PDF