Project

General

Profile

Bug #8638

file_ensure_lines_present_in_ini_section.cf test fails with CFEngine 3.9

Added by Alexis MOUSSET over 3 years ago. Updated over 3 years ago.

Status:
Released
Priority:
N/A
Category:
Generic methods
Target version:
Severity:
User visibility:
Effort required:
Priority:

Subtasks

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

Associated revisions

Revision 3652e3fe (diff)
Added by Alexis MOUSSET over 3 years ago

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

History

#1

Updated by Alexis MOUSSET over 3 years ago

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)

#2

Updated by Alexis MOUSSET over 3 years ago

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.

#3

Updated by Alexis MOUSSET over 3 years ago

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.

#5

Updated by Alexis MOUSSET over 3 years ago

  • Category set to Generic methods
#6

Updated by Alexis MOUSSET over 3 years ago

  • Status changed from New to In progress
#7

Updated by Alexis MOUSSET over 3 years ago

  • 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
#8

Updated by Alexis MOUSSET over 3 years ago

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

Updated by Alexis MOUSSET over 3 years ago

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

Updated by Vincent MEMBRÉ over 3 years ago

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

Updated by Vincent MEMBRÉ over 3 years ago

  • 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.

Also available in: Atom PDF