-
Notifications
You must be signed in to change notification settings - Fork 159
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
E2e tests. implement tests for searching #7395
Comments
@ScharfViktor for character limit of search length simplescreenrecorder-2022-09-02_16.56.36.mp4Also, I found a weird behavior while searching for a folder and navigating to it through the displayed search list
simplescreenrecorder-2022-09-02_16.36.32.mp4
simplescreenrecorder-2022-09-02_16.53.35.mp4Now Seems like a bug |
There is existing bug: #7391 |
I tested it (works correct for project and personal space).try test again after merging owncloud/ocis#4485 |
search by project space name: please skip this step owncloud/ocis#4506 |
@ScharfViktor I found a weird behavior
The test also fails here for the same reason https://drone.owncloud.com/owncloud/web/28072/12/13 passes in |
yes weird, after I search in all files the issue is not reproducible at all |
I created an issue here owncloud/ocis#4517 |
@ScharfViktor as for |
created an issue for public link searching #7603 |
I think it it normal behaviors |
okay |
@ScharfViktor for shares jail is it enough with what is done in PR #7583 or do we need to add more and for |
I think it's enough for e2e tests. We have api tests with limit checking |
Okay I'll continue on adding steps for |
@ScharfViktor #7603 this issue has been fixed by #7849 which hides the search bar in the public link. So what do we do about the |
I think we should remove this case from issue as |
need to implement:
cannot be implemented
see Hide search bar in public link context #7849 search for public (public link) (after this issue is fixed Searching in public link takes you to the personal of the creator of the public link #7603)cases:
pending or declined share(API tests added, no need for e2e - for context E2e tests. implement tests for searching #7395 (comment))deleted files.(API tests added, no need for e2e - for context E2e tests. implement tests for searching #7395 (comment))check limit- 5 and morechecked in api testsThe text was updated successfully, but these errors were encountered: