Arvados

• Can you explain to me what's the intention of the "checks that Public favorites does not appear under shared with me" test case? I know it's a test from other past branch, but it doesn't make sense to me the way it's written, following its name I think it should create a public favorite, and then click on the "shared with me" entry and confirm the just created favorite isn't there, but instead is just asserting that some string ("Public favorites") isn't there.
• I think that re-using createSharedProjects() code isn't convenient, because it takes a lot of time to simulate a user sharing something many times, specially on a "Favorites" test suite. Could you instead create a similar function that's available to all the test suites (on cypress/support/commands.js) that uses API calls instead of interacting with the UI? I can help you with this if needed.
• Also, the current createSharedProjects() code could be used on a new sharing.spec.js file to test just once the sharing dialog, what do you think?
• There's one test case with the .only() call that avoids the rest to run.

New version first commit: https://dev.arvados.org/projects/arvados-workbench-2/repository/revisions/335fee64075faf3dac519fba45da5ecbe5008f33
Test run: developer-tests-workbench2: #305
Branch: 17436-Favorites-in-workflow-picker-dialog-is-different-to-favorite-list

Code cleanup

Some more testing related comments:

• I think the Cypress command createSharedProjects() is too specific to be in the commands.js file. It even has side effects that make its use difficult to follow (setting predefined global variables that need to be accessed later). Do you think it would be better to just make a createSharedProject() (singular) command that takes an optional name and just returns the created+shared project, leaving all the small details of how many, and to which global var will be assigned, to its caller?
• I think that testing the UI interaction for sharing is useful, but testing it many times doesn't have additional benefits and OTOH it adds test running time. The code that tested sharing interactions is now replaced with API calls completely, so now we aren't testing the sharing UI. I think it should be re-added to its own sharing.spec.js file or similar, do you agree?
• The cy.loginAs() calls at file cypress/integration/favorites.spec.js lines 49, 102 and 140 are not needed and add testing time.

New version first commit: https://dev.arvados.org/projects/arvados-workbench-2/repository/revisions/d2817690f88befb0c306e637d6e87367016c76e3
Test run: developer-tests-workbench2: #309
Branch: 17436-Favorites-in-workflow-picker-dialog-is-different-to-favorite-list

Added generic code for project creation within cypress tests

This LGTM, thanks!

New version first commit: https://dev.arvados.org/projects/arvados-workbench-2/repository/revisions/a45b5f6f12e4e87c3ff0ce6f79a4999fef6e8bc5
Test run: developer-tests-workbench2: #3 ​68/
Branch: 17436-Favorites-in-workflow-picker-dialog-is-different-to-favorites

Added additional test for directory picker

New version first commit: https://dev.arvados.org/projects/arvados-workbench-2/repository/revisions/2b3fa73903f8f485df9783f16f00ad05098f9270
Test run: developer-tests-workbench2: #3 ​69/
Branch: 17436-Favorites-in-workflow-picker-dialog-is-different-to-favorites

Removed 'only' from test suite