Fix: Use Decimal Writer for CBOR Spatial Geography Point

[no1melman]

Thank you for submitting this pull request! We appreciate you spending the time to work on these changes.

What is the motivation?

Current bug where surreal is expecting an array of two decimal values

What does this change do?

Ensures the converter for the Geography Point writes two decimal values

What is your testing strategy?

Yes

Is this related to any issues?

#175

Have you read the Contributing Guidelines?

• I have read the Contributing Guidelines

[Odonno]

Thank you for your contribution @no1melman

I can see there is indeed a change from decimal (as a string) to f64 type. Can you change the Write and Read methods so that each use the DoubleConverter?

Note that GeometryPointConverter also follows the same logic. Can you also make the change for this converter?

[no1melman]

@Odonno all done

I can't - for some reason - build the rust-embedded stuff, so I can't actually build the whole project and run tests...

Warning: corrupt .drectve at end of def file␍
          C:\Users\callu\code\my-surrealdb-net\fix-cbor-spatial-point\rust-embedded\target\debug\deps\liblibrocksdb_sys-98402df79e65f43b.rlib(34500ee411ddd5f5-convenience.o):convenience.cc:(.text$_ZTWN7rocksdb10perf_levelE[_ZTWN7rocksdb10perf_levelE]+0x15): relocation truncated to fit: IMAGE_REL_AMD64_REL32 against undefined symbol `TLS init function for rocksdb::perf_level'␍
          C:\Users\callu\code\my-surrealdb-net\fix-cbor-spatial-point\rust-embedded\target\debug\deps\liblibrocksdb_sys-98402df79e65f43b.rlib(34500ee411ddd5f5-memtable.o):memtable.cc:(.text$_ZTWN7rocksdb15ConcurrentArena9tls_cpuidE[_ZTWN7rocksdb15ConcurrentArena9tls_cpuidE]+0x15): relocation truncated to fit: IMAGE_REL_AMD64_REL32 against undefined symbol `TLS init function for rocksdb::ConcurrentArena::tls_cpuid'␍
          collect2.exe: error: ld returned 1 exit status

On Win 11, x64, rust version: 1.84.1 (cargo the same).

[Odonno]

@Odonno all done

Perfect!

I can't - for some reason - build the rust-embedded stuff, so I can't actually build the whole project and run tests...

Warning: corrupt .drectve at end of def file␍
          C:\Users\callu\code\my-surrealdb-net\fix-cbor-spatial-point\rust-embedded\target\debug\deps\liblibrocksdb_sys-98402df79e65f43b.rlib(34500ee411ddd5f5-convenience.o):convenience.cc:(.text$_ZTWN7rocksdb10perf_levelE[_ZTWN7rocksdb10perf_levelE]+0x15): relocation truncated to fit: IMAGE_REL_AMD64_REL32 against undefined symbol `TLS init function for rocksdb::perf_level'␍
          C:\Users\callu\code\my-surrealdb-net\fix-cbor-spatial-point\rust-embedded\target\debug\deps\liblibrocksdb_sys-98402df79e65f43b.rlib(34500ee411ddd5f5-memtable.o):memtable.cc:(.text$_ZTWN7rocksdb15ConcurrentArena9tls_cpuidE[_ZTWN7rocksdb15ConcurrentArena9tls_cpuidE]+0x15): relocation truncated to fit: IMAGE_REL_AMD64_REL32 against undefined symbol `TLS init function for rocksdb::ConcurrentArena::tls_cpuid'␍
          collect2.exe: error: ld returned 1 exit status

On Win 11, x64, rust version: 1.84.1 (cargo the same).

You can run the tests for remote (HTTP, WS) by disabling the embedded mode. See this section in the readme file: https://github.com/surrealdb/surrealdb.net?tab=readme-ov-file#embedded-mode

If serialization works for remote, it should work for embedded too.

[no1melman]

@Odonno I've updated the affected tests, they all work now.... bar 1 test which had a depth problem

GeographyMultiPolygonConverterTests.cs:58
Maximum recursion depth of 5 was reached.  Increase MaxDepth on AssertionScope or AssertionOptions to get more details.

[Odonno]

This is weird. Can you try adjusting the settings like this:

AssertionOptions.FormattingOptions.MaxDepth = 10;

You can increase this value up to 100 in case it's still failing.

Does it fix the problem?