/*
 * Copyright (C) 2011 Benoit GUEROUT <bguerout at gmail dot com> and Yves AMSELLEM <amsellem dot yves at gmail dot com>
 *
 * Licensed under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.
 * You may obtain a copy of the License at
 *
 *     http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */

package org.jongo;


import com.mongodb.WriteConcern;
import junit.framework.Assert;
import org.bson.types.ObjectId;
import org.jongo.marshall.jackson.oid.Id;
import org.jongo.model.ExternalFriend;
import org.jongo.util.JongoTestCase;
import org.junit.After;
import org.junit.Before;
import org.junit.Test;

import static org.assertj.core.api.Assertions.assertThat;

public class AnnotationsMisusedTest extends JongoTestCase {

    private MongoCollection collection;

    @Before
    public void setUp() throws Exception {
        collection = createEmptyCollection("friends");
    }

    @After
    public void tearDown() throws Exception {
        dropCollection("friends");
    }

    @Test
    public void savingAPojoWithAnEmptyStringCustomId() throws Exception {

        ExternalFriend friend = ExternalFriend.createFriendWithoutId("Robert");

        collection.withWriteConcern(WriteConcern.SAFE).save(friend);

        ExternalFriend externalFriend = collection.findOne().as(ExternalFriend.class);
        assertThat(ObjectId.isValid(externalFriend.getId())).isTrue();//mapped because this is a String
    }

    @Test
    public void savingAPojoWithAnEmptyCustomIntegerId() throws Exception {

        WithIntegerId custom = new WithIntegerId();

        collection.withWriteConcern(WriteConcern.SAFE).save(custom);

        try {
            collection.findOne().as(WithIntegerId.class);
            Assert.fail("Should not be able to unmarshall an ObjectId into an Integer");
        } catch (Exception e) {
        }
    }

    private static class WithIntegerId {
        @Id
        private Integer id;
    }
}

@@ -1678,32 +1678,202 @@
 alFriend.class);
+%0A%0A        /*%0A        * Works because Bson4jackson try to convert ObjectId into String%0A        * see com.fasterxml.jackson.databind.deser.std.StringDeserializer%0A        */
 %0A        assertT
@@ -1931,41 +1931,8 @@
 e();
-//mapped because this is a String
 %0A   

Add comments about how Bson4Jackson deserializes ObjectIs