-
Notifications
You must be signed in to change notification settings - Fork 56
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Stop running some nativeapp tests on v1 projects and ws command (#1699)
Since we now convert v1 definitions to v2 in-memory when running `snow app` command, we don't need to run all integration tests for v1 example projects anymore. Also, since the status of the `snow ws` command is uncertain (`snow app` and `snow ws` now call the same code under the hood), we're removing the `snow ws` parametrization from most tests. All in all, this reduces the Native App integration test suite from 300 unique tests to 128, which greatly speeds it up (from about 35 minutes to 19 minutes). For safety, we're keeping full parametrization on a simple test for each feature, to make sure that the v1 to v2 conversion and `snow ws` command don't regress.
- Loading branch information
1 parent
8aa1eef
commit cc9cba5
Showing
12 changed files
with
95 additions
and
574 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.