Skip to content

Commit

Permalink
Wrote better unit tests for array serialization. Detected and fixed o…
Browse files Browse the repository at this point in the history
…ne edge case.
  • Loading branch information
rstam committed Nov 23, 2010
1 parent 62b72c4 commit 3a4b30d
Show file tree
Hide file tree
Showing 4 changed files with 272 additions and 141 deletions.
3 changes: 2 additions & 1 deletion Bson/Serialization/BsonSerializer.cs
Expand Up @@ -252,7 +252,8 @@ bool serializeIdFirst
return;
}

var serializer = LookupSerializer(value.GetType());
var actualType = (value == null) ? nominalType : value.GetType();
var serializer = LookupSerializer(actualType);
serializer.Serialize(bsonWriter, nominalType, value, serializeIdFirst);
}

Expand Down
2 changes: 1 addition & 1 deletion BsonUnitTests/BsonUnitTests.csproj
Expand Up @@ -117,7 +117,7 @@
<Compile Include="ObjectModel\BsonValueCompareToTests.cs" />
<Compile Include="Properties\AssemblyInfo.cs" />
<Compile Include="DefaultSerializer\BsonDefaultSerializerTests.cs" />
<Compile Include="DefaultSerializer\SerializeArrayTests.cs" />
<Compile Include="DefaultSerializer\Serializers\ArraySerializerTests.cs" />
<Compile Include="DefaultSerializer\SerializeEnumTests.cs" />
<Compile Include="DefaultSerializer\Serializers\GenericEnumerableSerializerTests.cs" />
<Compile Include="DefaultSerializer\Serializers\NetPrimitiveSerializerTests.cs" />
Expand Down
139 changes: 0 additions & 139 deletions BsonUnitTests/DefaultSerializer/SerializeArrayTests.cs

This file was deleted.

0 comments on commit 3a4b30d

Please sign in to comment.