huaweicloud: Modernize record conversion

[TomOnTime]

Dear @huihuimoe. I could use your help. I'm doing upgrades to the code base and I have no way to test your provider. Please run the integration test and let me know the result. How to run integration tests is here: https://docs.dnscontrol.org/developer-info/integration-tests

Summary

• replace legacy record construction with current domain-aware factories
• replace the combined legacy getter with typed RDATA output
• preserve raw TXT and provider metadata handling
• add focused RRset conversion tests

Validation

• ../../bin/is_modern.sh (Steps 1-7 clean)
• go test ./providers/huaweicloud
• bin/generate-all.sh
• go test ./...

[huihuimoe]

The integration test failed.
It looks like there's a difference in how double quotes in TXT records are handled before and after the change.

log:

=== RUN   TestDNSProviders/dnscontroltest-hwc.org/09:TXT:Create_TXT
    helpers_integration_test.go:250:
        + CREATE testtxt.dnscontroltest-hwc.org TXT "simple" weight=1 line=default_view key= ttl=300
    helpers_integration_test.go:272: Expected 0 corrections on second run, but found 1.
    helpers_integration_test.go:274: UNEXPECTED #0: ± MODIFY testtxt.dnscontroltest-hwc.org TXT ("\"simple\"" ttl=300) -> ("simple" ttl=300)

P.S: I haven’t had time to investigate the exact difference yet, but I’ll take a closer look later.

[TomOnTime]

Take a look at convert.go. I think it should use NewRecordConfigParse() for TXT records too.

                if recType == "TXT" {
                        rc, err = dc.NewRecordConfig(label, ttl, recType, value)
                } else {
                        rc, err = dc.NewRecordConfigParse(label, ttl, recType, value)
                }

I've pushed a change to use NewRecordConfigParse() in both situations. PTAL

Thanks!

[huihuimoe]

Testing Profile="HUAWEICLOUD" (TYPE="HUAWEICLOUD")
    provider_test.go:153: Skipping. Deduplication logic is not implemented for this provider.
--- SKIP: TestDuplicateNameservers (1.13s)
PASS
ok      github.com/DNSControl/dnscontrol/v5/integrationTest     601.171s

LGTM, now PASS integrationTest

[TomOnTime]

Thanks!