Project

General

Profile

Actions
Copy link

Bug #24719

closed

MockLdapServer needs to be a class instantiated for each test

Added by François ARMAND 7 months ago. Updated 7 months 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
 #1

Updated by François ARMAND 7 months ago

  • Status changed from New to In progress
  • Assignee set to François ARMAND
Actions
Copy link
 #2

Updated by François ARMAND 7 months ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from François ARMAND to Vincent MEMBRÉ
  • Pull Request set to https://github.com/Normation/rudder/pull/5603
Actions
Copy link
 #3

Updated by Anonymous 7 months ago

  • Status changed from Pending technical review to Pending release
Actions
Copy link
 #4

Updated by Alexis Mousset 7 months ago

  • Fix check changed from To do to Checked
Actions
Copy link
 #5

Updated by Vincent MEMBRÉ 7 months ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 8.1.1 which was released today.

Actions
Copy link

Also available in: Atom PDF