Follow the https://github.com/neo-project/non-native-contracts/blob/14f43ba8cf169323b61c23a3a701ac77d9a4e3eb/src/NameService/NameService.cs#L69.

Signed-off-by: Anna Shaleva <[email protected]>

Signed-off-by: Anna Shaleva <[email protected]>

Signed-off-by: Anna Shaleva <[email protected]>

Signed-off-by: Anna Shaleva <[email protected]>

Reuse getAllRecords for GetAllRecords.

Signed-off-by: Anna Shaleva <[email protected]>

Do not include CNAME to the resulting list if we're looking for another
record type. If it's CNAME than it must be resolved.

Signed-off-by: Anna Shaleva <[email protected]>

Do not move parts of SetRecord/AddRecord to a separate functions,
it makes the contract code more complicated. Also, improve
documentation a bit.

Signed-off-by: Anna Shaleva <[email protected]>

a.b.c.d can be an "a.b" record of "c.d" and can be "a" record of "b.c.d", the
first one would fail to setRecord currently:

  at instruction 4287 (THROW): unhandled exception: "parent domain has expired"

which is a 5758c84 regression.

Signed-off-by: Roman Khimov <[email protected]>

Don't repeat it each time.

Signed-off-by: Anna Shaleva <[email protected]>

It doesn't save VM opcodes, but allows to keep record key creation
logic in a single place which prevents the code from bugs appearance.

Signed-off-by: Anna Shaleva <[email protected]>

Less code repeating.

Signed-off-by: Anna Shaleva <[email protected]>

Signed-off-by: Anna Shaleva <[email protected]>

Signed-off-by: Anna Shaleva <[email protected]>

If conflicting records '*.domain' are present on new domain
registration, then `isAvailable` should return false for this
domain. Ref.
f25296b.

Signed-off-by: Anna Shaleva <[email protected]>

Signed-off-by: Anna Shaleva <[email protected]>

These numbers are pretty much arbitrary, but we currently don't need more than
that.

Signed-off-by: Roman Khimov <[email protected]>

Part of #411.

Signed-off-by: Roman Khimov <[email protected]>

1. splitAndCheck doesn't need allowMultipleFragments since there is exactly
   one user of it and it checks for the number of fragments anyway.
2. We can panic right inside of it with a bit more details, nil fragments are
   never valid.
3. Except for one case in checkRecord where the error must be different, so
   safeSplitAndCheck is introduced.

Fixes #411.

Signed-off-by: Roman Khimov <[email protected]>

Sometimes 10 is exactly the value, so changing it doesn't make the ID different:

              	Error Trace:	/home/runner/go/pkg/mod/github.com/nspcc-dev/[email protected]/pkg/neotest/basic.go:218
        	            				/home/runner/go/pkg/mod/github.com/nspcc-dev/[email protected]/pkg/neotest/client.go:108
        	            				/home/runner/work/neofs-contract/neofs-contract/tests/container_test.go:237
        	Error:      	Not equal:
        	            	expected: 0x1
        	            	actual  : 0x2
        	Test:       	TestContainerPut/standard_deploy/with_nice_names/register_in_advance
        	Messages:   	at instruction 2149 (THROW): unhandled exception: "container was previously deleted"

Signed-off-by: Roman Khimov <[email protected]>