Store multiple documents with StoreDoc bug

[davidst]

Describe the bug
In the Wolverine V3.4.0 StoreDoc<T> now stores an object[] with one element of type T[].

To Reproduce

public recod Foo;

public void MartenOps_store_test()
{
    var docs = new[] { new Foo(), new Foo() };

    var storeOp = MartenOps.Store<Foo>(docs);

    storeOp.Documents.ShouldBe(docs);
}

Expected behavior
StoreDoc<T> should store an object[] with elements of type T.

Additional context

• The bug was introduced in this commit.
• The initial change from this commit introduced a breaking change in a minor version.

[haefele]

Fix is already prepared in #1158

[davidst]

The bug is still there in the public StoreManyDocs(params T[] documents) : base(documents) constructor calling the base constructor, because there is no implicit conversion from T[] to object[].
A quick fix would be to first cast the T elements to object then pass the array to the base constructor public StoreManyDocs(params T[] documents) : base(documents.Cast<Object>().ToArray()).

[jay-zahiri]

I missed this issue while reorganizing the code; it's fixed now. Here's a quick demo

[davidst]

Yes, that fixes the issue, but it's missing the params keyword which would make the usage nicer.

[jeremydmiller]

Fixes coming late today / early tomorrow