Skip to content

Commit 7353ce9

Browse files
committed
NC | Fix test_nc_upgrade_manager cleanup
Signed-off-by: Romy <35330373+romayalon@users.noreply.github.com>
1 parent 97084c7 commit 7353ce9

File tree

1 file changed

+8
-7
lines changed

1 file changed

+8
-7
lines changed

src/test/unit_tests/jest_tests/test_nc_upgrade_manager.test.js

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -334,7 +334,7 @@ describe('nc upgrade manager - upgrade config directory', () => {
334334

335335
describe('nc upgrade manager - _run_nc_upgrade_scripts', () => {
336336
const from_version = '0.0.0';
337-
const to_version = '0.0.9';
337+
const to_version = '0.0.9-test-nc-upgrade-manager';
338338
const custom_upgrade_scripts_dir = path.join(TMP_PATH, 'custom_upgrade_scripts_dir');
339339
const custom_upgrade_scripts_dir_version_path = path.join(TMP_PATH, 'custom_upgrade_scripts_dir', to_version);
340340
const default_upgrade_script_dir_version_path = path.join(DEFAULT_NC_UPGRADE_SCRIPTS_DIR, to_version);
@@ -352,11 +352,11 @@ describe('nc upgrade manager - upgrade config directory', () => {
352352
const nc_upgrade_manager_custom_dir = new NCUpgradeManager(config_fs, { custom_upgrade_scripts_dir });
353353

354354
beforeEach(async () => {
355-
await fs_utils.create_path(custom_upgrade_scripts_dir_version_path, 777);
355+
await fs_utils.create_path(custom_upgrade_scripts_dir_version_path, config.BASE_MODE_DIR);
356356

357357
try {
358-
await fs_utils.create_path(DEFAULT_NC_UPGRADE_SCRIPTS_DIR, 777);
359-
await fs_utils.create_path(default_upgrade_script_dir_version_path, 777);
358+
await fs_utils.create_path(DEFAULT_NC_UPGRADE_SCRIPTS_DIR, config.BASE_MODE_DIR);
359+
await fs_utils.create_path(default_upgrade_script_dir_version_path, config.BASE_MODE_DIR);
360360
} catch (err) {
361361
console.log('couldnt create upgrade scripts dir err', default_upgrade_script_dir_version_path, err);
362362
}
@@ -365,6 +365,7 @@ describe('nc upgrade manager - upgrade config directory', () => {
365365
afterEach(async () => {
366366
await fs_utils.folder_delete(custom_upgrade_scripts_dir_version_path);
367367
await fs_utils.folder_delete(custom_upgrade_scripts_dir);
368+
await fs_utils.folder_delete(default_upgrade_script_dir_version_path);
368369
});
369370

370371
it('_run_nc_upgrade_scripts - no scripts', async () => {
@@ -436,7 +437,7 @@ describe('nc upgrade manager - upgrade config directory', () => {
436437
const system_data = old_expected_system_json;
437438
const options = {
438439
config_dir_from_version: '0.0.0',
439-
config_dir_to_version: '0.0.9',
440+
config_dir_to_version: '0.0.9-test-nc-upgrade-manager',
440441
package_from_version: '5.16.0',
441442
package_to_version: '5.17.0',
442443
};
@@ -538,7 +539,7 @@ describe('nc upgrade manager - upgrade config directory', () => {
538539
'running_host': hostname,
539540
'completed_scripts': [],
540541
'config_dir_from_version': '0.0.0',
541-
'config_dir_to_version': '0.0.9',
542+
'config_dir_to_version': '0.0.9-test-nc-upgrade-manager',
542543
'package_from_version': '5.17.0',
543544
'package_to_version': '5.18.0',
544545
'error': new Error('this is a last failure error').stack
@@ -633,7 +634,7 @@ describe('nc upgrade manager - upgrade config directory', () => {
633634
'running_host': hostname,
634635
'completed_scripts': [],
635636
'config_dir_from_version': '0.0.0',
636-
'config_dir_to_version': '0.0.9',
637+
'config_dir_to_version': '0.0.9-test-nc-upgrade-manager',
637638
'package_from_version': '5.17.0',
638639
'package_to_version': '5.18.0',
639640
'error': new Error('this is a last failure error').stack

0 commit comments

Comments
 (0)