Move old bson package (#4177)

.golangci.yml

@@ -26,12 +26,12 @@ linters-settings:
             desc: use `github.com/jackc/pgx/v5` package instead
           - pkg: github.com/jackc/pgx/v4
             desc: use `github.com/jackc/pgx/v5` package instead
-      bson:
+      oldbson:
         files:
           - $all
-          - "!**/internal/bson2/*_test.go"
+          - "!**/internal/bson2/bson2_test.go"
         deny:
-          - pkg: github.com/FerretDB/FerretDB/internal/bson$
+          - pkg: github.com/FerretDB/FerretDB/internal/bson/oldbson$
       bson2:
         files:
           - $all

internal/bson2/bson2_test.go

@@ -24,8 +24,8 @@ import (
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"
 
-	"github.com/FerretDB/FerretDB/internal/bson"
 	"github.com/FerretDB/FerretDB/internal/bson2"
+	"github.com/FerretDB/FerretDB/internal/bson2/oldbson"
 	"github.com/FerretDB/FerretDB/internal/types"
 	"github.com/FerretDB/FerretDB/internal/util/must"
 	"github.com/FerretDB/FerretDB/internal/util/testutil"
@@ -587,9 +587,9 @@ var decodeTestCases = []decodeTestCase{
 func TestNormal(t *testing.T) {
 	for _, tc := range normalTestCases {
 		t.Run(tc.name, func(t *testing.T) {
-			t.Run("bson", func(t *testing.T) {
+			t.Run("oldbson", func(t *testing.T) {
 				t.Run("ReadFrom", func(t *testing.T) {
-					var doc bson.Document
+					var doc oldbson.Document
 					buf := bufio.NewReader(bytes.NewReader(tc.raw))
 					err := doc.ReadFrom(buf)
 					require.NoError(t, err)
@@ -603,7 +603,7 @@ func TestNormal(t *testing.T) {
 				})
 
 				t.Run("MarshalBinary", func(t *testing.T) {
-					doc, err := bson.ConvertDocument(tc.tdoc)
+					doc, err := oldbson.ConvertDocument(tc.tdoc)
 					require.NoError(t, err)
 
 					raw, err := doc.MarshalBinary()
@@ -688,9 +688,9 @@ func TestDecode(t *testing.T) {
 		require.NotNil(t, tc.decodeDeepErr, "invalid test case %q", tc.name)
 
 		t.Run(tc.name, func(t *testing.T) {
-			t.Run("bson", func(t *testing.T) {
+			t.Run("oldbson", func(t *testing.T) {
 				t.Run("ReadFrom", func(t *testing.T) {
-					var doc bson.Document
+					var doc oldbson.Document
 					buf := bufio.NewReader(bytes.NewReader(tc.raw))
 					err := doc.ReadFrom(buf)
 
@@ -746,9 +746,9 @@ func TestDecode(t *testing.T) {
 func BenchmarkDocument(b *testing.B) {
 	for _, tc := range normalTestCases {
 		b.Run(tc.name, func(b *testing.B) {
-			b.Run("bson", func(b *testing.B) {
+			b.Run("oldbson", func(b *testing.B) {
 				b.Run("ReadFrom", func(b *testing.B) {
-					var doc bson.Document
+					var doc oldbson.Document
 					var buf *bufio.Reader
 					var err error
 					br := bytes.NewReader(tc.raw)
@@ -768,7 +768,7 @@ func BenchmarkDocument(b *testing.B) {
 				})
 
 				b.Run("MarshalBinary", func(b *testing.B) {
-					doc, err := bson.ConvertDocument(tc.tdoc)
+					doc, err := oldbson.ConvertDocument(tc.tdoc)
 					require.NoError(b, err)
 
 					var raw []byte
@@ -956,7 +956,7 @@ func testRawDocument(t *testing.T, rawDoc bson2.RawDocument) {
 		br := bytes.NewReader(rawDoc)
 		bufr := bufio.NewReader(br)
 
-		var doc1 bson.Document
+		var doc1 oldbson.Document
 		err1 := doc1.ReadFrom(bufr)
 
 		if err1 != nil {
@@ -994,7 +994,7 @@ func testRawDocument(t *testing.T, rawDoc bson2.RawDocument) {
 
 		// encode
 
-		doc1e, err := bson.ConvertDocument(tdoc1)
+		doc1e, err := oldbson.ConvertDocument(tdoc1)
 		require.NoError(t, err)
 
 		doc2e, err := bson2.ConvertDocument(tdoc2)

internal/bson/array.go → internal/bson2/oldbson/array.go

@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-package bson
+package oldbson
 
 import (
 	"bufio"

internal/bson/array_test.go → internal/bson2/oldbson/array_test.go

@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-package bson
+package oldbson
 
 import (
 	"testing"

internal/bson/binary.go → internal/bson2/oldbson/binary.go

@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-package bson
+package oldbson
 
 import (
 	"bufio"

internal/bson/binary_test.go → internal/bson2/oldbson/binary_test.go

@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-package bson
+package oldbson
 
 import (
 	"testing"

internal/bson/bool.go → internal/bson2/oldbson/bool.go

@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-package bson
+package oldbson
 
 import (
 	"bufio"

internal/bson/bool_test.go → internal/bson2/oldbson/bool_test.go

@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-package bson
+package oldbson
 
 import (
 	"testing"

internal/bson/cstring.go → internal/bson2/oldbson/cstring.go

@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-package bson
+package oldbson
 
 import (
 	"bufio"

internal/bson/cstring_test.go → internal/bson2/oldbson/cstring_test.go

@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-package bson
+package oldbson
 
 import (
 	"testing"

internal/bson/datetime.go → internal/bson2/oldbson/datetime.go

@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-package bson
+package oldbson
 
 import (
 	"bufio"

internal/bson/datetime_test.go → internal/bson2/oldbson/datetime_test.go

@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-package bson
+package oldbson
 
 import (
 	"testing"

internal/bson/document.go → internal/bson2/oldbson/document.go

@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-package bson
+package oldbson
 
 import (
 	"bufio"

internal/bson/document_test.go → internal/bson2/oldbson/document_test.go

@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-package bson
+package oldbson
 
 import (
 	"testing"

internal/bson/double.go → internal/bson2/oldbson/double.go

@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-package bson
+package oldbson
 
 import (
 	"bufio"

internal/bson/double_test.go → internal/bson2/oldbson/double_test.go

@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-package bson
+package oldbson
 
 import (
 	"math"

internal/bson/int32.go → internal/bson2/oldbson/int32.go

@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-package bson
+package oldbson
 
 import (
 	"bufio"

internal/bson/int32_test.go → internal/bson2/oldbson/int32_test.go

@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-package bson
+package oldbson
 
 import (
 	"math"

internal/bson/int64.go → internal/bson2/oldbson/int64.go

@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-package bson
+package oldbson
 
 import (
 	"bufio"

internal/bson/int64_test.go → internal/bson2/oldbson/int64_test.go

@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-package bson
+package oldbson
 
 import (
 	"math"

internal/bson/null.go → internal/bson2/oldbson/null.go

@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-package bson
+package oldbson
 
 import (
 	"bufio"

internal/bson/object_id.go → internal/bson2/oldbson/object_id.go

@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-package bson
+package oldbson
 
 import (
 	"bufio"

internal/bson/object_id_test.go → internal/bson2/oldbson/object_id_test.go

@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-package bson
+package oldbson
 
 import (
 	"testing"

internal/bson/bson.go → internal/bson2/oldbson/oldbson.go

@@ -12,37 +12,8 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-// Package bson provides converters from/to BSON for built-in and `types` types.
-//
-// See contributing guidelines and documentation for package `types` for details.
-//
-// # Mapping
-//
-// Composite types
-//
-//	Alias      types package    bson package
-//
-//	object     *types.Document  *bson.Document
-//	array      *types.Array     *bson.arrayType
-//
-// Scalar types
-//
-//	Alias      types package    bson package
-//
-//	double     float64          *bson.doubleType
-//	string     string           *bson.stringType
-//	binData    types.Binary     *bson.binaryType
-//	objectId   types.ObjectID   *bson.objectIDType
-//	bool       bool             *bson.boolType
-//	date       time.Time        *bson.dateTimeType
-//	null       types.NullType   *bson.nullType
-//	regex      types.Regex      *bson.regexType
-//	int        int32            *bson.int32Type
-//	timestamp  types.Timestamp  *bson.timestampType
-//	long       int64            *bson.int64Type
-//
-//nolint:dupword // false positive
-package bson
+// Package oldbson contains old implementation of BSON that is used for cross-testing only.
+package oldbson
 
 import (
 	"bufio"

internal/bson/bson_test.go → internal/bson2/oldbson/oldbson_test.go

@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-package bson
+package oldbson
 
 import (
 	"bufio"

internal/bson/regex.go → internal/bson2/oldbson/regex.go

@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-package bson
+package oldbson
 
 import (
 	"bufio"

internal/bson/regex_test.go → internal/bson2/oldbson/regex_test.go

@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-package bson
+package oldbson
 
 import (
 	"testing"

internal/bson/string.go → internal/bson2/oldbson/string.go

@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-package bson
+package oldbson
 
 import (
 	"bufio"

internal/bson/string_test.go → internal/bson2/oldbson/string_test.go

@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-package bson
+package oldbson
 
 import (
 	"testing"

internal/bson/tag.go → internal/bson2/oldbson/tag.go

@@ -12,9 +12,9 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-package bson
+package oldbson
 
-//go:generate ../../bin/stringer -linecomment -type tag
+//go:generate ../../../bin/stringer -linecomment -type tag
 
 type tag byte