jcberquist / sublimetext-cfml

CFML (ColdFusion and Lucee) package for Sublime Text
MIT License
115 stars 24 forks source link

[Feature Request] Use box.json to pick up testbox settings #54

Closed cybersonic closed 7 years ago

cybersonic commented 7 years ago

At the moment we use the settings either from the user package settings of project settings file. If there already is a box.json file the settings for running testbox should already be there.

The box.json file would have

    [
        ...
        "testbox"           : {
            "runner" : [
                { "cf10"    : "http://cf10cboxdev.jfetmac/testbox/tests/runner.cfm" },
                { "cf11"    : "http://cf11cboxdev.jfetmac/testbox/tests/runner.cfm" },
                { "lucee"   : "http://luceecboxdev.jfetmac/testbox/tests/runner.cfm" }
            ]
        }
    ]

Which should be enough to at least run the tests. We could also add any more info to that file no?

jcberquist commented 7 years ago

This sounds interesting. How do you envision the box.json file being discovered? Searching a project's root folders? Specifying it in the project file?

cybersonic commented 7 years ago

I would just search the root folders

cybersonic commented 7 years ago

It should be in the root folder. I think that should be enough?

At most it could be in a child folder but I would just use the root. (Until someone complains) :)

Mark Drew

On 24 Oct 2016, at 03:39, jcberquist notifications@github.com wrote:

This sounds interesting. How do you envision the box.json file being discovered? Searching a project's root folders? Specifying it in the project file?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

jcberquist commented 7 years ago

This certainly seems doable. I guess if using the box.json settings, I should add reporter=JSON to the url if it isn't there?

jcberquist commented 7 years ago

I think this should be supported now, but please let me know if you find issues with it.