Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.5.0-beta1
    • Fix Version/s: 2.5.0-beta1
    • Component/s: Java
    • Labels:
      None

      Description

      Patch to support View level features and locales which is an approved proposal for OpenSocial 2.0.

      http://docs.opensocial.org/display/OSD/View+Level+Features+Proposal

      Patch includes updated JUnit tests and new EndToEnd test.

      http://codereview.appspot.com/4077043/

        Activity

        Hide
        Paul Lindner added a comment -

        part of 2.5.0-beta1 release.

        Show
        Paul Lindner added a comment - part of 2.5.0-beta1 release.
        Hide
        Paul Lindner added a comment -

        patch applied. Thanks!

        Show
        Paul Lindner added a comment - patch applied. Thanks!
        Hide
        Matthew Marum added a comment -

        I've made some changes, tests now pass on IBM and Sun 1.6 JREs.

        Show
        Matthew Marum added a comment - I've made some changes, tests now pass on IBM and Sun 1.6 JREs.
        Hide
        Paul Lindner added a comment -

        still failing for me on mac and linux, jdk 1.6 I think you just need to adjust your mocks for some other code that changed underneath you..

        Show
        Paul Lindner added a comment - still failing for me on mac and linux, jdk 1.6 I think you just need to adjust your mocks for some other code that changed underneath you..
        Hide
        Matthew Marum added a comment -
        Show
        Matthew Marum added a comment - Latest patch from http://codereview.appspot.com/4077043/
        Hide
        Matthew Marum added a comment -

        Unfortunately no, all the test pass in my environment. Those "getFeatureResources" calls are setup in the expectFeatureCalls method of that test.

        Are you taking the latest patch set from the code review tool or the one from here? The one attached here is old. I'll update it just in case.

        Show
        Matthew Marum added a comment - Unfortunately no, all the test pass in my environment. Those "getFeatureResources" calls are setup in the expectFeatureCalls method of that test. Are you taking the latest patch set from the code review tool or the one from here? The one attached here is old. I'll update it just in case.
        Hide
        Paul Lindner added a comment -

        Any ideas about these test failures when I apply this patch?

        Tests in error:
        unsupportedFeatureThrows(org.apache.shindig.gadgets.render.RenderingGadgetRewriterTest): Unexpected exception, expected<org.apache.shindig.gadgets.rewrite.RewritingException> but was<java.lang.AssertionError>
        unsupportedViewFeatureThrows(org.apache.shindig.gadgets.render.RenderingGadgetRewriterTest): Unexpected exception, expected<org.apache.shindig.gadgets.rewrite.RewritingException> but was<java.lang.AssertionError>


        Caused by: java.lang.AssertionError:
        Unexpected method call getFeatureResources(org.apache.shindig.gadgets.render.RenderingGadgetRewriterTest$MapGadgetContext@4ded4d06, [core, foo], []):
        getFeatureResources(same(org.apache.shindig.gadgets.render.RenderingGadgetRewriterTest$MapGadgetContext@4ded4d06), [foo, core], []): expected: 1, actual: 0
        at org.easymock.internal.MockInvocationHandler.invoke(MockInvocationHandler.java:45)
        at org.easymock.internal.ObjectMethodsFilter.invoke(ObjectMethodsFilter.java:73)
        at org.easymock.internal.ClassProxyFactory$MockMethodInterceptor.intercept(ClassProxyFactory

        Caused by: java.lang.AssertionError:
        Unexpected method call getFeatureResources(org.apache.shindig.gadgets.render.RenderingGadgetRewriterTest$MapGadgetContext@770fba26, [core, foo], []):
        getFeatureResources(same(org.apache.shindig.gadgets.render.RenderingGadgetRewriterTest$MapGadgetContext@770fba26), [foo, core], []): expected: 1, actual: 0
        at org.easymock.internal.MockInvocationHandler.invoke(MockInvocationHandler.java:45)
        at org.easymock.internal.ObjectMethodsFilter.invoke(ObjectMethodsFilter.java:73)
        at org.easymock.internal.ClassProxyFactory$MockMethodInterceptor.intercept(ClassProxyFactory.java:92)
        at org.apache.shindig.gadgets.features.FeatureRegistry$$EnhancerByCGLIB$$f7357840.getFeatureResources(<generated>)

        Show
        Paul Lindner added a comment - Any ideas about these test failures when I apply this patch? Tests in error: unsupportedFeatureThrows(org.apache.shindig.gadgets.render.RenderingGadgetRewriterTest): Unexpected exception, expected<org.apache.shindig.gadgets.rewrite.RewritingException> but was<java.lang.AssertionError> unsupportedViewFeatureThrows(org.apache.shindig.gadgets.render.RenderingGadgetRewriterTest): Unexpected exception, expected<org.apache.shindig.gadgets.rewrite.RewritingException> but was<java.lang.AssertionError> Caused by: java.lang.AssertionError: Unexpected method call getFeatureResources(org.apache.shindig.gadgets.render.RenderingGadgetRewriterTest$MapGadgetContext@4ded4d06, [core, foo] , []): getFeatureResources(same(org.apache.shindig.gadgets.render.RenderingGadgetRewriterTest$MapGadgetContext@4ded4d06), [foo, core] , []): expected: 1, actual: 0 at org.easymock.internal.MockInvocationHandler.invoke(MockInvocationHandler.java:45) at org.easymock.internal.ObjectMethodsFilter.invoke(ObjectMethodsFilter.java:73) at org.easymock.internal.ClassProxyFactory$MockMethodInterceptor.intercept(ClassProxyFactory Caused by: java.lang.AssertionError: Unexpected method call getFeatureResources(org.apache.shindig.gadgets.render.RenderingGadgetRewriterTest$MapGadgetContext@770fba26, [core, foo] , []): getFeatureResources(same(org.apache.shindig.gadgets.render.RenderingGadgetRewriterTest$MapGadgetContext@770fba26), [foo, core] , []): expected: 1, actual: 0 at org.easymock.internal.MockInvocationHandler.invoke(MockInvocationHandler.java:45) at org.easymock.internal.ObjectMethodsFilter.invoke(ObjectMethodsFilter.java:73) at org.easymock.internal.ClassProxyFactory$MockMethodInterceptor.intercept(ClassProxyFactory.java:92) at org.apache.shindig.gadgets.features.FeatureRegistry$$EnhancerByCGLIB$$f7357840.getFeatureResources(<generated>)
        Hide
        Matthew Marum added a comment -

        Patch code review on appspot.

        http://codereview.appspot.com/4077043/

        Show
        Matthew Marum added a comment - Patch code review on appspot. http://codereview.appspot.com/4077043/
        Hide
        Matthew Marum added a comment -

        Updated patch to fix some code formatting.

        Show
        Matthew Marum added a comment - Updated patch to fix some code formatting.
        Hide
        Matthew Marum added a comment -

        Patch for View level features/locales

        Show
        Matthew Marum added a comment - Patch for View level features/locales

          People

          • Assignee:
            Unassigned
            Reporter:
            Matthew Marum
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development