Author: lindner
Date: Thu Jan 22 12:21:38 2009
New Revision: 736777
URL: http://svn.apache.org/viewvc?rev=736777&view=rev
Log:
SHINDIG-560 | Modified Patch from Ian Boston | Add support for @supportedFields
Modified:
incubator/shindig/trunk/java/common/src/main/java/org/apache/shindig/auth/AuthenticationHandler.java
incubator/shindig/trunk/java/samples/src/test/java/org/apache/shindig/social/opensocial/jpa/spi/integration/JpaTestGuiceModule.java
incubator/shindig/trunk/java/server/src/test/java/org/apache/shindig/server/endtoend/EndToEndModule.java
incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/core/config/SocialApiGuiceModule.java
incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/core/oauth/OAuthConsumerRequestAuthenticationHandler.java
incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/opensocial/service/ActivityHandler.java
incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/opensocial/service/AppDataHandler.java
incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/opensocial/service/DataRequestHandler.java
incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/opensocial/service/PersonHandler.java
incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/opensocial/service/RestfulRequestItem.java
incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/sample/service/SampleContainerHandler.java
incubator/shindig/trunk/java/social-api/src/test/java/org/apache/shindig/social/SocialApiTestsGuiceModule.java
incubator/shindig/trunk/java/social-api/src/test/java/org/apache/shindig/social/opensocial/service/ActivityHandlerTest.java
incubator/shindig/trunk/java/social-api/src/test/java/org/apache/shindig/social/opensocial/service/AppDataHandlerTest.java
incubator/shindig/trunk/java/social-api/src/test/java/org/apache/shindig/social/opensocial/service/PersonHandlerTest.java
incubator/shindig/trunk/java/social-api/src/test/java/org/apache/shindig/social/opensocial/spi/DataRequestHandlerTest.java
Modified:
incubator/shindig/trunk/java/common/src/main/java/org/apache/shindig/auth/AuthenticationHandler.java
URL:
http://svn.apache.org/viewvc/incubator/shindig/trunk/java/common/src/main/java/org/apache/shindig/auth/AuthenticationHandler.java?rev=736777&r1=736776&r2=736777&view=diff
==============================================================================
---
incubator/shindig/trunk/java/common/src/main/java/org/apache/shindig/auth/AuthenticationHandler.java
(original)
+++
incubator/shindig/trunk/java/common/src/main/java/org/apache/shindig/auth/AuthenticationHandler.java
Thu Jan 22 12:21:38 2009
@@ -51,17 +51,17 @@
* @return Header value for a WWW-Authenticate Header
*/
String getWWWAuthenticateHeader(String realm);
-
+
/**
* An exception thrown by an AuthenticationHandler in the situation where
* a malformed credential or token is passed. A handler which throws this
exception
* is required to include the appropriate error state in the servlet response
*/
public static final class InvalidAuthenticationException extends
RuntimeException {
-
+
private Map<String,String> additionalHeaders;
private String redirect;
-
+
/**
* @param message Message to output in error response
* @param cause Underlying exception
@@ -69,11 +69,12 @@
public InvalidAuthenticationException(String message, Throwable cause) {
this(message, cause, null, null);
}
-
+
/**
* @param message Message to output in error response
- * @param additionalHeaders Headers to add to error response
* @param cause Underlying exception
+ * @param additionalHeaders Headers to add to error response
+ * @param redirect URL to redirect to on error
*/
public InvalidAuthenticationException(String message, Throwable cause,
Map<String,String> additionalHeaders, String redirect) {
@@ -81,11 +82,11 @@
this.additionalHeaders = additionalHeaders;
this.redirect = redirect;
}
-
+
public Map<String, String> getAdditionalHeaders() {
return additionalHeaders;
}
-
+
public String getRedirect() {
return redirect;
}
Modified:
incubator/shindig/trunk/java/samples/src/test/java/org/apache/shindig/social/opensocial/jpa/spi/integration/JpaTestGuiceModule.java
URL:
http://svn.apache.org/viewvc/incubator/shindig/trunk/java/samples/src/test/java/org/apache/shindig/social/opensocial/jpa/spi/integration/JpaTestGuiceModule.java?rev=736777&r1=736776&r2=736777&view=diff
==============================================================================
---
incubator/shindig/trunk/java/samples/src/test/java/org/apache/shindig/social/opensocial/jpa/spi/integration/JpaTestGuiceModule.java
(original)
+++
incubator/shindig/trunk/java/samples/src/test/java/org/apache/shindig/social/opensocial/jpa/spi/integration/JpaTestGuiceModule.java
Thu Jan 22 12:21:38 2009
@@ -18,6 +18,8 @@
package org.apache.shindig.social.opensocial.jpa.spi.integration;
+import org.apache.shindig.social.core.util.ContainerConf;
+import org.apache.shindig.social.core.util.JsonContainerConf;
import org.apache.shindig.social.opensocial.jpa.AccountDb;
import org.apache.shindig.social.opensocial.jpa.ActivityDb;
import org.apache.shindig.social.opensocial.jpa.AddressDb;
@@ -43,6 +45,7 @@
import org.apache.shindig.social.opensocial.model.Organization;
import org.apache.shindig.social.opensocial.model.Person;
import org.apache.shindig.social.opensocial.model.Url;
+import org.apache.shindig.social.opensocial.service.PersonHandler;
import org.apache.shindig.social.opensocial.spi.ActivityService;
import org.apache.shindig.social.opensocial.spi.AppDataService;
import org.apache.shindig.social.opensocial.spi.PersonService;
@@ -50,7 +53,10 @@
import javax.persistence.EntityManager;
import com.google.inject.AbstractModule;
+import com.google.inject.Inject;
+import com.google.inject.Provider;
import com.google.inject.Scopes;
+import com.google.inject.TypeLiteral;
/**
* Provides component injection for tests
@@ -60,6 +66,7 @@
*/
public class JpaTestGuiceModule extends AbstractModule {
+
private EntityManager entityManager;
JpaTestGuiceModule(EntityManager entityManager) {
@@ -91,5 +98,7 @@
this.bind(Organization.class).to(OrganizationDb.class);
this.bind(Person.class).to(PersonDb.class);
this.bind(Url.class).to(UrlDb.class);
+ this.bind(ContainerConf.class).to(JsonContainerConf.class);
+
}
}
Modified:
incubator/shindig/trunk/java/server/src/test/java/org/apache/shindig/server/endtoend/EndToEndModule.java
URL:
http://svn.apache.org/viewvc/incubator/shindig/trunk/java/server/src/test/java/org/apache/shindig/server/endtoend/EndToEndModule.java?rev=736777&r1=736776&r2=736777&view=diff
==============================================================================
---
incubator/shindig/trunk/java/server/src/test/java/org/apache/shindig/server/endtoend/EndToEndModule.java
(original)
+++
incubator/shindig/trunk/java/server/src/test/java/org/apache/shindig/server/endtoend/EndToEndModule.java
Thu Jan 22 12:21:38 2009
@@ -24,6 +24,8 @@
import org.apache.shindig.social.core.util.BeanJsonConverter;
import org.apache.shindig.social.core.util.BeanXStreamAtomConverter;
import org.apache.shindig.social.core.util.BeanXStreamConverter;
+import org.apache.shindig.social.core.util.ContainerConf;
+import org.apache.shindig.social.core.util.JsonContainerConf;
import org.apache.shindig.social.opensocial.service.BeanConverter;
import org.apache.shindig.social.opensocial.service.DataServiceServletFetcher;
@@ -58,5 +60,6 @@
bind(new TypeLiteral<List<AuthenticationHandler>>(){}).toProvider(
AuthenticationHandlerProvider.class);
+ bind(ContainerConf.class).to(JsonContainerConf.class);
}
}
Modified:
incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/core/config/SocialApiGuiceModule.java
URL:
http://svn.apache.org/viewvc/incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/core/config/SocialApiGuiceModule.java?rev=736777&r1=736776&r2=736777&view=diff
==============================================================================
---
incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/core/config/SocialApiGuiceModule.java
(original)
+++
incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/core/config/SocialApiGuiceModule.java
Thu Jan 22 12:21:38 2009
@@ -25,6 +25,8 @@
import org.apache.shindig.social.core.util.BeanJsonConverter;
import org.apache.shindig.social.core.util.BeanXStreamAtomConverter;
import org.apache.shindig.social.core.util.BeanXStreamConverter;
+import org.apache.shindig.social.core.util.ContainerConf;
+import org.apache.shindig.social.core.util.JsonContainerConf;
import org.apache.shindig.social.opensocial.service.BeanConverter;
import org.apache.shindig.social.opensocial.service.DataServiceServletFetcher;
import org.apache.shindig.social.opensocial.service.StandardHandlerDispatcher;
@@ -50,7 +52,7 @@
/** {...@inheritdoc} */
@Override
protected void configure() {
- bind(HandlerDispatcher.class).toProvider(HandlerDispatcherProvider.class);
+
bind(HandlerDispatcher.class).toProvider(HandlerDispatcherProvider.class);
bind(ParameterFetcher.class).annotatedWith(Names.named("DataServiceServlet"))
.to(DataServiceServletFetcher.class);
@@ -71,6 +73,7 @@
bind(new TypeLiteral<List<AuthenticationHandler>>(){}).toProvider(
AuthenticationHandlerProvider.class);
+ bind(ContainerConf.class).to(JsonContainerConf.class);
}
/**
Modified:
incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/core/oauth/OAuthConsumerRequestAuthenticationHandler.java
URL:
http://svn.apache.org/viewvc/incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/core/oauth/OAuthConsumerRequestAuthenticationHandler.java?rev=736777&r1=736776&r2=736777&view=diff
==============================================================================
---
incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/core/oauth/OAuthConsumerRequestAuthenticationHandler.java
(original)
+++
incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/core/oauth/OAuthConsumerRequestAuthenticationHandler.java
Thu Jan 22 12:21:38 2009
@@ -69,7 +69,7 @@
if (service.thirdPartyHasAccessToUser(requestMessage, containerKey,
userId)) {
return service.getSecurityToken(containerKey, userId);
} else {
- return null;
+ throw new InvalidAuthenticationException("Access for app not
allowed",null);
}
} catch (OAuthException oae) {
throw new InvalidAuthenticationException(oae.getMessage(), oae);
Modified:
incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/opensocial/service/ActivityHandler.java
URL:
http://svn.apache.org/viewvc/incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/opensocial/service/ActivityHandler.java?rev=736777&r1=736776&r2=736777&view=diff
==============================================================================
---
incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/opensocial/service/ActivityHandler.java
(original)
+++
incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/opensocial/service/ActivityHandler.java
Thu Jan 22 12:21:38 2009
@@ -17,10 +17,14 @@
*/
package org.apache.shindig.social.opensocial.service;
+import org.apache.shindig.social.core.util.ContainerConf;
import org.apache.shindig.social.opensocial.model.Activity;
import org.apache.shindig.social.opensocial.spi.ActivityService;
import org.apache.shindig.social.opensocial.spi.SocialSpiException;
import org.apache.shindig.social.opensocial.spi.UserId;
+import org.apache.shindig.common.ContainerConfigException;
+import org.apache.shindig.common.util.ImmediateFuture;
+import org.apache.shindig.social.opensocial.spi.RestfulCollection;
import org.apache.shindig.social.opensocial.spi.CollectionOptions;
import com.google.common.collect.Sets;
@@ -29,15 +33,23 @@
import java.util.List;
import java.util.Set;
import java.util.concurrent.Future;
+import java.util.logging.Level;
+import java.util.logging.Logger;
+import java.util.regex.Matcher;
+import java.util.regex.Pattern;
public class ActivityHandler extends DataRequestHandler {
+ private final static Logger logger =
Logger.getLogger(ActivityHandler.class.getName());
private final ActivityService service;
private static final String ACTIVITY_ID_PATH
= "/activities/{userId}+/{groupId}/{appId}/{activityId}+";
+ private static final String ACTIVITY_SUP_FIELDS_REGEX =
"/activities/@supportedFields";
+ private static Pattern activityPatternSupFields =
Pattern.compile(ACTIVITY_SUP_FIELDS_REGEX);
@Inject
- public ActivityHandler(ActivityService service) {
+ public ActivityHandler(ActivityService service, ContainerConf containerConf)
{
+ super(containerConf);
this.service = service;
}
@@ -104,6 +116,15 @@
@Override
protected Future<?> handleGet(RequestItem request)
throws SocialSpiException {
+ if (isValidSupportedFieldsRestCall(request)) {
+ List<String> activityFieldsList = null;
+ try {
+ activityFieldsList = containerConf.getActivityFieldsList();
+ } catch (ContainerConfigException e) {
+ logger.log(Level.SEVERE,"Not able to retrieve ActivityFields
from container.js",e);
+ }
+ return ImmediateFuture.newInstance(new
RestfulCollection<String>(activityFieldsList));
+ }
request.applyUrlTemplate(ACTIVITY_ID_PATH);
Set<UserId> userIds = request.getUsers();
@@ -134,5 +155,11 @@
// getSortBy(params), getFilterBy(params), getStartIndex(params),
getCount(params),
request.getFields(), options, request.getToken());
}
-
+
+ private boolean isValidSupportedFieldsRestCall(RequestItem request) {
+ String url = ((RestfulRequestItem)request).getUrl();
+ Matcher supFieldsMatcher = activityPatternSupFields.matcher(url);
+ boolean isValidSupFieldsUrl = supFieldsMatcher.matches();
+ return isValidSupFieldsUrl;
+ }
}
Modified:
incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/opensocial/service/AppDataHandler.java
URL:
http://svn.apache.org/viewvc/incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/opensocial/service/AppDataHandler.java?rev=736777&r1=736776&r2=736777&view=diff
==============================================================================
---
incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/opensocial/service/AppDataHandler.java
(original)
+++
incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/opensocial/service/AppDataHandler.java
Thu Jan 22 12:21:38 2009
@@ -18,6 +18,7 @@
package org.apache.shindig.social.opensocial.service;
import org.apache.shindig.social.ResponseError;
+import org.apache.shindig.social.core.util.ContainerConf;
import org.apache.shindig.social.opensocial.spi.AppDataService;
import org.apache.shindig.social.opensocial.spi.SocialSpiException;
import org.apache.shindig.social.opensocial.spi.UserId;
@@ -36,7 +37,8 @@
private static final String APP_DATA_PATH =
"/appdata/{userId}+/{groupId}/{appId}";
@Inject
- public AppDataHandler(AppDataService service) {
+ public AppDataHandler(AppDataService service, ContainerConf containerConf) {
+ super(containerConf);
this.service = service;
}
Modified:
incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/opensocial/service/DataRequestHandler.java
URL:
http://svn.apache.org/viewvc/incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/opensocial/service/DataRequestHandler.java?rev=736777&r1=736776&r2=736777&view=diff
==============================================================================
---
incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/opensocial/service/DataRequestHandler.java
(original)
+++
incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/opensocial/service/DataRequestHandler.java
Thu Jan 22 12:21:38 2009
@@ -20,6 +20,7 @@
import org.apache.shindig.common.util.ImmediateFuture;
import org.apache.shindig.social.ResponseError;
import org.apache.shindig.social.opensocial.spi.SocialSpiException;
+import org.apache.shindig.social.core.util.ContainerConf;
import com.google.common.collect.ImmutableSet;
@@ -33,7 +34,16 @@
private static final Set<String> CREATE_SYNONYMS = ImmutableSet.of("post",
"create");
private static final Set<String> UPDATE_SYNONYMS = ImmutableSet.of("put",
"update");
private static final Set<String> DELETE_SYNONYMS = ImmutableSet.of("delete");
-
+ protected ContainerConf containerConf;
+
+ /**
+ *
+ */
+
+ public DataRequestHandler(ContainerConf containerConf) {
+ this.containerConf = containerConf;
+ }
+
public Future<?> handleItem(RequestItem request) {
if (request.getOperation() == null) {
return ImmediateFuture.errorInstance(new
SocialSpiException(ResponseError.NOT_IMPLEMENTED,
Modified:
incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/opensocial/service/PersonHandler.java
URL:
http://svn.apache.org/viewvc/incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/opensocial/service/PersonHandler.java?rev=736777&r1=736776&r2=736777&view=diff
==============================================================================
---
incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/opensocial/service/PersonHandler.java
(original)
+++
incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/opensocial/service/PersonHandler.java
Thu Jan 22 12:21:38 2009
@@ -17,7 +17,10 @@
*/
package org.apache.shindig.social.opensocial.service;
+import org.apache.shindig.common.ContainerConfigException;
+import org.apache.shindig.common.util.ImmediateFuture;
import org.apache.shindig.social.ResponseError;
+import org.apache.shindig.social.core.util.ContainerConf;
import org.apache.shindig.social.opensocial.model.Person;
import org.apache.shindig.social.opensocial.spi.CollectionOptions;
import org.apache.shindig.social.opensocial.spi.GroupId;
@@ -27,19 +30,25 @@
import com.google.common.collect.Sets;
import com.google.inject.Inject;
-
import java.util.Set;
import java.util.concurrent.Future;
+import java.util.logging.Logger;
+import java.util.regex.Matcher;
+import java.util.regex.Pattern;
public class PersonHandler extends DataRequestHandler {
-
+ private final static Logger logger =
Logger.getLogger(PersonHandler.class.getName());
private final PersonService personService;
private static final String PEOPLE_PATH =
"/people/{userId}+/{groupId}/{personId}+";
-
+ private static final String PEOPLE_SUP_FIELDS_REGEX =
"/people/@supportedFields";
+ private static Pattern peoplePatternSupFields =
Pattern.compile(PEOPLE_SUP_FIELDS_REGEX);
+ private Object personFields = null;
+
@Inject
- public PersonHandler(PersonService personService) {
+ public PersonHandler(PersonService personService, ContainerConf
containerConf) {
+ super(containerConf);
this.personService = personService;
}
@@ -66,7 +75,20 @@
@Override
protected Future<?> handleGet(RequestItem request) throws SocialSpiException
{
request.applyUrlTemplate(PEOPLE_PATH);
-
+ if (request instanceof RestfulRequestItem) {
+ if (isValidSupportedFieldsRestCall(request)) {
+ if (personFields == null) {
+ logger.fine("personFieldsList is null");
+ try {
+ personFields = this.containerConf.getPersonFields();
+ } catch (ContainerConfigException e) {
+ throw new SocialSpiException(ResponseError.INTERNAL_ERROR,
+ "Error trying to Supported Person Fields from
container.js", e);
+ }
+ }
+ return ImmediateFuture.newInstance(personFields);
+ }
+ }
GroupId groupId = request.getGroup();
Set<String> optionalPersonId =
Sets.newLinkedHashSet(request.getListParameter("personId"));
Set<String> fields = request.getFields(Person.Field.DEFAULT_FIELDS);
@@ -106,4 +128,11 @@
// Every other case is a collection response.
return personService.getPeople(userIds, groupId, options, fields,
request.getToken());
}
+
+ private boolean isValidSupportedFieldsRestCall(RequestItem request) {
+ String url = ((RestfulRequestItem)request).getUrl();
+ Matcher supFieldsMatcher =peoplePatternSupFields.matcher(url);
+ boolean isValidSupFieldsUrl = supFieldsMatcher.matches();
+ return isValidSupFieldsUrl;
+ }
}
Modified:
incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/opensocial/service/RestfulRequestItem.java
URL:
http://svn.apache.org/viewvc/incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/opensocial/service/RestfulRequestItem.java?rev=736777&r1=736776&r2=736777&view=diff
==============================================================================
---
incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/opensocial/service/RestfulRequestItem.java
(original)
+++
incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/opensocial/service/RestfulRequestItem.java
Thu Jan 22 12:21:38 2009
@@ -244,4 +244,8 @@
}
return stringList;
}
+
+ public String getUrl() {
+ return this.url;
+ }
}
Modified:
incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/sample/service/SampleContainerHandler.java
URL:
http://svn.apache.org/viewvc/incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/sample/service/SampleContainerHandler.java?rev=736777&r1=736776&r2=736777&view=diff
==============================================================================
---
incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/sample/service/SampleContainerHandler.java
(original)
+++
incubator/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/sample/service/SampleContainerHandler.java
Thu Jan 22 12:21:38 2009
@@ -25,6 +25,7 @@
import org.apache.commons.httpclient.methods.GetMethod;
import org.apache.shindig.common.util.ImmediateFuture;
import org.apache.shindig.social.ResponseError;
+import org.apache.shindig.social.core.util.ContainerConf;
import org.apache.shindig.social.opensocial.service.DataRequestHandler;
import org.apache.shindig.social.opensocial.service.RequestItem;
import org.apache.shindig.social.opensocial.spi.SocialSpiException;
@@ -42,7 +43,8 @@
private static final String POST_PATH = "/samplecontainer/{type}/{doevil}";
@Inject
- public SampleContainerHandler(JsonDbOpensocialService dbService) {
+ public SampleContainerHandler(JsonDbOpensocialService dbService,
ContainerConf containerConf) {
+ super(containerConf);
this.service = dbService;
}
Modified:
incubator/shindig/trunk/java/social-api/src/test/java/org/apache/shindig/social/SocialApiTestsGuiceModule.java
URL:
http://svn.apache.org/viewvc/incubator/shindig/trunk/java/social-api/src/test/java/org/apache/shindig/social/SocialApiTestsGuiceModule.java?rev=736777&r1=736776&r2=736777&view=diff
==============================================================================
---
incubator/shindig/trunk/java/social-api/src/test/java/org/apache/shindig/social/SocialApiTestsGuiceModule.java
(original)
+++
incubator/shindig/trunk/java/social-api/src/test/java/org/apache/shindig/social/SocialApiTestsGuiceModule.java
Thu Jan 22 12:21:38 2009
@@ -21,6 +21,8 @@
import org.apache.shindig.common.servlet.ParameterFetcher;
import org.apache.shindig.social.core.util.BeanJsonConverter;
import org.apache.shindig.social.core.util.BeanXStreamConverter;
+import org.apache.shindig.social.core.util.ContainerConf;
+import org.apache.shindig.social.core.util.JsonContainerConf;
import org.apache.shindig.social.opensocial.service.BeanConverter;
import org.apache.shindig.social.opensocial.service.DataServiceServletFetcher;
@@ -44,6 +46,8 @@
BeanXStreamConverter.class);
bind(BeanConverter.class).annotatedWith(Names.named("shindig.bean.converter.json")).to(
BeanJsonConverter.class);
+ bind(ContainerConf.class).to(JsonContainerConf.class);
+
}
}
Modified:
incubator/shindig/trunk/java/social-api/src/test/java/org/apache/shindig/social/opensocial/service/ActivityHandlerTest.java
URL:
http://svn.apache.org/viewvc/incubator/shindig/trunk/java/social-api/src/test/java/org/apache/shindig/social/opensocial/service/ActivityHandlerTest.java?rev=736777&r1=736776&r2=736777&view=diff
==============================================================================
---
incubator/shindig/trunk/java/social-api/src/test/java/org/apache/shindig/social/opensocial/service/ActivityHandlerTest.java
(original)
+++
incubator/shindig/trunk/java/social-api/src/test/java/org/apache/shindig/social/opensocial/service/ActivityHandlerTest.java
Thu Jan 22 12:21:38 2009
@@ -25,6 +25,8 @@
import org.apache.shindig.social.EasyMockTestCase;
import org.apache.shindig.social.core.model.ActivityImpl;
import org.apache.shindig.social.core.util.BeanJsonConverter;
+import org.apache.shindig.social.core.util.ContainerConf;
+import org.apache.shindig.social.core.util.JsonContainerConf;
import org.apache.shindig.social.opensocial.model.Activity;
import org.apache.shindig.social.opensocial.spi.ActivityService;
import org.apache.shindig.social.opensocial.spi.CollectionOptions;
@@ -60,8 +62,8 @@
token.setAppId("appId");
converter = EasyMock.createMock(BeanJsonConverter.class);
activityService = EasyMock.createMock(ActivityService.class);
-
- handler = new ActivityHandler(activityService);
+ ContainerConf containerConf = new JsonContainerConf();
+ handler = new ActivityHandler(activityService, containerConf);
}
@Override
Modified:
incubator/shindig/trunk/java/social-api/src/test/java/org/apache/shindig/social/opensocial/service/AppDataHandlerTest.java
URL:
http://svn.apache.org/viewvc/incubator/shindig/trunk/java/social-api/src/test/java/org/apache/shindig/social/opensocial/service/AppDataHandlerTest.java?rev=736777&r1=736776&r2=736777&view=diff
==============================================================================
---
incubator/shindig/trunk/java/social-api/src/test/java/org/apache/shindig/social/opensocial/service/AppDataHandlerTest.java
(original)
+++
incubator/shindig/trunk/java/social-api/src/test/java/org/apache/shindig/social/opensocial/service/AppDataHandlerTest.java
Thu Jan 22 12:21:38 2009
@@ -24,6 +24,8 @@
import org.apache.shindig.common.util.ImmediateFuture;
import org.apache.shindig.social.EasyMockTestCase;
import org.apache.shindig.social.core.util.BeanJsonConverter;
+import org.apache.shindig.social.core.util.ContainerConf;
+import org.apache.shindig.social.core.util.JsonContainerConf;
import org.apache.shindig.social.opensocial.spi.AppDataService;
import org.apache.shindig.social.opensocial.spi.DataCollection;
import org.apache.shindig.social.opensocial.spi.GroupId;
@@ -59,8 +61,8 @@
token = new FakeGadgetToken();
converter = EasyMock.createMock(BeanJsonConverter.class);
appDataService = EasyMock.createMock(AppDataService.class);
-
- handler = new AppDataHandler(appDataService);
+ ContainerConf containerConf = new JsonContainerConf();
+ handler = new AppDataHandler(appDataService, containerConf);
}
@Override
Modified:
incubator/shindig/trunk/java/social-api/src/test/java/org/apache/shindig/social/opensocial/service/PersonHandlerTest.java
URL:
http://svn.apache.org/viewvc/incubator/shindig/trunk/java/social-api/src/test/java/org/apache/shindig/social/opensocial/service/PersonHandlerTest.java?rev=736777&r1=736776&r2=736777&view=diff
==============================================================================
---
incubator/shindig/trunk/java/social-api/src/test/java/org/apache/shindig/social/opensocial/service/PersonHandlerTest.java
(original)
+++
incubator/shindig/trunk/java/social-api/src/test/java/org/apache/shindig/social/opensocial/service/PersonHandlerTest.java
Thu Jan 22 12:21:38 2009
@@ -22,6 +22,8 @@
import org.apache.shindig.social.EasyMockTestCase;
import org.apache.shindig.social.ResponseError;
import org.apache.shindig.social.core.model.PersonImpl;
+import org.apache.shindig.social.core.util.ContainerConf;
+import org.apache.shindig.social.core.util.JsonContainerConf;
import org.apache.shindig.social.opensocial.model.Person;
import org.apache.shindig.social.opensocial.spi.CollectionOptions;
import org.apache.shindig.social.opensocial.spi.GroupId;
@@ -71,8 +73,8 @@
super.setUp();
token = new FakeGadgetToken();
personService = EasyMock.createMock(PersonService.class);
-
- handler = new PersonHandler(personService);
+ ContainerConf containerConf = new JsonContainerConf();
+ handler = new PersonHandler(personService, containerConf);
}
@Override
Modified:
incubator/shindig/trunk/java/social-api/src/test/java/org/apache/shindig/social/opensocial/spi/DataRequestHandlerTest.java
URL:
http://svn.apache.org/viewvc/incubator/shindig/trunk/java/social-api/src/test/java/org/apache/shindig/social/opensocial/spi/DataRequestHandlerTest.java?rev=736777&r1=736776&r2=736777&view=diff
==============================================================================
---
incubator/shindig/trunk/java/social-api/src/test/java/org/apache/shindig/social/opensocial/spi/DataRequestHandlerTest.java
(original)
+++
incubator/shindig/trunk/java/social-api/src/test/java/org/apache/shindig/social/opensocial/spi/DataRequestHandlerTest.java
Thu Jan 22 12:21:38 2009
@@ -21,6 +21,8 @@
import org.apache.shindig.common.util.ImmediateFuture;
import org.apache.shindig.social.ResponseError;
+import org.apache.shindig.social.core.util.ContainerConf;
+import org.apache.shindig.social.core.util.JsonContainerConf;
import org.apache.shindig.social.opensocial.service.DataRequestHandler;
import org.apache.shindig.social.opensocial.service.RequestItem;
import org.apache.shindig.social.opensocial.service.RestfulRequestItem;
@@ -35,7 +37,8 @@
@Override
protected void setUp() throws Exception {
super.setUp();
- drh = new DataRequestHandler() {
+ ContainerConf containerConf = new JsonContainerConf();
+ drh = new DataRequestHandler(containerConf) {
@Override
protected Future<?> handleDelete(RequestItem request) {
return ImmediateFuture.newInstance("DELETE");