Project

General

Profile

Actions

Bug #8638

closed

file_ensure_lines_present_in_ini_section.cf test fails with CFEngine 3.9

Bug #8638: file_ensure_lines_present_in_ini_section.cf test fails with CFEngine 3.9

Added by Alexis Mousset over 9 years ago. Updated over 3 years ago.

Status:
Released
Priority:
N/A
Category:
Generic methods
Target version:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
0
Name check:
Fix check:
Regression:

Subtasks 1 (0 open1 closed)

Bug #9070: Conditionnal loading does not work as expectedReleasedBenoît PECCATTEActions

Related issues 1 (1 open0 closed)

Related to Rudder - Bug #18009: File in INI section method does not work as intendedPending technical reviewAlexis MoussetActions

Updated by Alexis Mousset over 9 years ago Actions #1

The problem is the second test, the difference in the verbose logs:

verbose: Additional promise info: version '1.0' source path '/root/ncf/tests/acceptance/../../tree/20_cfe_basics/ncf_lib.cf' at line 336 comment 'Insert line in section section_test2'
 verbose: The promised end pattern '\[.*\]\s*' was not found when selecting region to insert in '/root/ncf/tests/acceptance/workdir/__30_generic_methods_file_ensure_lines_present_in_ini_section_cf/tmp/test2.ini'

(only on 3.9)

Updated by Alexis Mousset over 9 years ago Actions #2

Maybe linked to:

EOF is  matched as an end of the region in edit_line
      promises only if 'select_end_match_eof' parameter is true.
      (Jira CFE-2263)

from 3.9 changelog.

Updated by Alexis Mousset over 9 years ago Actions #3

That's it! We cannot use select_end_match_eof though as it is not available in 3.7, we have to fix the test.

Updated by Alexis Mousset over 9 years ago Actions #5

  • Category set to Generic methods

Updated by Alexis Mousset over 9 years ago Actions #6

  • Status changed from New to In progress

Updated by Alexis Mousset over 9 years ago Actions #7

  • Status changed from In progress to Pending technical review
  • Assignee changed from Alexis Mousset to Nicolas CHARLES
  • Pull Request set to https://github.com/Normation/ncf/pull/438

Updated by Alexis Mousset over 9 years ago Actions #8

  • Status changed from Pending technical review to Pending release
  • % Done changed from 0 to 100

Updated by Alexis Mousset over 9 years ago Actions #9

  • Related to Bug #9070: Conditionnal loading does not work as expected added

Updated by Vincent MEMBRÉ over 9 years ago Actions #10

  • Related to deleted (Bug #9070: Conditionnal loading does not work as expected)

Updated by Vincent MEMBRÉ over 9 years ago Actions #11

  • Status changed from Pending release to Released

This bug has been fixed in ncf version that comes with Rudder 3.1.15/14 and 3.2.8/7 which were released today.

Updated by Félix DALLIDET over 5 years ago Actions #12

  • Related to Bug #18009: File in INI section method does not work as intended added

Updated by Alexis Mousset over 3 years ago Actions #13

  • Target version changed from 0.x to ncf-0.x
  • Priority set to 0

Updated by Alexis Mousset over 3 years ago Actions #14

  • Project changed from 41 to Rudder
  • Category changed from Generic methods to Generic methods
Actions

Also available in: PDF Atom