add acl-connect component integration test

[ldt1996]

closes #364

[socket-security]

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff	Package	Supply Chain Security	Vulnerability	Quality	Maintenance	License
	@​harperdb/​acl-connect@​1.0.10

View full report

[socket-security]

Warning

Review the following alerts detected in dependencies.

According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.

Action	Severity	Alert  (click "▶" to expand/collapse)
Warn		Publisher changed: npm @harperdb/acl-connect is now published by ethan_arrowood Author: ethan_arrowood From: integrationTests/components/fixtures/acl-connect-with-sys/package.json → npm/@harperdb/acl-connect@1.0.10 ℹ Read more on: This package | This alert | What is unstable ownership? Next steps: Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support@socket.dev. Suggestion: Try to reduce the number of authors you depend on to reduce the risk to malicious actors gaining access to your supply chain. Packages should remove inactive collaborators with publishing rights from packages on npm. Mark the package as acceptable risk. To ignore this alert only in this pull request, reply with the comment @SocketSecurity ignore npm/@harperdb/acl-connect@1.0.10. You can also ignore all packages with @SocketSecurity ignore-all. To ignore an alert for all future pull requests, use Socket's Dashboard to change the triage state of this alert.
Warn		Low adoption: npm @harperdb/acl-connect Location: Package overview From: integrationTests/components/fixtures/acl-connect-with-sys/package.json → npm/@harperdb/acl-connect@1.0.10 ℹ Read more on: This package | This alert | What are unpopular packages? Next steps: Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support@socket.dev. Suggestion: Unpopular packages may have less maintenance and contain other problems. Mark the package as acceptable risk. To ignore this alert only in this pull request, reply with the comment @SocketSecurity ignore npm/@harperdb/acl-connect@1.0.10. You can also ignore all packages with @SocketSecurity ignore-all. To ignore an alert for all future pull requests, use Socket's Dashboard to change the triage state of this alert.

View full report

[Ethan-Arrowood]

very nice! just some esm nits but good to go otherwise.

[Ethan-Arrowood]

Suggested change

	const require = createRequire(import.meta.url);
	const SETTINGS = require('./connect.json');
	import SETTINGS from './connect.json' with { type: 'json' };

https://nodejs.org/api/esm.html#json-modules

[Ethan-Arrowood]

Does import.meta.dirname work instead? Or are we not treating these tests as ESM 🤔

[cb1kenobi]

Looks good, though I agree with Ethan's findings.

[ldt1996]

thank you for your comments @Ethan-Arrowood , I agree with all of them, updated accodringly

[claude]

Reviewed; no blockers found.