You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cordova.apache.org by Git at Apache <gi...@git.apache.org> on 2012/07/06 16:05:55 UTC
incubator-cordova-ios pull request: CB-382 - LocalStorage Unit Tests

GitHub user agrieve opened a pull request:

    https://github.com/apache/incubator-cordova-ios/pull/26

    CB-382 - LocalStorage Unit Tests

    Let me know if you want this slice & diced differently. I haven't used github before and so wasn't sure if you want each commit in a different branch or if it's fine to have several commits in one pull request.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/agrieve/incubator-cordova-ios unittests

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-cordova-ios/pull/26.patch

----
commit 84353192dadb58aedb3b539825283a9da8c3c48b
Author: Andrew Grieve <ag...@chromium.org>
Date:   2012-07-05T08:23:34-07:00

    Add VERSION file to the resources of CordovaLibApp
    
    Required so that CDVViewController::cordovaVersion does not return nil.

commit b1ee0c3e8e8d95f5518a8d524926cfd9b3588854
Author: Andrew Grieve <ag...@chromium.org>
Date:   2012-07-05T09:13:27-07:00

    Remove unit test header files and simplify CDVWebViewTest.
    
    Unit tests are isolated, so there is no need to export their information
    in headers. Removing them simplifies things a bit.
    
    CDVWebViewTest was using reflection unnecessarily. Simplified it with
    direct selectors.

commit 14592e7418b71d325d4efbd83aff17b2feb2fb82
Author: Andrew Grieve <ag...@chromium.org>
Date:   2012-07-06T06:23:54-07:00

    Adds a dispose method to CDVViewController.
    
    This method attempts to break retain cycles. We may want to consider
    breaking these cycles using weak references in the future. This function
    will be used by unit tests.

commit 83ce8ae4ae0a8973ddfe8367091adc5241592249
Author: Andrew Grieve <ag...@chromium.org>
Date:   2012-07-06T06:27:52-07:00

    Adds a way for CordovaLibApp to recreate the ViewController.
    
    This does not change the behaviour of CordovaLibApp when run directly,
    but causes the root ViewController to not be created automatically when
    running the unit tests. The intention here is that unit tests that
    require a full ViewController will use the methods to create it and
    destroy it afterwards.

commit 69eca0c1084f0c65021927a171895ec390313443
Author: Andrew Grieve <ag...@chromium.org>
Date:   2012-07-06T06:30:51-07:00

    Refactor of CDVWebViewTest and partial implementation of CDVLocalStorageTests
    
    Thes CDVWebViewTest base class will now wait for the WebView to be fully
    loaded before returning a reference to it. It does this by running the
    run loop until a JS variable set at the bottom of index.html reports
    being set.

commit 1504202551de207a8b8b88466133b0b1125d669e
Author: Andrew Grieve <ag...@chromium.org>
Date:   2012-07-06T06:56:36-07:00

    Don't use @autoreleasepool.
    
    Doing so causes a runtime error in the 4.3 simulator.

----