Skip to content

test(e2e): a connection created from a folder lands in it (#1050) - #1265

Merged
ZhuchkaTriplesix merged 3 commits into
devfrom
issue/1050-folders
Oct 10, 2026
Merged

ZhuchkaTriplesix merged 3 commits into
devfrom
issue/1050-folders

Conversation

@ZhuchkaTriplesix

Copy link
Copy Markdown
Member

Summary

The folder scenario removed in c2ffcc8 failed for the same reason as the form scenarios (the type was never selected, see #1256). It is back, using the card tap:

  • right-click a folder → New connection → PostgreSQL → URI and name → Save;
  • the stored row carries the folder's id.

Not covered: moving a connection into a folder (drag and drop) and removing a folder; the first needs a drag gesture on the tree, the second a decision on what happens to its connections. #1050 stays open for them.

Refs #1050

Test plan

  • CI: the e2e suite passes (no local runs, per team rule)

…1050)

Right-click on a folder, New connection, pick the type by its card, type a
URI and a name, Save: the stored row carries the folder's id.
@github-actions github-actions Bot added tests Theme parser epic core Core library logic and services connections Database connections, URI parsing, pools P3 Low priority / Polish & Enhancements labels Oct 10, 2026
On a first launch without connections the tour opens once its settings read
returns, and its scrim covered the folder the scenario right-clicks.
ShadcnApp defaults to a sheet on the platform the test binding reports, and a
sheet needs a DrawerOverlay: a right click on a folder asserted.
@ZhuchkaTriplesix
ZhuchkaTriplesix merged commit 7a479e7 into dev Oct 10, 2026
13 checks passed
@ZhuchkaTriplesix
ZhuchkaTriplesix deleted the issue/1050-folders branch October 11, 2026 15:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

connections Database connections, URI parsing, pools core Core library logic and services P3 Low priority / Polish & Enhancements tests Theme parser epic

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant