Quick note: I added a node runner for our unit tests that will run for every build:
https://github.com/callback/callback-js/blob/master/test/runner.js#L23-58 If you have already been developing cordova.js you will need to run ./configure again to pull down jsDom to build. The intent is that these tests will test the javascript code (mocking interactions to native) and should be kept green. This is not a replacement for cordova-test but a place for us to quickly spec and test our javascript without the need to load or build into a simulator. to run the tests in node: jake or jake test to run the tests in the browser: jake btest Gord On Wed, Feb 15, 2012 at 1:21 PM, Filip Maj <[email protected]> wrote: > Ahh, so more of a packaging question. I'm not sure. I know Steve is > working on a packaging script so certainly integrating a cordova-js build > into that makes sense. > > I think you're on the right track, Drew. We used to include phonegap.js > into the sample app packaged in each repo - maybe we can keep with that > same theme for cordova-js and just add some stuff into the README > regarding where to get/build the latest cordova-js. > > On 12-02-15 9:28 AM, "Drew Walters" <[email protected]> wrote: > > >I think the way I stated my question is confusing. Let me try again. > >Right now the incubator-cordova-blackberry-webworks repo contains both > >the JS and native code. Some time soon I would anticipate a > >incubator-cordova-js repo. At that time the JS files in the webworks > >repo should go away. > > > >It seems to me that the "vendor" in of a platform specific cordova.js > >should happen when we build the end user package (ie. the one > >currently at phonegap.com). At development time, I'm always going to > >want the latest one out of incubator-cordova-js so I can just build > >that myself. Unless of course there is some mechanism to > >automatically build incubator-cordova-js when it is updated and copy > >the built files to the platform repos. > > > >Am I wrong? > > > >On Wed, Feb 15, 2012 at 11:15 AM, Filip Maj <[email protected]> wrote: > >> I think it'd be a pain to submodule cordova-js into the platform repos. > >> > >> So far the general agreement is to "vendor" cordova-js in as a built > >> script. > >> > >> It should only get out of sync if we let it :) > >> > >> On 12-02-15 9:11 AM, "Drew Walters" <[email protected]> wrote: > >> > >>>I need to do some more testing before I agree to move it to the apache > >>>code. Wanted to get the name change in completely first. Right now > >>>some of the api aren't working in my test app. > >>> > >>>This brings up a point though. What will this look like once its > >>>moved to apache. I would assume we'll have a separate repo for > >>>cordova-js, but do the individual platforms really need a copy of the > >>>built js file? From an end user perspective this could just be > >>>handled at packaging time when we build the distributable binary. > >>>Seems like including a built js from cordova-js would lead to it > >>>getting out of sync. > >>> > >>>On Wed, Feb 15, 2012 at 11:03 AM, Gord Tanner <[email protected]> > >>>wrote: > >>>> OMFG! > >>>> > >>>> You just saved my day! > >>>> > >>>> I have been looking for why blackberry wasn't firing device ready and > >>>>that > >>>> bad module ID was it! I was circling around the channel code > >>>>attempting to > >>>> figure out why it wasn't joining after the device and network channels > >>>>were > >>>> fired. > >>>> > >>>> Gonna look into this and pull it in, looks good at first glance. > >>>> > >>>> > >>>> Gord > >>>> > >>>> On Wed, Feb 15, 2012 at 11:53 AM, Drew Walters <[email protected]> > >>>>wrote: > >>>> > >>>>> Ok, I've submitted a pull request with my changes. > >>>>> > >>>>> https://github.com/callback/callback-js/pull/17 > >>>>> > >>>>> In my previous email, #3 was due to where I had inserted #2 (oops). > >>>>> The changes are mostly phonegap -> cordova changes. Additionally I > >>>>> had to fix a module require in lib/platform/blackberry.js which was > >>>>> specifying a bad module name. > >>>>> > >>>>> The only appearance of phonegap in common-js after my change is in > >>>>>the > >>>>> iOS and WP7 plugins where they reference native code since I didn't > >>>>> think those had been renamed yet. > >>>>> > >>>>> Eventually the BlackBerry native side should make use of require (as > >>>>> shown in Fil's commit), but that will wait till common-js gets added > >>>>> to apache source. For the time being a quick hack is to add the > >>>>> following at the bottom of the generated js: > >>>>> > >>>>> var Cordova = require('cordova'); > >>>>> > >>>>> On Wed, Feb 15, 2012 at 9:23 AM, Drew Walters <[email protected]> > >>>>>wrote: > >>>>> > So I've got callback-js "working" in my sandbox. Haven't submitted > >>>>> > pull request yet because I'm still debugging something. Here's > >>>>> > summary of changes I've made: > >>>>> > > >>>>> > 1. Modify rest of files to use "cordova" instead of "phonegap". > >>>>> > 2. Added a global define of "Cordova" as a temporary hack since > >>>>>native > >>>>> > PluginResult.toSuccessCallbackString expects it to exist. > >>>>> > 3. For some reason I had to make sure the cordova/utils module > >>>>>exists > >>>>> > in cordova.js before the cordova/channel module, otherwise I was > >>>>> > getting an error that the module didn't exist. > >>>>> > > >>>>> > With the above changes I was able to run some simple tests > >>>>>(verified > >>>>> > network works). I'm currently looking at proper fix for #2 and #3. > >>>>> > > >>>>> > On Wed, Feb 15, 2012 at 8:21 AM, Drew Walters <[email protected]> > >>>>> wrote: > >>>>> >> I'm looking at https://github.com/callback/callback-js and I see > a > >>>>>lot > >>>>> >> of references to "phonegap" in the lib/plugins directory though it > >>>>> >> seems like things one directory up (lib) has been changed. Am I > >>>>>not > >>>>> >> looking at an up to date project or should I send a pull request > >>>>>with > >>>>> >> that name change too? > >>>>> >> > >>>>> >> On Wed, Feb 15, 2012 at 8:17 AM, Gord Tanner <[email protected] > > > >>>>> wrote: > >>>>> >>> I am on BlackBerry today. > >>>>> >>> > >>>>> >>> Currently showing: > >>>>> >>> Error: SyntaxError: Unable to parse JSON string alert. > >>>>> >>> > >>>>> >>> I will attempt to get it up and running again today. > >>>>> >>> > >>>>> >>> I also have the start of a node runner for callback-js tests > >>>>>here: > >>>>> >>> https://github.com/gtanner/callback-js > >>>>> >>> > >>>>> >>> Gord > >>>>> >>> > >>>>> >>> On Wed, Feb 15, 2012 at 5:19 AM, Viras > >>>>><[email protected]> > >>>>> wrote: > >>>>> >>> > >>>>> >>>> Can you please drop me a line once cordova-js is on the official > >>>>> repos & > >>>>> >>>> mirrored? I will then give integrating the new common-JS into > >>>>> cordova-qt a > >>>>> >>>> go! > >>>>> >>>> > >>>>> >>>> Am 2012-02-15 00:09, schrieb Filip Maj: > >>>>> >>>> > >>>>> >>>> Update on unified JavaScript work in cordova for today: > >>>>> >>>>> > >>>>> >>>>> - looks like the WP7 implementation cannot run our mobile-spec, > >>>>> which is a > >>>>> >>>>> hurdle in making sure it supports the cross-platform cordova > >>>>>API. > >>>>> Before > >>>>> >>>>> we can move forward with cordova-js integration we really need > >>>>>to > >>>>> figure > >>>>> >>>>> that part of it out. > >>>>> >>>>> - I will work rest of day in getting cordova-js into the apache > >>>>>git > >>>>> repo > >>>>> >>>>> and finishing up that work. > >>>>> >>>>> - BlackBerry still needs work and testing - Tim, you available? > >>>>> >>>>> - Shaz and I have a remote hack sesh planned for tomorrow > >>>>>morning to > >>>>> get > >>>>> >>>>> iOS on track. Will have a better idea how that platform is > >>>>>looking > >>>>> >>>>> tomorrow. > >>>>> >>>>> - Bada 2.0 still needs a platform implementation before we can > >>>>>move > >>>>> with > >>>>> >>>>> cordova-js on it. > >>>>> >>>>> > >>>>> >>>> > >>>>> >>>> -- > >>>>> >>>> GOFG - Get On Fat Guy > >>>>> >>>> http://www.gofg.at/ - powered by Cordova > >>>>> >>>> > >>>>> >>> > >>>>> >>> > >>>>> >>> > >>>>> >>> -- > >>>>> >>> Gord Tanner > >>>>> >>> Senior Developer / Code Poet > >>>>> >>> tinyHippos Inc. > >>>>> >>> @tinyhippos > >>>>> > >>>> > >>>> > >>>> > >>>> -- > >>>> Gord Tanner > >>>> Senior Developer / Code Poet > >>>> tinyHippos Inc. > >>>> @tinyhippos > >> > > -- Gord Tanner Senior Developer / Code Poet tinyHippos Inc. @tinyhippos
