Project

General

Profile

Development #41939

Les choix d'utilisateurs à ajouter dans un rôle ne tienne pas compte du setting qui restreint la collectivité

Added by Valentin Deniaud 2 months ago. Updated 27 days ago.

Status:
Solution déployée
Priority:
Normal
Category:
-
Target version:
-
Start date:
21 Apr 2020
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

  • Mettre ROLE_MEMBERS_FROM_OU à True
  • Aller sur la page d'un rôle d'une OU
  • Dans la liste des utilisateurs ajoutables, on voit ceux de toutes les OU

C'est cassé depuis #41017, ça sera sûrement résolu par #37159, mais il faudrait fixer le test qui fait

    assert not response.context['form'].fields['user'].queryset.query.where

au lieu de vraiment regarder quels choix sont affichés.

0002-tests-check-for-actual-widget-choices-41939.patch View (2.01 KB) Valentin Deniaud, 28 May 2020 03:07 PM

0001-tests-add-select2-test-helper-41939.patch View (5.9 KB) Valentin Deniaud, 28 May 2020 03:07 PM

Associated revisions

Revision 80bb97d3 (diff)
Added by Valentin Deniaud about 1 month ago

tests: add select2 test helper (#41939)

Revision ffb04c1a (diff)
Added by Valentin Deniaud about 1 month ago

tests: check for actual widget choices (#41939)

History

#1 Updated by Valentin Deniaud about 1 month ago

  • Assignee set to Valentin Deniaud

#2 Updated by Valentin Deniaud about 1 month ago

#3 Updated by Benjamin Dauvergne about 1 month ago

  • Status changed from Solution proposée to Solution validée

#4 Updated by Valentin Deniaud about 1 month ago

  • Status changed from Solution validée to Résolu (à déployer)
commit ffb04c1ab31fa054d7334844cf03c39109e6a1fc
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date:   Thu May 28 15:05:37 2020 +0200

    tests: check for actual widget choices (#41939)

commit 80bb97d32842aa8fdb38ecb73d89aafb21238e88
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date:   Thu May 28 14:24:06 2020 +0200

    tests: add select2 test helper (#41939)

#5 Updated by Frédéric Péters 27 days ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF