Project

General

Profile

Actions
Copy link

Bug #24719

closed

MockLdapServer needs to be a class instantiated for each test

Bug #24719: MockLdapServer needs to be a class instantiated for each test

Added by François ARMAND almost 2 years ago. Updated over 1 year 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.

Updated by François ARMAND almost 2 years ago Actions
Copy link
 #1

  • Status changed from New to In progress
  • Assignee set to François ARMAND

Updated by François ARMAND almost 2 years ago Actions
Copy link
 #2

  • 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

Updated by Anonymous almost 2 years ago Actions
Copy link
 #3

  • Status changed from Pending technical review to Pending release

Updated by Alexis Mousset almost 2 years ago Actions
Copy link
 #4

  • Fix check changed from To do to Checked

Updated by Vincent MEMBRÉ over 1 year ago Actions
Copy link
 #5

  • 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: PDF Atom