Skip to content

fix(shell): Make sidecars work in tests (fix #13767)#3234

Merged
FabianLars merged 3 commits intotauri-apps:v2from
alesanmed-forked-projects:v2
Jan 26, 2026
Merged

fix(shell): Make sidecars work in tests (fix #13767)#3234
FabianLars merged 3 commits intotauri-apps:v2from
alesanmed-forked-projects:v2

Conversation

@alesanmed
Copy link
Contributor

@alesanmed alesanmed commented Jan 24, 2026

The work in this PR is based on this one.

When running tests, using platform::current_exe().parent() alone is not sufficient, since the executable is located inside the deps directory. To fix this, we need to go up one level.

Fixes tauri-apps/tauri#13767

@alesanmed alesanmed requested a review from a team as a code owner January 24, 2026 13:31
@github-actions
Copy link
Contributor

github-actions bot commented Jan 24, 2026

Package Changes Through 8c47c2c

There are 10 changes which include updater with minor, updater-js with minor, barcode-scanner with patch, barcode-scanner-js with patch, http with patch, http-js with patch, sql with patch, sql-js with patch, shell with patch, shell-js with patch

Planned Package Versions

The following package releases are the planned based on the context of changes in this pull request.

package current next
api-example 2.0.40 2.0.41
api-example-js 2.0.36 2.0.37
barcode-scanner 2.4.3 2.4.4
barcode-scanner-js 2.4.3 2.4.4
http 2.5.6 2.5.7
http-js 2.5.6 2.5.7
shell 2.3.4 2.3.5
shell-js 2.3.4 2.3.5
sql 2.3.1 2.3.2
sql-js 2.3.1 2.3.2
updater 2.9.0 2.10.0
updater-js 2.9.0 2.10.0

Add another change file through the GitHub UI by following this link.


Read about change files or the docs at github.com/jbolda/covector

@alesanmed
Copy link
Contributor Author

cc @FabianLars just checking in in case you need anything else from my side. I’ve already fixed the tests, so I’m not sure whether there’s something else missing or if you simply haven’t had time to look at it yet. No rush at all!

@FabianLars
Copy link
Member

sorry, been very busy and will be for the next few weeks. will try to squeeze this in when i have a free minute.

ironically this change will likely break for users of cargo's new build-dir once we fix sidecars with that config - at least i assume it's currently broken like tauri-apps/tauri#14790

Copy link
Member

@FabianLars FabianLars left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

would you look at that, i have a free minute

@FabianLars FabianLars merged commit d4613ff into tauri-apps:v2 Jan 26, 2026
16 checks passed
@alesanmed
Copy link
Contributor Author

sorry, been very busy and will be for the next few weeks. will try to squeeze this in when i have a free minute.

ironically this change will likely break for users of cargo's new build-dir once we fix sidecars with that config - at least i assume it's currently broken like tauri-apps/tauri#14790

Hmm, I see the linked issue... If you think I could help with that, don't hesitate to ping me. I can't promise I'll be able to figure out a solution, but I can definitely try.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[bug] Can't test an app with sidecar. Getting "No such file or directory (os error 2)"

2 participants