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

Move tests from /src/extensions/ to /src/tests #1221

Closed
latenitefilms opened this Issue May 11, 2018 · 1 comment

Comments

Projects
None yet
2 participants
@latenitefilms
Contributor

latenitefilms commented May 11, 2018

Currently the tests are stored within the /src/extensions and /src/plugins folders, which get included in the public releases of CommandPost. We should move all the tests and test code to /src/tests, so that these files are not included in the compiled application bundle, and the tests can only be run when you're running CommandPost in "developer mode" with CommandPost loading the scripts from the GitHub source, rather than the application bundle.

@latenitefilms latenitefilms added this to the 1.0.0-beta.30 milestone May 11, 2018

@latenitefilms latenitefilms modified the milestones: 1.0.0-beta.32, 1.0.0-beta.30 Jul 10, 2018

@latenitefilms

This comment has been minimized.

Show comment
Hide comment
@latenitefilms

latenitefilms Jul 11, 2018

Contributor

@randomeizer - It's probably best if you tackle this one, so I don't break anything.

The main thing I really want to achieve with this issue is moving the Final Cut Pro test libraries outside of the application bundle, as the FCPX Cache file always breaks Xcode, and needs to be manually "cleaned".

Contributor

latenitefilms commented Jul 11, 2018

@randomeizer - It's probably best if you tackle this one, so I don't break anything.

The main thing I really want to achieve with this issue is moving the Final Cut Pro test libraries outside of the application bundle, as the FCPX Cache file always breaks Xcode, and needs to be manually "cleaned".

randomeizer added a commit that referenced this issue Jul 17, 2018

#1221
* Added global `_tests(...)` function when in developer mode, which will search in the `/src/tests/` folder for a test matching the pattern.

randomeizer added a commit that referenced this issue Jul 17, 2018

#1221
* Moved all tests and resources to `/src/tests`
* Now you run tests via `_test(...)`
* Run all tests with `_test()()`
* Run a specific test with `_test("cp.app")()`
* Run a specific test inside a test with `_test("cp.app")("UI")`

randomeizer added a commit that referenced this issue Jul 17, 2018

#1221
* Fixing sticker issues

randomeizer added a commit that referenced this issue Jul 17, 2018

#1221
* Tweak

randomeizer added a commit that referenced this issue Jul 17, 2018

#1221
* Fixed some incorrect tests.

latenitefilms added a commit that referenced this issue Jul 17, 2018

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