[lorenzocorallo]

TestInstalledParsesEntries fails on any Linux machine where the agent is installed from the package:

linux_test.go:18: installed with no entries

The test points XDG_CONFIG_HOME at a temporary directory, but Installed() also reads the package's unit (/usr/lib/systemd/user/polinetwork-esign.service) and its global enablement (/etc/systemd/user/default.target.wants/…). With the package installed, it finds them and reports the agent as installed before the test has written anything. The code is right; the test was not isolated from the machine.

• isolate(t) points the user config directory and the package unit paths (systemUnits, globalEnabled) at temporary directories, and restores them after the test.
• New TestInstalledPackageUnit: the package unit counts as installed only once enabled for every user.

Test-only change. go test ./... passes on a machine with the agent installed.

[coderabbitai]

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 29 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

• Configuration used: Organization UI
• Review profile: CHILL
• Plan: Advanced
• Run ID: 751cfa3a-a74a-4324-9d1e-bee6c4cce524

📥 Commits

Reviewing files that changed from the base of the PR and between 9bfb731 and e5aa1a0.

📒 Files selected for processing: 1

• internal/autostart/linux_test.go

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}