Keep removal regression assertions aligned with explicit NAS ownership

Require the actual NAS quiescence/retirement caller rather than the removed best-effort autorun cleanup. Preserve Upstart exclusion from SysV while requiring the NAS exclusion too. No manager or process safety control is removed.

Change-source: pulse-maintainer
This commit is contained in:
pulse-triage[bot]
2026-10-12 20:06:06 +01:00
parent 4233fc9e56
commit d87785172e
2 changed files with 4 additions and 2 deletions
@@ -424,7 +424,7 @@ remove_safe_profile_state_dir() { builtin command rm -r -- "$SAFE_PROFILE_STATE_
self.assertLess(admission, remote)
self.assertLess(remote, retirement)
self.assertLess(retirement, erase)
self.assertIn('[[ "$uninstall_upstart" != true ]]', entry)
self.assertIn('[[ "$uninstall_upstart" != true && -z "$uninstall_nas_platform" ]]', entry)
self.assertFalse('initctl stop' in entry)
self.assertNotIn('initctl start', definitions(self.source))
+3 -1
View File
@@ -1830,7 +1830,9 @@ func TestInstallSHUsesQNAPStateForUninstallRecovery(t *testing.T) {
`qnap_state_dir=$(find_qnap_state_dir || true)`,
`aid_paths+=("$qnap_state_dir/agent-id")`,
`if [[ -n "$qnap_state_dir" ]] && trusted_connection_state_file "$qnap_state_dir/connection.env"; then`,
`remove_qnap_autorun_block "$AUTORUN_PATH"`,
`uninstall_nas_platform=$(nas_agent_removal_platform || true)`,
`teardown_nas_agent_boot_owner "$uninstall_nas_platform"`,
`nas_agent_boot_operation "$platform" observe`,
}
for _, needle := range required {
if !strings.Contains(script, needle) {