Project

General

Profile

Actions
Copy link

Bug #24719

closed

MockLdapServer needs to be a class instantiated for each test

Added by François ARMAND about 1 month ago. Updated 20 days ago.

Status:
Released
Priority:
N/A
Assignee:
Vincent MEMBRÉ
Category:
Architecture - Code maintenance
Target version:
8.1.1
Pull Request:
https://github.com/Normation/rudder/p...
Severity:
UX impact:
User visibility:
Effort required:
Priority:
0
Name check:
To do
Fix check:
Checked
Regression:
No

Description

In 8.1, we have a real LDAP server allowing to do real LDAP tests for nodes / NodeFactRepository.

That class was created as an object, which means that all tests share the same state. Depending on the order of tests, state is different, which creates surprises and inconsistencies.

Actions
Copy link

Also available in: Atom PDF