Project

General

Profile

Actions
Copy link

Bug #17358

closed

Bug #17348: when importing a technique with /api/internal/techniques, the order of parameters is not respected

ncf method write test in error

Added by François ARMAND almost 4 years ago. Updated almost 4 years ago.

Status:
Released
Priority:
N/A
Assignee:
Vincent MEMBRÉ
Category:
Web - Config management
Target version:
6.1.0~beta3
Pull Request:
https://github.com/Normation/rudder/p...
Severity:
UX impact:
User visibility:
Effort required:
Priority:
0
Name check:
To do
Fix check:
To do
Regression:

Description

Tests report:

Tests run: 16, Failures: 2, Errors: 0, Skipped: 0, Time elapsed: 0.201 sec <<< FAILURE!
Preparing files for technique Test Technique created through Rudder API should::Should generate expected metadata content for our technique(com.normation.rudder.ncf.TestTechniqueWriter)  Time elapsed: 0.004 sec  <<< FAILURE!
java.lang.Exception: /tmp/test-technique-writer-2020-05-07T18:39:42.119+02:00/techniques/ncf_techniques/technique_by_Rudder/1.0/metadata.xml is not the same as src/test/resources/configuration-repository/techniques/ncf_techniques/technique_by_Rudder/1.0/metadata.xml
....
    + 33.         <VALUE>(ParameterId(command),Write-Host &quot;testing special characters ` è &amp;é 'à é &quot;)</VALUE>
    - 33.         <VALUE>Write-Host &quot;testing special characters ` è &amp;é 'à é &quot;</VALUE>
    + 34.         <VALUE>(ParameterId(command),/bin/echo &quot;testing special characters ` è &amp;é 'à é &quot;\)</VALUE>
    - 34.         <VALUE>/bin/echo &quot;testing special characters ` è &amp;é 'à é &quot;\</VALUE>
      35.       </REPORTKEYS>

Likely a ._2 is missing on the (paramter, value)

Actions
Copy link
 #1

Updated by François ARMAND almost 4 years ago

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

Updated by François ARMAND almost 4 years 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/2961
Actions
Copy link
 #3

Updated by François ARMAND almost 4 years ago

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

Updated by Vincent MEMBRÉ almost 4 years ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 6.1.0~beta3 which was released today.

Actions
Copy link

Also available in: Atom PDF